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

Fix/license for cran #17

Merged
merged 3 commits into from
Aug 26, 2014
Merged

Fix/license for cran #17

merged 3 commits into from
Aug 26, 2014

Conversation

jefferis
Copy link
Contributor

I'm not certain that this is the right incantation but taking a bit of inspiration from e.g.

http://cran.rstudio.com/web/packages/optparse/COPYRIGHTS

I can offer this small patch to get things rolling again.

License: MIT + file LICENSE
Copyright: Original python appdirs module copyright (c) 2010 ActiveState
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also list Activestate in authors?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like so?

person("ActiveState", role="aut"),

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you want role = "cph"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listing ActiveState in DESCRIPTION under role = "cph" is good for
keeping CRAN happy.

As Gregory points out the "Copyrights" field in DESCRIPTION is a good place
for preserving
all the short copyright notices like the ActiveState copyright notice which
under the MIT license must be preserved somewhere or
you could probably change file LICENSE to something like:

YEAR: 2010, 2014
COPYRIGHT HOLDER: ActiveState, Hadley Wickham, RStudio

which probably fulfills the spirit of the MIT requirement to "preserve
copyright notice" without requiring a separate "Copyrights" field or
COPYRIGHTS file (the COPYRIGHTS file is useful if combining code from a
very long permissive license with code under the GPL like I did in the
argparse/optparse packages but probably overkill when all the code is under
a single permissive license). FYI when you use a COPYRIGHTS file to in
order to fulfill the "preseve copyright notice" requirements of all your
licenses you still need duplicate a list of all the authors and copyright
holders in DESCRIPTION to satisfy CRAN.

Technically if people have been contributing code (4+ lines) to the project
under the MIT license and not explicitly assigning copyright to RStudio or
Hadley then there should probably be even more copyright holders in LICENSE
or the "Copyrights" field of DESCRIPTION although given that they are all
properly credited in DESCRIPTION and the package is released
non-commercially under such a permissive license I doubt that any of the
contributors or CRAN care strongly that this copyright information is
recorded perfectly although ActiveState is a company and maybe cares more
strongly.

Trevor

On Thu, Aug 21, 2014 at 9:28 AM, Hadley Wickham notifications@github.com
wrote:

In DESCRIPTION:

License: MIT + file LICENSE
+Copyright: Original python appdirs module copyright (c) 2010 ActiveState

I think you want role = "cph"


Reply to this email directly or view it on GitHub
https://github.com/hadley/rappdirs/pull/17/files#r16550093.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the right place to discuss, but I guess it would make sense to create a CLA (http://en.wikipedia.org/wiki/Contributor_license_agreement) for rappdirs (and other projects). Just to be safe in the future. Anyway, just bringing this up, it's up to the owner to decide, obviously.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gregory Jefferis

On 21 Aug 2014, at 18:29, trevorld notifications@github.com wrote:

In DESCRIPTION:

License: MIT + file LICENSE
+Copyright: Original python appdirs module copyright (c) 2010 ActiveState
Listing ActiveState in DESCRIPTION under role = "cph" is good for keeping CRAN happy. As Gregory points out the "Copyrights" field in DESCRIPTION is a good place for preserving all the short copyright notices like the ActiveState copyright notice which under the MIT license must be preserved somewhere or you could probably change file LICENSE to something like: YEAR: 2010, 2014 COPYRIGHT HOLDER: ActiveState, Hadley Wickham, RStudio which probably fulfills the spirit of the MIT

So is the consensus that it would be better simply to squash the license file back down to two lines as above? The copyright field in description would still give a bit more info.

Technically if people have been contributing code (4+ lines) to the project under the MIT license and not explicitly assigning copyright

FWIW I hereby assign copyright of my portions of this code to Hadley Wickham, RStudio.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I preferred the explicitness of the original license file. @jefferis can you please restore?

@gaborcsardi maybe, but I think a CLA just adds more hassle.

@jefferis
Copy link
Contributor Author

@hadley, did you want me to reset this branch to 96a10af? This probably makes more sense than reverting, which I did hurriedly on Thu last.

@hadley
Copy link
Member

hadley commented Aug 26, 2014

Whatever is easiest for you

@jefferis
Copy link
Contributor Author

OK, I reset to give you a clean branch at 96a10af with a detailed LICENSE file with 2 entries. Best, Greg.

hadley added a commit that referenced this pull request Aug 26, 2014
@hadley hadley merged commit 307ba16 into r-lib:master Aug 26, 2014
@hadley
Copy link
Member

hadley commented Aug 26, 2014

Resubmitted. Fingers crossed.

@jefferis
Copy link
Contributor Author

jefferis commented Sep 1, 2014

@hadley I'm guessing the detailed license didn't fly ...

@hadley
Copy link
Member

hadley commented Sep 2, 2014

Note, apparently that needs to go in LICENSE.note (but don't actually need it here)

@jefferis jefferis deleted the fix/license-for-cran branch September 2, 2014 20:00
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.

4 participants