Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing proto path in config.yml Reverts to Default with ignite g proto-go #3979

Closed
taryune opened this issue Feb 20, 2024 · 0 comments · Fixed by #4071
Closed

Changing proto path in config.yml Reverts to Default with ignite g proto-go #3979

taryune opened this issue Feb 20, 2024 · 0 comments · Fixed by #4071
Assignees
Labels
component:configs type:error Something isn't working

Comments

@taryune
Copy link

taryune commented Feb 20, 2024

Describe the bug
Referring this docs, I changed proto directory to myproto
but it doesn't seems to be applied.

To reproduce
Steps to reproduce the behavior:

  1. ignite s chain mars
  2. mv proto myproto
  3. update proto path to myproto in config.yml
  4. update directory to myproto from proto in buf.work.yml
  5. ignite g proto-go
    then, running initialization
Do you want to proceed without committing your saved changes: y
Now ignite supports the `buf.build` (https://buf.build) registry to manage the protobuf dependencies. The embed protoc binary was deprecated and, your blockchain is still using it. Would you like to upgrade and add the `buf.build` config files to `proto/` folder: y

🎉 buf.build files added:

/Users/taru/dev/sandbox/mars/proto/buf.gen.gogo.yaml
/Users/taru/dev/sandbox/mars/proto/buf.gen.pulsar.yaml
/Users/taru/dev/sandbox/mars/proto/buf.gen.sta.yaml
/Users/taru/dev/sandbox/mars/proto/buf.gen.swagger.yaml
/Users/taru/dev/sandbox/mars/proto/buf.gen.ts.yaml
/Users/taru/dev/sandbox/mars/proto/buf.lock
/Users/taru/dev/sandbox/mars/proto/buf.yaml

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x18 pc=0x101034124]

goroutine 1 [running]:
github.com/ignite/cli/v28/ignite/pkg/cosmosgen.(*generator).resolveIncludes(0x1400036c000, {0x10312a600, 0x140000daff0}, {0x14001896700, 0x3b})
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/pkg/cosmosgen/generate.go:277 +0x164
github.com/ignite/cli/v28/ignite/pkg/cosmosgen.(*generator).setup(0x1400036c000, {0x10312a600, 0x140000daff0})
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/pkg/cosmosgen/generate.go:188 +0xc18
github.com/ignite/cli/v28/ignite/pkg/cosmosgen.Generate({0x10312a600, 0x140000daff0}, {{0x14001415620?, 0x11?}, {0x102ba2f88?, 0x1?}}, {0x140013b6fa0, 0x1c}, {0x14000ba6888, 0x7}, ...)
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/pkg/cosmosgen/cosmosgen.go:169 +0x3c0
github.com/ignite/cli/v28/ignite/services/chain.(*Chain).Generate(0x140005d0500, {0x10312a600, 0x140000daff0}, {{0x14001415620?, 0x0?}, {0x102ba2f88?, 0x1400148fa58?}}, 0x1030f75e8, {0x0, 0x0, ...})
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/services/chain/generate.go:298 +0xf4c
github.com/ignite/cli/v28/ignite/cmd.generateGoHandler(0x14000b0c600, {0x107811d00?, 0x4?, 0x101e9e4ae?})
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/cmd/generate_go.go:47 +0x258
github.com/spf13/cobra.(*Command).execute(0x14000b0c600, {0x107811d00, 0x0, 0x0})
        /home/runner/go/pkg/mod/github.com/spf13/[email protected]/command.go:983 +0x840
github.com/spf13/cobra.(*Command).ExecuteC(0x14000ace000)
        /home/runner/go/pkg/mod/github.com/spf13/[email protected]/command.go:1115 +0x344
github.com/spf13/cobra.(*Command).Execute(...)
        /home/runner/go/pkg/mod/github.com/spf13/[email protected]/command.go:1039
github.com/spf13/cobra.(*Command).ExecuteContext(0x140006a7ea0?, {0x10312a600?, 0x140000daff0?})
        /home/runner/go/pkg/mod/github.com/spf13/[email protected]/command.go:1032 +0x48
main.run()
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/cmd/ignite/main.go:45 +0x188
main.main()
        /home/runner/work/cli/cli/go/src/github.com/ignite/cli/ignite/cmd/ignite/main.go:23 +0x1c

What version are you using?
Ignite CLI version: v28.2.0
Ignite CLI build date: 2024-02-06T11:21:56Z
Ignite CLI source hash: af25183
Ignite CLI config version: v1

Provide the output of the ignite version command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:configs type:error Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants