Enhancement #782

rsb_version changes

Added by J. Moringen over 12 years ago. Updated over 12 years ago.

Status:ResolvedStart date:12/26/2011
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:C++
Target version:-

Description

I think, the following things regarding rsb_version could be changed:
  • Rename to rsb_info (since more information than just the version is displayed)
  • Use of Boost.ProgramOptions
    • Make rsb-cpp depend on Boost.ProgramOptions
    • Move rsb_info to cpp/tools sub-project
  • Display wire-format version

Associated revisions

Revision c0622c69
Added by J. Moringen over 12 years ago

Use Boost.ProgramOptions in apps/rsbversion/rsbversion.cpp
fixes #782
  • apps/rsbversion/rsbversion.cpp: added commandline flags (via
    Boost.ProgramOptions) which control display of individual
    information items
  • CMakeLists.txt: added Boost.Program options to find_package(Boost)

History

#1 Updated by J. Moringen over 12 years ago

  • Category set to C++

#2 Updated by J. Moringen over 12 years ago

  • Tracker changed from Bug to Enhancement

#3 Updated by J. Wienke over 12 years ago

I'm not sure that it is a good idea to move this binary to a subproject, which needs to be compiled separatey. Then, there is no more simple way to find out the version of RSB with only the basic pakage being compiled and installed.

#4 Updated by J. Moringen over 12 years ago

  • Status changed from Feedback to In Progress
  • % Done changed from 0 to 30

Johannes complained that rsb_version does not understand --version and since there are no other objections or suggestions, I will just let RSB depend on Boost.ProgramOptions. I have a feeling that this may cause problem, though. But we'll see.

#5 Updated by J. Wienke over 12 years ago

--help ;)

program options is ok to my mind.

#6 Updated by J. Moringen over 12 years ago

Yeah, right. rsb_version --version will be supported for conformance reasons, though.

#7 Updated by J. Moringen over 12 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 30 to 100

Applied in changeset r3218.

Also available in: Atom PDF