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

name the FlagSet for the shell and add a version flag #2501

Merged
merged 1 commit into from
May 28, 2015

Conversation

neonstalwart
Copy link
Contributor

i don't know if you want this but it took very little effort and i thought it was useful to have another way to get the version of the install

> influx -version
InfluxDB shell 0.9

> influx -h
Usage of InfluxDB REPL version 0.9:
  -database="": database to connect to the server.
  -dump=false: dump the contents of the given database to stdout
  -execute="": Execute command and quit.
  -host="localhost": influxdb host to connect to
  -output="column": format specifies the format of the server responses:  json, csv, or column
  -password="": password to connect to the server.  Leaving blank will prompt for password (--password="")
  -port=8086: influxdb port to connect to
  -username="": username to connect to the server.
  -version=false: Displays the InfluxDB version.

@otoolep
Copy link
Contributor

otoolep commented May 6, 2015

Thanks @neonstalwart -- you know you can do influxd version, right?

@otoolep
Copy link
Contributor

otoolep commented May 6, 2015

$ ./influxd version
[srvr] 2015/05/06 12:29:29 InfluxDB v0.9 (git: unknown)

@neonstalwart
Copy link
Contributor Author

@otoolep yes, i discovered that while adding this 😄 but for some reason i couldn't find it when i wanted it before i did this

@neonstalwart
Copy link
Contributor Author

so, the one new thing i added here is a better description for the FlagSet. that's got to be worth something 😉

@corylanou
Copy link
Contributor

@neonstalwart the usage was recently redone to be clearer and more accurate as the basic usage that Go puts out is not accurate.

See #2603

Feel free to rebase and add this new flag there (or update usage to show how to use the existing option that already existed).

Thanks!

@neonstalwart
Copy link
Contributor Author

@corylanou rebased

@corylanou
Copy link
Contributor

+1

@toddboom
Copy link
Contributor

+1

corylanou added a commit that referenced this pull request May 28, 2015
name the FlagSet for the shell and add a version flag
@corylanou corylanou merged commit ae54d1f into influxdata:master May 28, 2015
@neonstalwart neonstalwart deleted the version-flag branch May 28, 2015 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants