Merge ~powersj/cloud-init:devel/fix-debian-copyright into cloud-init:ubuntu/devel

[Joshua Powers]

d/copyright: Reorganize file, add Apache 2.0 license

The license was updated to be dual-licensed under Apache 2.0 and
GPL 3.0, however, the text of the Apache 2.0 license was not added
to the debian/copyright file.

The file was also updated to be lintian clean and reorganized to
list the licenses at the bottom after declaring the metadata and
file information.

LP: #1718681

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:9b26debc236bb99ba90243d9dc5be97123986f96
https://jenkins.ubuntu.com/server/job/cloud-init-ci/343/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    SUCCESS: MAAS Compatability Testing
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/343/rebuild

review: Approve (continuous-integration)

[Ryan Harper (raharper)]

I don't think we want to change the License: field, just add the 2.0 text

review: Needs Fixing

[Scott Moser (smoser)]

Hi, comment 'b' below I'm asking Steve for insight on
I have a few questions here. Things that were broken or unclear before, but we might as well fix.

a.) 'Files:' is required now. So lets add a 'Files: *'

b.) Lets fix the 'Format' line:
- Format-Specification: http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn?op=file&rev=135
+ Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/

c.) Are we supposed to have an empty line between 'Licence' fields?
Josh has no empty line, but the specification in examples has one.

d.) Should the 'License:' sections be at the bottom or in order?

Then, lets just get all 4 relevant 'copyright' files to have the same content.
  ubuntu/xenial:debian/copyright
  ubuntu/zesty:debian/copyright
  ubuntu/devel:debian/copyright
  master:packages/debian/copyright

[Steve Langasek (vorlon)]

> I don't think we want to change the License: field, just add the 2.0 text

Generally you do want to specify the version as part of the license field, same as is done for GPL, as this assists with automated analysis.

> b.) Lets fix the 'Format' line:
> - Format-Specification: http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn?op=file&rev=135
> + Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/

Is this the bit you wanted my insight on? The new declaration is the current one that you want here, yes.

[Joshua Powers (powersj)]

@vorlon @smoser

Does this look right:
https://paste.ubuntu.com/25628692/

* Updated format
* Added "Files: *"
* Added blank lines between licenses
* Added line continuation in license text (e.g. ' .')

[Joshua Powers (powersj)]

I wasn't sure if I should put all three license's text at the bottom

[Steve Langasek (vorlon)]

On Wed, Sep 27, 2017 at 06:59:21PM -0000, Joshua Powers wrote:
> @vorlon @smoser

> Does this look right:
> https://paste.ubuntu.com/25628692/

> * Updated format
> * Added "Files: *"
> * Added blank lines between licenses
> * Added line continuation in license text (e.g. ' .')

This does not parse in dep5:

  This package was debianized by Soren Hansen <email address hidden> on
  Thu, 04 Sep 2008 12:49:15 +0200 as ec2-init. It was later renamed to
  cloud-utils by Scott Moser <email address hidden>

  Upstream Author: Scott Moser <email address hidden>
      Soren Hansen <email address hidden>
      Chuck Short <email address hidden>

IMHO putting the license: stanzas together at the bottom is more readable,
but there's no requirement in the standard that you do so.

And lintian should give you feedback about any syntax errors, if I'm not
mistaken.

[Scott Moser (smoser)]

lintian doesn't complain at all about the debian/copyright.
Steve can you give a quick look at
http://paste.ubuntu.com/25633185/

Changes versus Josh's last are:
  http://paste.ubuntu.com/25633193/

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:b7cb83b25458a6aa89d178b16fc3601a1e9992de
https://jenkins.ubuntu.com/server/job/cloud-init-ci/360/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    SUCCESS: MAAS Compatability Testing
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/360/rebuild

review: Approve (continuous-integration)

[Steve Langasek (vorlon)]

On Thu, Sep 28, 2017 at 01:01:36PM -0000, Scott Moser wrote:
> lintian doesn't complain at all about the debian/copyright.
> Steve can you give a quick look at
> http://paste.ubuntu.com/25633185/

LGTM.

diff --git a/debian/copyright b/debian/copyright
index cf4a76f..c223670 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -1,35 +1,48 @@
-Format-Specification: http://svn.debian.org/wsvn/dep/web/deps/dep5.mdwn?op=file&rev=135
-Name: cloud-init
-Maintainer: Scott Moser <[e-mail]>
+Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
+Upstream-Name: cloud-init
+Upstream-Contact: [e-mail]
 Source: https://launchpad.net/cloud-init
 
-This package was debianized by Soren Hansen <[e-mail]> on
-Thu, 04 Sep 2008 12:49:15 +0200 as ec2-init.  It was later renamed to
-cloud-utils by Scott Moser <[e-mail]>
-
-Upstream Author: Scott Moser <[e-mail]>
-    Soren Hansen <[e-mail]>
-    Chuck Short <[e-mail]>
-
+Files: *
 Copyright: 2010, Canonical Ltd.
 License: GPL-3 or Apache-2.0
+
+Files: cloudinit/boto_utils.py
+Copyright: 2006,2007, Mitch Garnaat http://garnaat.org/
+License: MIT
+
+License: GPL-3
  This program is free software: you can redistribute it and/or modify
  it under the terms of the GNU General Public License version 3, as
  published by the Free Software Foundation.
-
+ .
  This program is distributed in the hope that it will be useful,
  but WITHOUT ANY WARRANTY; without even the implied warranty of
  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  GNU General Public License for more details.
-
+ .
  You should have received a copy of the GNU General Public License
  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
+ .
  The complete text of the GPL version 3 can be seen in
  /usr/share/common-licenses/GPL-3.
 
-Files: cloudinit/boto_utils.py
-Copyright: 2006,2007, Mitch Garnaat http://garnaat.org/
+License: Apache-2.0
+ Licensed under the Apache License, Version 2.0 (the "License");
+ you may not use this file except in compliance with the License.
+ You may obtain a copy of the License at
+ .
+    http://www.apache.org/licenses/LICENSE-2.0
+ .
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+ .
+ On Debian-based systems the full text of the Apache version 2.0 license
+ can be found in `/usr/share/common-licenses/Apache-2.0'.
+
 License: MIT
  Permission is hereby granted, free of charge, to any person obtaining a
  copy of this software and associated documentation files (the
@@ -38,10 +51,10 @@ License: MIT
  tribute, sublicense, and/or sell copies of the Software, and to permit
  persons to whom the Software is furnished to do so, subject to the fol-
  lowing conditions:
-
+ .
  The above copyright notice and this permission notice shall be included
  in all copies or substantial portions of the Software.
-
+ .
  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
  OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABIL-
  ITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT