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

Rewrite CLI options handling and output #5

Closed
wants to merge 10 commits into from
Closed

Rewrite CLI options handling and output #5

wants to merge 10 commits into from

Commits on Dec 23, 2013

  1. Rewrite CLI options handling 

    * Use argparse for option handling
* Remove global variables and put them under an argparse namespace
    @amrali
    amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    bf995da View commit details
    Browse the repository at this point in the history

  2. Add logging mechanism 

    * Replace 'print' statements with the appropriate log calls
* Replace sys.stderr calls with the appropriate log calls
* Remove global variable HUPTIME_DEBUG
    @amrali
    amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    7f1047d View commit details
    Browse the repository at this point in the history

  3. Fix typo and $PID example in README

    @JackDanger @amrali
    JackDanger authored and amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    2c723e5 View commit details
    Browse the repository at this point in the history

  4. Update README.md

    @derekchiang @amrali
    derekchiang authored and amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    40c440a View commit details
    Browse the repository at this point in the history

  5. Fix example.

    @amscanne @amrali
    amscanne authored and amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    ba54c48 View commit details
    Browse the repository at this point in the history

  6. Fix header guards (fixes amscanne#2). 

    TIL: Leading underscores followed by a capital letter is
actually a reserved identifier. This replaces the header
guards with a non-reserved identifier.
    @amscanne @amrali
    amscanne authored and amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    9451254 View commit details
    Browse the repository at this point in the history

  7. Fix error setting env variable HYPTIME_UNLINK to None

    @amrali
    amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    386373e View commit details
    Browse the repository at this point in the history

  8. Remove short arguments support

    @amrali
    amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    3938b23 View commit details
    Browse the repository at this point in the history

  9. Assign 'true' or 'false' to HUPTIME_MULTI

    @amrali
    amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    ec06579 View commit details
    Browse the repository at this point in the history

  10. Make --multi more useful by killing children when parent dies. 

    This will automatically clean-up all the child processes spawned
by --multi whenever the parent dies. This is much more useful for
integration with a supervisor (like upstart) because you can easily
kill the collection without needing to use killall or --stop.
    @amscanne @amrali
    amscanne authored and amrali committed Dec 23, 2013
    Configuration menu
    Copy the full SHA
    46935f8 View commit details
    Browse the repository at this point in the history