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

Tickets/dm 3200 #7

Merged
merged 10 commits into from Nov 10, 2015
Merged

Tickets/dm 3200 #7

merged 10 commits into from Nov 10, 2015

Conversation

timj
Copy link
Member

@timj timj commented Nov 10, 2015

  • Support OS X El Capitan by allowing DYLD_LIBRARY_PATH to be passed to tests.
  • Allow the #! to be rewritten on certain platforms.
  • Fixes to allow spaces in directory paths.
  • PEP-8 fixes.

Uses utility function to determine whether to set DYLD_LIBRARY_PATH
By default files are found in bin.src/ and are copied to bin/

Only rewrites shebang if required
Define it once at the top of the file as a tuple.
All files in bin.src should require rewrites so it is odd
if one is found that does not.
* doxygen.conf files use double quotes for paths
* Quote the path to the doxygen.conf file when calling doxygen
Does not fix:

   E265 block comment should start with '# '

because it seems that starting a comment section with "##" is part
of the doxygen style
timj added a commit that referenced this pull request Nov 10, 2015
Tickets/dm 3200

* Support OS X El Capitan by allowing DYLD_LIBRARY_PATH to be passed to tests.
* Allow the #! to be rewritten on certain platforms.
* Fixes to allow spaces in directory paths.
* PEP-8 fixes.
@timj timj merged commit 3ed848b into master Nov 10, 2015
@ktlim ktlim deleted the tickets/DM-3200 branch August 25, 2018 06:16
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

1 participant