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

Update copyright file in debian directory #1324

Merged
merged 5 commits into from
Jul 3, 2020
Merged

Conversation

alexmyczko
Copy link
Contributor

i'd suggest dropping debian/ altogehter, making an official debian package, file an RFP/ITP bug, close it via debian/changelog
and that'll also you help easily have ubuntu copy your work (from debian) (officially distributing binaries of your software)

i'd suggest dropping debian/ altogehter, making an official debian package, file an RFP/ITP bug, close it via debian/changelog
and that'll also you help easily have ubuntu copy your work (from debian) (officially distributing binaries of your software)
@terhorstd terhorstd added ZC: Infrastructure DO NOT USE THIS LABEL I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) ZP: PR Created DO NOT USE THIS LABEL S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. labels Nov 11, 2019
Copy link
Contributor

@jougs jougs left a comment

Choose a reason for hiding this comment

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

I am mostly fine with the changes, although I think the description of the PR does not belong here, but should rather be in #1321. Please see my comments regarding the email address of the mailing list.

debian/copyright Outdated

Files: librandom/*
Copyright: other people
License: other license

Files: debian/*
Copyright: 2019 The NEST Initiative <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

See above.

Copy link
Contributor

Choose a reason for hiding this comment

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

2004, not 2019, see comment above.

Copy link
Contributor

Choose a reason for hiding this comment

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

See above, this must also be 2004.

Copy link
Contributor

@heplesser heplesser left a comment

Choose a reason for hiding this comment

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

@alexmyczko Please see my comments concerning copyright date and license.

debian/copyright Outdated
Source: https://github.com/nest/nest-simulator/

Files: *
Copyright: 2004-2019 The NEST Initiative <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

As a conscious policy decision, we always use 2004, the year of the first public release, as year of the NEST copyright, even for changes added later. So it should only be 2004 here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

you can use the initial copyright, but debian uses whenever the source was started being written, to when the last change has been made to it, so 2004-2019 for debian/copyright is perfectly fine. and you shouldn't have that file at all anyways in a source tarball release, see top suggestion

Copy link
Contributor

Choose a reason for hiding this comment

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

@alexmyczko Looking at https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/#copyright-field, there seems no requirement to include a mail address on the copyright line, so it should be removed here. I also do not find elsewhere (https://www.debian.org/doc/debian-policy/ch-docs.html#s-copyrightfile, https://www.debian.org/doc/manuals/maint-guide/dreq.en.html#copyright) any explicit policy on copyright dates for Debian. Therefore, we should use 2004, and only 2004 here in keeping with general NEST Copyright policy.

debian/copyright Outdated

Files: *
Copyright: 2004-2019 The NEST Initiative <[email protected]>
License: GPL-2
Copy link
Contributor

Choose a reason for hiding this comment

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

NEST license is GPL v2 or later, by choice of the user.

debian/copyright Outdated

Files: librandom/*
Copyright: other people
License: other license

Files: debian/*
Copyright: 2019 The NEST Initiative <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

2004, not 2019, see comment above.

debian/copyright Outdated

Files: debian/*
Copyright: 2019 The NEST Initiative <[email protected]>
License: GPL-2
Copy link
Contributor

Choose a reason for hiding this comment

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

GPL v2 or later, see above.

debian/copyright Outdated

Files: debian/*
Copyright: 2019 The NEST Initiative <[email protected]>
License: GPL-2

License: GPL-2
Copy link
Contributor

Choose a reason for hiding this comment

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

GPL v2 or later.

Copy link
Contributor

@steffengraber steffengraber left a comment

Choose a reason for hiding this comment

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

Hi @alexmyczko . Thank you for your help with that. I added my question and coments.

@heplesser heplesser changed the title Update copyright Update copyright file in debian directory Dec 9, 2019
@jougs
Copy link
Contributor

jougs commented Jan 9, 2020

@alexmyczko: can you please have a look at the comments by @steffengraber and merge upstream to resolve the conflicts? Thanks!

Copy link
Contributor

@steffengraber steffengraber left a comment

Choose a reason for hiding this comment

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

Perfect, Thanks.

Copy link
Contributor

@jougs jougs left a comment

Choose a reason for hiding this comment

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

Thanks for your changes and the discussion.

@jougs jougs requested a review from heplesser February 13, 2020 09:14
@jougs
Copy link
Contributor

jougs commented Feb 26, 2020

@alexmyczko: can you please resolve the conflicts in your branch, so we can merge this PR? Thanks!

steffengraber added a commit to steffengraber/nest-simulator that referenced this pull request Mar 17, 2020
Copy link
Contributor

@heplesser heplesser left a comment

Choose a reason for hiding this comment

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

@alexmyczko Please see my inline comments (remove mail address, 2004 as only copyright date).

debian/copyright Outdated
Source: https://github.com/nest/nest-simulator/

Files: *
Copyright: 2004-2019 The NEST Initiative <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

@alexmyczko Looking at https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/#copyright-field, there seems no requirement to include a mail address on the copyright line, so it should be removed here. I also do not find elsewhere (https://www.debian.org/doc/debian-policy/ch-docs.html#s-copyrightfile, https://www.debian.org/doc/manuals/maint-guide/dreq.en.html#copyright) any explicit policy on copyright dates for Debian. Therefore, we should use 2004, and only 2004 here in keeping with general NEST Copyright policy.

debian/copyright Outdated

Files: librandom/*
Copyright: other people
License: other license

Files: debian/*
Copyright: 2019 The NEST Initiative <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

See above, this must also be 2004.

steffengraber added a commit to steffengraber/nest-simulator that referenced this pull request Mar 18, 2020
@heplesser heplesser self-assigned this Apr 7, 2020
@heplesser heplesser removed ZC: Infrastructure DO NOT USE THIS LABEL ZP: PR Created DO NOT USE THIS LABEL labels Apr 7, 2020
@jougs
Copy link
Contributor

jougs commented Jul 3, 2020

Many thanks for this contribution and sorry for the messy review. In a group session during our ongoing hackathon, we addressed all open issues with this PR and came to the conclusion that we rather merge it as is instead of having this open for even longer.

@jougs jougs merged commit 8eb3bdb into nest:master Jul 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.
Projects
Development

Successfully merging this pull request may close these issues.

5 participants