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.

Sign up for GitHub

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

Jump to bottom

Add optional flag to gaiad testnet to make config directory of daemon and cli configurable #1992

Closed
4 tasks
philipstanislaus opened this issue Aug 12, 2018 · 0 comments · Fixed by #1993
Closed
4 tasks

Add optional flag to gaiad testnet to make config directory of daemon and cli configurable #1992

philipstanislaus opened this issue Aug 12, 2018 · 0 comments · Fixed by #1993
Assignees
@philipstanislaus
Labels
C:CLI T: UX

Comments

@philipstanislaus
Copy link
Contributor

philipstanislaus commented Aug 12, 2018 •
edited
Loading

Summary

Add optional flag to gaiad testnet to make config directory of daemon (default gaiad) and cli (default gaiacli) configurable.

Problem Definition

Right now, gaiad testnet always creates config files in subfolders called gaiad and gaiacli. For apps build with Cosmos SDK, the naming should be adjustable.

The change would be downwards compatible.

Proposal

Add optional flags to adjust names, will create a PR in a moment PR: #1993 .

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@philipstanislaus philipstanislaus changed the title Add optional flag to gaiad testnet to make name of daemon and cli configurable Add optional flag to gaiad testnet to make config directory of daemon and cli configurable Aug 12, 2018
philipstanislaus added a commit to philipstanislaus/cosmos-sdk that referenced this issue Aug 12, 2018
@philipstanislaus
Add optional flag to gaiad testnet to make config directory of daem…
7d70300 
…on and cli configurable, fixes cosmos#1992
@philipstanislaus philipstanislaus mentioned this issue Aug 12, 2018
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philipstanislaus philipstanislaus

Labels
C:CLI T: UX
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add optional flag to gaiad testnet to make config directory of daemon… philipstanislaus/cosmos-sdk
2 participants
@ValarDragon @philipstanislaus