Opened 10 years ago

Closed 5 years ago

Last modified 5 years ago

#1172 closed defect (fixed)

change the watkins license in SAGE_ROOT/COPYING

Reported by: was Owned by: was
Priority: minor Milestone: sage-5.1
Component: packages: standard Keywords:
Cc: Merged in: sage-5.1.rc0
Authors: Keshav Kini Reviewers: R. Andrew Ohana
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Change the watkins license in SAGE_ROOT/COPYING to the following, since Mark Watkins told me to do this.

The SYMPOW COPYING file says:


Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are met:
 * Redistribution of source code must retain the above copyright notice,
 this list of conditions and the following disclaimer.
 * Redistribution in binary form must reproduce the above copyright
 notice, this list of conditions and the following disclaimer in the
 documentation and/or other materials provided with the distribution.
 * If redistribution is done as a part of a compilation that has a more
 restrictive license (such as the GPL), then the fact that SYMPOW has
 a less restrictive license must be made clear to the recipient.
 For example, a line like (include bracketed text if SYMPOW is modified):
 "This compilation includes [a modification of] SYMPOW whose [original]
  code has a less-restrictive license than the entire compilation."
 should appear in a suitable place in the COPYING and/or LICENSE file.

[followed by the BSD disclaimer]

Attachments (1)

trac_1172-watkins-license.patch (2.1 KB) - added by kini 5 years ago.
apply to $SAGE_ROOT

Download all attachments as: .zip

Change History (8)

comment:1 Changed 5 years ago by kini

  • Authors set to Keshav Kini
  • Report Upstream set to N/A
  • Status changed from new to needs_review

This was a pretty easy patch to make...

Changed 5 years ago by kini

apply to $SAGE_ROOT

comment:2 Changed 5 years ago by ohanar

  • Reviewers set to R. Andrew Ohana
  • Status changed from needs_review to positive_review

Looks good to me.

comment:3 Changed 5 years ago by jdemeyer

The last clause sounds awfully much like the BSD advertising clause. Are we sure this is GPL-compatible? IANAL, but it's a natural question to ask.

comment:4 Changed 5 years ago by jdemeyer

  • Merged in set to sage-5.1.rc0
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:5 follow-up: Changed 5 years ago by kini

Jeroen: should this be merged if your question remains unanswered?

comment:6 in reply to: ↑ 5 Changed 5 years ago by jdemeyer

Replying to kini:

Jeroen: should this be merged if your question remains unanswered?

I guess so. This patch doesn't change the license, it changes the documentation of the license. Whether or not this patch gets merged, my question stands.

comment:7 Changed 5 years ago by kini

Good point.

Note: See TracTickets for help on using tickets.