Skip to content

Conversation

@ihrivnac
Copy link
Contributor

… agreed in the O2 project

@ihrivnac
Copy link
Contributor Author

@ktf, the link should be working now without login, can you check again?
When I go to the page with "http", the browser then changes to "https", so it is working as it is; but we can update it to 'https", if it is safer.

Copy link
Collaborator

@rpezzi rpezzi left a comment

Choose a reason for hiding this comment

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

Listing institutions is nice for aesthetical reasons but I am afraid the written statement does not have legal value. By default, authors are the copyright holders of their creations and I am sure that not every O2 contributor have transferred the copyright of their production to their hosting institution. Formal transfers of copyrights are needed to back the new wording in the header + linked list. To illustrate, here are two examples of collaborative projects with a complex and simple solution to this dilemma, respectively: :

  • GNU Project: FSF request formal transfer of ownership to defend the rights in court
  • Wikipedia: Simply states on each article "Author: Wikipedia contributors"

@ihrivnac ihrivnac dismissed stale reviews from iarsene and bovulpes via 03a02f1 June 16, 2021 14:49
@ihrivnac
Copy link
Contributor Author

Listing institutions is nice for aesthetical reasons but I am afraid the written statement does not have legal value. By default, authors are the copyright holders of their creations and I am sure that not every O2 contributor have transferred the copyright of their production to their hosting institution. Formal transfers of copyrights are needed to back the new wording in the header + linked list. To illustrate, here are two examples of collaborative projects with a complex and simple solution to this dilemma, respectively: :

  • GNU Project: FSF request formal transfer of ownership to defend the rights in court
  • Wikipedia: Simply states on each article "Author: Wikipedia contributors"

@rpezzi,
This update of header the files text was consulted with the CERN Legal and Knowledge Transfer Teams. Please, see more details in JIRA O2-1381. It has been then discussed in several O2 meetings (including PDP/FLP Plenaries) and approved by all three O2 project leaders, so the purpose of this PR is not to reopen a discussion, but to put the agreed text in place.

@rpezzi
Copy link
Collaborator

rpezzi commented Jun 16, 2021

@rpezzi,
This update of header the files text was consulted with the CERN Legal and Knowledge Transfer Teams. Please, see more details in JIRA O2-1381. It has been then discussed in several O2 meetings (including PDP/FLP Plenaries) and approved by all three O2 project leaders, so the purpose of this PR is not to reopen a discussion, but to put the agreed text in place.

Sorry for not following the discussion in time and on the proper place. But I have say that it is odd to read that CERN is claiming copyright over something it does not have nor does a third party on a list. The current header is vague enough Copyright CERN and copyright holders of ALICE O2, i.e., without an exhaustive list. I am surprised the legal team did not raise this issue. The simplest solution I can think of to simply add something like "and others" to the bottom of the copyright owners list.

However, it is also possible that I missed copyright transfer clause on my employment contract or on the ALICE membership agreement. If this is the case, I apologize for the noise.

@awegrzyn awegrzyn self-requested a review June 17, 2021 10:11
awegrzyn
awegrzyn previously approved these changes Jun 17, 2021
@ihrivnac ihrivnac force-pushed the update-copyright branch 2 times, most recently from a532c32 to 21e84fa Compare June 23, 2021 08:20
@ihrivnac
Copy link
Contributor Author

@rpezzi,
This update of header the files text was consulted with the CERN Legal and Knowledge Transfer Teams. Please, see more details in JIRA O2-1381. It has been then discussed in several O2 meetings (including PDP/FLP Plenaries) and approved by all three O2 project leaders, so the purpose of this PR is not to reopen a discussion, but to put the agreed text in place.

Sorry for not following the discussion in time and on the proper place. But I have say that it is odd to read that CERN is claiming copyright over something it does not have nor does a third party on a list. The current header is vague enough Copyright CERN and copyright holders of ALICE O2, i.e., without an exhaustive list. I am surprised the legal team did not raise this issue. The simplest solution I can think of to simply add something like "and others" to the bottom of the copyright owners list.

However, it is also possible that I missed copyright transfer clause on my employment contract or on the ALICE membership agreement. If this is the case, I apologize for the noise.

@rpezzi, we cannot now change the text, which was discussed and approved by the procedure which I described above. Also note, that the update in this PR does not change the way the copyright holders are presented wrt what was there before.

rpezzi
rpezzi previously approved these changes Jun 23, 2021
vkucera
vkucera previously approved these changes Jun 23, 2021
@ihrivnac ihrivnac dismissed stale reviews from vkucera and rpezzi via d79bcef June 23, 2021 11:51
@ihrivnac ihrivnac merged commit 03608ff into AliceO2Group:dev Jun 23, 2021
TimoWilken added a commit to TimoWilken/alidist that referenced this pull request Jun 24, 2021
AliceO2Group/AliceO2#6430 updated all files in O2, so
now PRs are failing this check. Update the "template" notice to match.
ktf pushed a commit to alisw/alidist that referenced this pull request Jun 24, 2021
AliceO2Group/AliceO2#6430 updated all files in O2, so
now PRs are failing this check. Update the "template" notice to match.
noferini pushed a commit to noferini/AliceO2 that referenced this pull request Jun 24, 2021
…6430)

* Updated copyright headers in all files with new text agreed in the O2 project
shahor02 added a commit that referenced this pull request Jun 26, 2021
* generalize TOF matcher

* add MatchInfoTOFReco class

* cleanup TOF reco workflow

* Updated copyright headers in all files with new text … (#6430)

* Updated copyright headers in all files with new text agreed in the O2 project

* fix copyrights new TOF classes

* Fixes

* fix for checks

Co-authored-by: Ivana Hrivnacova <Ivana.Hrivnacova@cern.ch>
Co-authored-by: shahoian <ruben.shahoyan@cern.ch>
knopers8 pushed a commit to knopers8/AliceO2 that referenced this pull request Jun 28, 2021
…6430)

* Updated copyright headers in all files with new text agreed in the O2 project
mcoquet642 pushed a commit to mcoquet642/AliceO2 that referenced this pull request Jul 4, 2021
* generalize TOF matcher

* add MatchInfoTOFReco class

* cleanup TOF reco workflow

* Updated copyright headers in all files with new text … (AliceO2Group#6430)

* Updated copyright headers in all files with new text agreed in the O2 project

* fix copyrights new TOF classes

* Fixes

* fix for checks

Co-authored-by: Ivana Hrivnacova <Ivana.Hrivnacova@cern.ch>
Co-authored-by: shahoian <ruben.shahoyan@cern.ch>
EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 2021
…6430)

* Updated copyright headers in all files with new text agreed in the O2 project
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

7 participants