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

BRL-CAD Fixes (previous pull request should be closed as obsolete) #5

Merged
merged 10 commits into from
Apr 4, 2013

Conversation

tbrowder
Copy link
Contributor

@tbrowder tbrowder commented Apr 2, 2013

Fixes are included to (1) enable shared AND static libraries for all (users can choose to turn off either at ../configure time, (2) add missing compile switch -fPIC for all libraries (thanks due to Cristina Precup, BRL-CAD contributor), and (3) mention BRL-CAD as a user.

tbrowder and others added 10 commits March 30, 2013 16:02
added macro for libtool (added comment to aid user); choosing shared lib; turned on AC_PROG_INSTALL
added missing header
added missing library libavoid
comment out nonexistent file
add BRL-CAD as a using project
@mjwybrow
Copy link
Owner

mjwybrow commented Apr 4, 2013

Thanks Tom, these changes looks fine.

Would you be happy to assign the copyright on these changes to Monash University? This is important to us if we want to put your changes in the standard version of the library since (in addition to the open source licence) we currently commercially licence the library to a few people and this requires Monash to hold the copyright for the whole codebase. A simple statement as a comment would suffice, saying "I hereby assign copyright in this code to Monash University, to be licensed under the same terms as the rest of the code."

@tbrowder
Copy link
Contributor Author

tbrowder commented Apr 4, 2013

Happy to do so, Michael:

I hereby assign copyright in this code to Monash University, to be licensed under the same terms as the rest of the code.

-Tom

@mjwybrow mjwybrow merged commit 3171fec into mjwybrow:master Apr 4, 2013
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

3 participants