-
Notifications
You must be signed in to change notification settings - Fork 0
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
This fork violates the LPPL license #2
Comments
Le mar. 28 août 2018 à 11:53, Frank Mittelbach <[email protected]> a
écrit :
Hi Bastien,
the package as it is placed on CTAN (or here on github) is a very
problematical fork as it doesn't tell anywhere what is changed (or why).
Instead it is a 1-2-1 copy of the original with just footmisc replaced by
footmisx (which is good but makes it of course very hard to see what you
have done to the code written by Robin).
LPPL requires for derived works (ie forks):
Every component of the Derived Work contains prominent notices
detailing the nature of the changes to that component, or a
prominent reference to another file that is distributed as part
of the Derived Work and that contains a complete and accurate log
of the changes.
and while "complete and accurate" may be a matter of opinion but nothing
here comes even close to it.
Looks like that by the end of the day all your changes come down to a few
lines
footmisx-real-changes.txt
<https://github.com/bastien-roucaries/footmisx/files/2327494/footmisx-real-changes.txt>
and even that could be further reduced.
Given that you make minimal real changes, it would be much better to
simply load footmisc (passing the options on) and then have a few lines
containing your changes (plus some documentation why and how). And along
the way an example that tells a user why he or she should use your code
over the well-established footmisc would also help.
So please update your package or withdraw it from here and CTAN.
Will add a changelog... And document changes
… —
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AEnaHOcw8kKCkvJ_jntHounyCFvq_QDSks5uVRMPgaJpZM4WPR64>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi Bastien,
the package as it is placed on CTAN (or here on github) is a very problematical fork as it doesn't tell anywhere what is changed (or why). Instead it is a 1-2-1 copy of the original with just footmisc replaced by footmisx (which is good but makes it of course very hard to see what you have done to the code written by Robin).
LPPL requires for derived works (ie forks):
and while "complete and accurate" may be a matter of opinion but nothing here comes even close to it.
Looks like that by the end of the day all your changes come down to a few lines
footmisx-real-changes.txt
and even that could be further reduced.
Given that you make minimal real changes, it would be much better to simply load footmisc (passing the options on) and then have a few lines containing your changes (plus some documentation why and how). And along the way an example that tells a user why he or she should use your code over the well-established footmisc would also help.
So please update your package or withdraw it from here and CTAN.
The text was updated successfully, but these errors were encountered: