-
Notifications
You must be signed in to change notification settings - Fork 664
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
New xdrlib information in AUTHORS and LICENSE is missing/outdated #679
Comments
Sure we can also name it to libxdrfile2. Or libxdrfile3 to sow that there has been an update. |
It could also be a different name to avoid confusion if the original libxdrfile get a major update. Also, |
We should have a name different from the original gromacs one.
... Oliver Beckstein Am Jan 29, 2016 um 1:49 schrieb Jonathan Barnoud [email protected]:
|
I like |
I second @kain88-de's suggestion or something slightly more readable, like More opinions? |
Oops, it was actually @orbeckst's idea. Anyway, it has my vote (or the underscore version). |
I don't think those names ever have underscores in them for some reason? (libblas, libgfortran, liblapack?) |
Yea, a quick search through my own system shows that the usage of underscores seems to be mainly to distinguish different complile/linking versions of the same library. So we stick to |
No sounds good to me |
@kain88-de, I'm having trouble finding the newer, BSD-licensed, version of libxdrfile from the Gromacs people. Could you point me to it? |
http://www.gromacs.org/Developer_Zone/Programming_Guide/XTC_Library This is the most recent version I can find, according to version numbers. The website says that it is licensed under LGPL but when you open the tarball you'll notice that the license in the files used is BSD. Surprisingly that is not the version they have in their git repository, the git repo has an older version licensed under LGPL. |
@kain88-de you should contact GROMACS dev for correct answer. |
Further standardized XDR error message reporting and exceptions (now mostly IOError). Removed reference to the old SWIG numpy.i in the credits.
Further standardized XDR error message reporting and exceptions (now mostly IOError). Removed reference to the old SWIG numpy.i in the credits. Made logo LICENSE info consistent
Rebranded our xdrlib with cython bindings as libmdaxdr. Further standardized XDR error message reporting and exceptions (now mostly IOError). Removed reference to the old SWIG numpy.i in the credits. Made logo LICENSE info consistent
Rebranded our xdrlib with cython bindings as libmdaxdr. Further standardized XDR error message reporting and exceptions (now mostly IOError). Removed reference to the old SWIG numpy.i in the credits. Made logo LICENSE info consistent
Updated LICENSE and AUTHOR info regarding xdrfile (closes #679)
I'm opening this as a reminder so as not to let 0.14.0 ship without these fixes.
The new license aspects of xdrlib are outdated. The Gromacs base is now BSD.
We also haven't decided if the name libxdrfile2 stays, or if we call our modifications something else: with @kain88-de's cython approach (#441) and splitting of
xtc_seek.c
/trr_seek.c
fromxdrfile.c
(#661) there is less of a clear new library to speak of.@orbeckst has advocated keeping the name. I agree with that.
Right now we don't provide an actual
libxdrfile2.so
orlibxdrfile2.a
, but rather a mix of that with cython bindings (calledxdrlib.so
). This gets confusing butxdrlib.so
could be renamedlibxdrfile2
to keep the external appearance and conform to the name we were using before.The text was updated successfully, but these errors were encountered: