[PATCH] gnu: polkit.scm: Add missing copyright headers.

  • Done
  • quality assurance status badge
Details
2 participants
  • Morgan.J.Smith
  • Maxim Cournoyer
Owner
unassigned
Submitted by
Morgan.J.Smith
Severity
normal

Debbugs page

Morgan.J.Smith wrote 4 years ago
(address . guix-patches@gnu.org)(name . Morgan Smith)(address . Morgan.J.Smith@outlook.com)
BYAPR05MB4023E2534B1533DA96EA7677C5949@BYAPR05MB4023.namprd05.prod.outlook.com
From: Morgan Smith <Morgan.J.Smith@outlook.com>

* gnu/packages/polkit.scm: Add missing copyright headers.
---

I don't know if this is usually done. I just noticed that the file I was
working on had very few copyright lines so I filled it in with info from the
git history.

---
gnu/packages/polkit.scm | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)

Toggle diff (28 lines)
diff --git a/gnu/packages/polkit.scm b/gnu/packages/polkit.scm
index d868aceec2..7c7fc58bb6 100644
--- a/gnu/packages/polkit.scm
+++ b/gnu/packages/polkit.scm
@@ -1,11 +1,17 @@
;;; GNU Guix --- Functional package management for GNU
;;; Copyright © 2014 Andreas Enge <andreas@enge.fr>
-;;; Copyright © 2015 Andy Wingo <wingo@igalia.com>
-;;; Copyright © 2015 Mark H Weaver <mhw@netris.org>
-;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2015 Sou Bunnbu <iyzsong@gmail.com>
+;;; Copyright © 2015, 2018, 2019 Ludovic Courtès <ludo@gnu.org>
+;;; Copyright © 2015, 2016 Andy Wingo <wingo@igalia.com>
+;;; Copyright © 2015, 2016 Mark H Weaver <mhw@netris.org>
+;;; Copyright © 2016, 2019 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2016 Alex Kost <alezost@gmail.com>
+;;; Copyright © 2016 David Craven <david@craven.ch>
;;; Copyright © 2017 Huang Ying <huang.ying.caritas@gmail.com>
;;; Copyright © 2018 Tobias Geerinckx-Rice <me@tobias.gr>
-;;; Copyright © 2018 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2018, 2019 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2018 Leo Famulari <leo@famulari.name>
+;;; Copyright © 2019 Marius Bakke <mbakke@fastmail.com>
;;;
;;; This file is part of GNU Guix.
;;;
--
2.30.1
Maxim Cournoyer wrote 3 years ago
(address . Morgan.J.Smith@outlook.com)(address . 46994@debbugs.gnu.org)
87lf3334gv.fsf@gmail.com
Hello,

Morgan.J.Smith@outlook.com writes:

Toggle quote (9 lines)
> From: Morgan Smith <Morgan.J.Smith@outlook.com>
>
> * gnu/packages/polkit.scm: Add missing copyright headers.
> ---
>
> I don't know if this is usually done. I just noticed that the file I was
> working on had very few copyright lines so I filled it in with info from the
> git history.

This could be done if, for example, the packages got move but their
copyright attribution forgotten. In which case careful review should be
done to make sure the copyrights are moved or copied, accordingly.

Also, code removal and trivial edits (such as updating < 10 lines, for
example by bumping the version string and the hash of a package) are not
usually considered significant enough to be copyrighted, so this may
explain the discrepancy.

Were the changes in the git history copyright-worthy?

Thanks,

Maxim
Morgan Smith wrote 3 years ago
(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)(address . 46994@debbugs.gnu.org)
BYAPR05MB4023D248807505DC8D618B92C5B29@BYAPR05MB4023.namprd05.prod.outlook.com
Toggle quote (7 lines)
> Also, code removal and trivial edits (such as updating < 10 lines, for
> example by bumping the version string and the hash of a package) are not
> usually considered significant enough to be copyrighted, so this may
> explain the discrepancy.
>
> Were the changes in the git history copyright-worthy?

Most of these are trivial and do not deserve copyright. I was unaware of
these exceptions.

I believe the following copyright should be added:
Ludo 2015 for cb151c68bf and 222e3319bd

Neither of those two changes are greater than 10 lines but I'd call them
non-trivial.

Feel free to close this bug after you've decided if you'll add that one
copyright. Had I known what I know now I wouldn't have opened this bug.

Thanks,

Morgan
Maxim Cournoyer wrote 3 years ago
control message for bug #46994
(address . control@debbugs.gnu.org)
87r1ct1nmp.fsf@gmail.com
close 46994
quit
?
Your comment

This issue is archived.

To comment on this conversation send an email to 46994@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 46994
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help