[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
M
M
Morgan.J.Smith wrote on 7 Mar 2021 22:08
(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
M
M
Maxim Cournoyer wrote on 8 Oct 2021 17:45
(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
M
M
Morgan Smith wrote on 8 Oct 2021 20:30
(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
M
M
Maxim Cournoyer wrote on 10 Oct 2021 06:59
control message for bug #46994
(address . control@debbugs.gnu.org)
87r1ct1nmp.fsf@gmail.com
close 46994
quit
?