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

Home flag gets ignored when running help #11508

Closed
3 of 4 tasks
maiph opened this issue Mar 30, 2022 · 6 comments · Fixed by #11645
Closed
3 of 4 tasks

Home flag gets ignored when running help #11508

maiph opened this issue Mar 30, 2022 · 6 comments · Fixed by #11645
Assignees

Comments

@maiph
Copy link

maiph commented Mar 30, 2022

Summary of Bug

Running simd --home /data query -h ignores the --home flag and creates ~/.simapp

Version

At commit v0.44.6

Steps to Reproduce

rm ~/.simapp
simd --home /data query -h
ls ~/.simapp

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@maiph
Copy link
Author

maiph commented Mar 31, 2022

I think I remember this working, maybe it was introduced on tendermint v0.35.0 ?

[config] Add --mode flag and config variable. See ADR-52 @dongsam

After writing this I went to check if cosmos-sdk v0.44.6 was at least using tendermint v0.35.0, but it was at v0.34.14 so my guess is not true ...

@maiph
Copy link
Author

maiph commented Mar 31, 2022

Commit v0.46.0-beta1 also has the same problem and running tendermint v0.35.2

@anilcse
Copy link
Collaborator

anilcse commented Mar 31, 2022

I believe queries will be just based on the rpc endpoint. Not sure what query you are testing. It shouldn't consider home flag at all

@maiph
Copy link
Author

maiph commented Mar 31, 2022

But printing the help menu should not need to create the home directory, and if it needs it should honor the home flag.

On my use case I don't let the user write on the home directory and attach a volume on /data, this makes it impossible to ever print the help menu because the cli will always try to create that directory on the user home dir

@anilcse
Copy link
Collaborator

anilcse commented Mar 31, 2022

I think we can remove the home flag for queries. wdyt @AmauryM @robert-zaremba

@alexanderbez
Copy link
Contributor

Yeah, there's no need for it (unless we ready from a client.toml, not sure if we still have that).

@julienrbrt julienrbrt self-assigned this Apr 1, 2022
@amaury1093 amaury1093 moved this to In Progress in Cosmos SDK Maintenance Apr 1, 2022
@julienrbrt julienrbrt moved this from In Progress to In Review in Cosmos SDK Maintenance Apr 13, 2022
@julienrbrt julienrbrt linked a pull request Apr 14, 2022 that will close this issue
21 tasks
@amaury1093 amaury1093 moved this from In Review to In Progress in Cosmos SDK Maintenance Apr 14, 2022
@julienrbrt julienrbrt moved this from In Progress to In Review in Cosmos SDK Maintenance Apr 21, 2022
Repository owner moved this from In Review to Done in Cosmos SDK Maintenance Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants