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

Add setup files #17

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add setup files #17

wants to merge 11 commits into from

Conversation

JeffJacobson
Copy link

I needed to make some enhancements to your script, so I am contributing them back.

Changes

  • Added files to allow setup via pip.
  • Configuration file is no longer stored in repository (and is added to .gitignore), but instead user can generate this sample file using the script. The reason for this change was to prevent contributors from accidentally submitting their own configs with sensitive info into the repository
  • Default configuration is now read from the working directory rather than the same directory as the script.
  • Script arguments are now handled using argparse to provide better user experience. E.g., user can now get help at command line using --help or -h arguments with the script.
  • Added script to build package files, suitable for uploading to Python Package Index (PyPI).
  • Updated README.md to reflect changes.

jmhauck and others added 11 commits January 30, 2017 11:36
Add call to find\delete previous instance of gdb prior to upload.
Do not use the key tag in the inital gdb search rather evaluate the tags
after inital query. If a file is found with the same name but does not
have the key tag rasie an exception to stop the script and alert the
user.
Config file can now be generated by the script using --generate-config
argument.
Modified README.md
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.

2 participants