[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: New: "cme run paste-license script" (was: Re: pasting license text into debian/copyright)



On Sat, Oct 21, 2017 at 06:40:37PM +0200, gregor herrmann wrote:
> On Sat, 21 Oct 2017 16:20:58 +0200, Andreas Tille wrote:
> 
> > $ gbp clone https://anonscm.debian.org/git/debian-med/beads.git
> > $ cd beads
> > beads(master) $ cme run paste-license  --arg license=CeCILL --arg COPYING 
> 
> % cme run paste-license  --arg license=CeCILL --arg file=COPYING
> 
> seems to work; note the "file=" part as in dod's original mail:

Well, OK, when I do not miss the "file=" part I can do
 
$ cme run paste-license  --arg license=CeCILL --arg file=COPYING > copyright.patch
Log4perl: Seems like no initialization happened. Forgot to call init()?
Warning: Files:"*" License short_name skipping value CeCILL because of the following errors:
license 'CeCILL' is not declared in main License section. Expected 

Warning: Files:"debian/*" License short_name skipping value CeCILL because of the following errors:
license 'CeCILL' is not declared in main License section. Expected 

License CeCILL is not used in Files: section
Configuration item 'Files:"*" License short_name' has a wrong value:
        Undefined mandatory value.


I admit I do not really understand all the output to stderr.  The output
to stdout (in my example redirected to copyright.patch) is


-- 
hanges applied to dpkg-copyright configuration:
- License:CeCILL text:
@@ -1 +1,505 @@
-<undef>
+CeCILL FREE SOFTWARE LICENSE AGREEMENT
+
+
+    Notice
+
+This Agreement is a Free Software license agreement that is the result
+of discussions between its authors in order to ensure compliance with
+the two main principles guiding its drafting:
+
+    * firstly, compliance with the principles governing the distribution
...


So this does not really help since its neither a valid patch for
d/copyright nor can I add this content there without editing.  It just
added a '+' to the original license text which is not really helpful.
May be I'm continuously missing the point but shouldn't it add rather
a ' ' instead of a '+' and replace empty lines by ' .'?

Kind regards

        Andreas.


Reply to: