Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-21 Thread Dominique Dumont
On Friday 21 January 2011 14:32:54 Thomas Preud'homme wrote: > > 2011/01/13 16:30:47 Invalid line: Copyright : � 2010, The Tango team > > > > Could it be because of the space between Copyright and the colon? Yes. That was not recognized by the Dpkg syntax parser. This is now accepted but correc

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-21 Thread Thomas Preud'homme
Le jeudi 13 janvier 2011 16:35:06, Picca Frédéric-Emmanuel a écrit : > Le Thu, 13 Jan 2011 13:47:53 +0100, > Dominique Dumont a écrit : > > hello, another try :) > > picca@grisette:~/Debian/tango/tango$ config-edit -application > dpkg-copyright -ui none You should install Config::Model::TkUI or

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-20 Thread Dominique Dumont
On Thursday 20 January 2011 15:33:17 Adam Borowski wrote: > On Thu, Jan 20, 2011 at 02:27:55PM +0100, Dominique Dumont wrote: > > On Thursday 13 January 2011 16:35:06 Picca Frédéric-Emmanuel wrote: > > > 16:30:47 Invalid line: Copyright : � 1997-1999 AT&T Laboratories > > > Cambridge > >

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-20 Thread Adam Borowski
On Thu, Jan 20, 2011 at 02:27:55PM +0100, Dominique Dumont wrote: > On Thursday 13 January 2011 16:35:06 Picca Frédéric-Emmanuel wrote: > > 16:30:47 Invalid line: Copyright : � 1997-1999 AT&T Laboratories Cambridge ^ Aren't those supposed to be UTF-8 encoded?

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-20 Thread Dominique Dumont
On Saturday 15 January 2011 17:04:30 Joey Hess wrote: > One more thing, "License: GPL-2+ | Expat" was an old syntax on the wiki, > and it seems the parser only looks for an expansion of the GPL-2+ > license in this case, ignoring the Expat part. That was a bug in the Parse::RecDescent grammar snip

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-20 Thread Dominique Dumont
On Thursday 13 January 2011 23:54:04 Charles Plessy wrote: > By the way, Dominique, some licence version numbers can contain > alphanumeric characters. For instance the LaTeX project public license, > version 1.3c. Given that License keywords are fuzzy, moving targets. I've dropped any attempt at

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-20 Thread Dominique Dumont
Hello Frédéric On Thursday 13 January 2011 16:35:06 Picca Frédéric-Emmanuel wrote: > hello, another try :) > > picca@grisette:~/Debian/tango/tango$ config-edit -application > dpkg-copyright -ui none You should install Config::Model::TkUI or > Config::Model::CursesUI for a more friendly user inter

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-17 Thread Charles Plessy
Le Sun, Jan 16, 2011 at 01:17:23PM +0100, Christoph Anton Mitterer a écrit : > > I've also wondered whether it is allowed (or not), when having a Files > paragraph, that contains the verbatim license (not referring to a > standalone License tag), e.g.: > >Files: * > >License: FOO > > This is my wo

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-16 Thread Dominique Dumont
Le dimanche 16 janvier 2011 13:48:55, Charles Plessy a écrit : > Given that there are only 9 different fields in the current DEP-5 syntax, I > think that parsers can simply incorporate the full list of them rather than > rely on a X- prefix to determine if a field is in the specification or not. S

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-16 Thread Charles Plessy
Le Sun, Jan 16, 2011 at 01:12:46PM +0100, Christoph Anton Mitterer a écrit : > On Fri, 2011-01-14 at 07:58 +0900, Charles Plessy wrote: > > In the candidate version of the DEP, it is not recommended anymore to add > > an X- > > prefix to extra fields. > btw: Why was this removed? Adding the X- is

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-16 Thread Christoph Anton Mitterer
Oh, and I've also would like to see the "X-" on personal license short names. Perhaps one could also add a namespace that is "privately" managed, but still global, e.g. in the style of: "org.example.license.foobar" Specifying globally unique a license "foobar" from Example Org. But not sure whet

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-16 Thread Christoph Anton Mitterer
On Fri, 2011-01-14 at 12:09 -0400, Joey Hess wrote: > I probably misread DEP5 -- when it says "formatted text, no synopsis", > it probably means that the entire field including the first line > is treated as one thing. So both "Comment: foo\n" and "Comment:\n foo\n" are > the same value. I've also

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-16 Thread Christoph Anton Mitterer
On Fri, 2011-01-14 at 07:58 +0900, Charles Plessy wrote: > In the candidate version of the DEP, it is not recommended anymore to add an > X- > prefix to extra fields. btw: Why was this removed? Adding the X- is not only conventional style in email headers and many other RFCs but also (in a similar

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-15 Thread Joey Hess
One more thing, "License: GPL-2+ | Expat" was an old syntax on the wiki, and it seems the parser only looks for an expansion of the GPL-2+ license in this case, ignoring the Expat part. -- see shy jo signature.asc Description: Digital signature

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-15 Thread Charles Plessy
Le Fri, Jan 14, 2011 at 12:09:33PM -0400, Joey Hess a écrit : > > I probably misread DEP5 -- when it says "formatted text, no synopsis", > it probably means that the entire field including the first line > is treated as one thing. So both "Comment: foo\n" and "Comment:\n foo\n" are > the same valu

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-14 Thread Joey Hess
Dominique Dumont wrote: > > Also, note that "Comment: foo" is not flagged as a problem (despite > > being apparently illegal -- DEP-5 puzzlingly does not allow a comment > > to have a synopsis). > > Now that I have finally understood this notion of synopsis, I can fix these > issues. I probably

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-14 Thread Dominique Dumont
On Thursday 13 January 2011 22:48:48 Joey Hess wrote: > Also, it's missing BSD-2-clause etc. ok. Fixed. Unless I remove all these license keyword check... Still thinking ... > The original format from the wiki page uses comma to separate > Files. Might be worth detecting and converting those? D

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-14 Thread Dominique Dumont
On Thursday 13 January 2011 22:27:02 Joey Hess wrote: > Thanks for your work, here are a few things I stumbled on. Thanks for trying and reporting issues :-) > Worst problem: -save *removes* all Comment fields except for one > in the header section. Looks like I messed up the routine that transf

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Charles Plessy
Le Thu, Jan 13, 2011 at 04:35:06PM +0100, Picca Frédéric-Emmanuel a écrit : > Expected: > 'Format','Upstream-Name','Upstream-Contact','Source','Disclaimer','Comment','Copyright','Files','License','Format-Specification','Name','Maintainer','Upstream-Maintainer','Upstream-Source' > o

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Charles Plessy
Le Thu, Jan 13, 2011 at 03:06:56PM +0100, Picca Frédéric-Emmanuel a écrit : > > Warning in 'License': key 'CeCILLv2' should match > ^(?i:Apache|Artistic|BSD|FreeBSD|ISC|CC-BY|CC-BY-SA|CC-BY-ND|CC-BY-NC|CC-BY-NC-SA|CC-BY-NC-ND|CC0|CDDL|CPL|Eiffel|Expat|GPL|LGPL|GFDL|GFDL-NIV|LPPL|MIT|MPL|Perl|PSF|

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Joey Hess
Also, it's missing BSD-2-clause etc. The original format from the wiki page uses comma to separate Files. Might be worth detecting and converting those? -- see shy jo signature.asc Description: Digital signature

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Joey Hess
Thanks for your work, here are a few things I stumbled on. (Version: 1.229-1) Worst problem: -save *removes* all Comment fields except for one in the header section. Also, -save converted in the header "Disclaimer:\n Foo" into "Disclaimer: Foo". If I'm reading DEP-5 right, that's not correct, t

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Steve Langasek
On Thu, Jan 13, 2011 at 04:51:53PM +0100, Dominique Dumont wrote: > On Thursday 13 January 2011 16:38:31 Steve Langasek wrote: > > Please make your model recognize when a stand-alone license stanza is > > present which defines a new license name in the file. DEP5 is not > > intended to have an aut

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread gregor herrmann
On Thu, 13 Jan 2011 13:47:53 +0100, Dominique Dumont wrote: > The new version of libconfig-model-perl 1.229 is now available in Sid. This > new version provides a model of DEP-5 updated according to the CANDIDATE > version. (Among other features [1], like editor/validator for debian/control). >

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Dominique Dumont
On Thursday 13 January 2011 16:35:06 Picca Frédéric-Emmanuel wrote: > I attached the copyright file Interesting. Your file is formatted according to the spec archived on Debian wiki. Looks like this spec was done before the DEP-5 work begun. Since then, some fields like Debianized-* were dropped

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Dominique Dumont
On Thursday 13 January 2011 16:38:31 Steve Langasek wrote: > Please make your model recognize when a stand-alone license stanza is > present which defines a new license name in the file. DEP5 is not > intended to have an authoritative list of all licenses in use, only the > common ones; Currentl

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Steve Langasek
On Thu, Jan 13, 2011 at 03:53:14PM +0100, Dominique Dumont wrote: > > Expected: > > 'Format','Upstream-Name','Upstream-Contact','Source','Disclaimer','Comment > > ','Copyright','Files','License','Format-Specification','Name','Maintainer', > > 'Upstream-Maintainer','Upstream-Source' or an acceptable

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Picca Frédéric-Emmanuel
Le Thu, 13 Jan 2011 13:47:53 +0100, Dominique Dumont a écrit : hello, another try :) picca@grisette:~/Debian/tango/tango$ config-edit -application dpkg-copyright -ui none You should install Config::Model::TkUI or Config::Model::CursesUI for a more friendly user interface 2011/01/13 16:30:47 In

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Dominique Dumont
On Thursday 13 January 2011 15:06:56 Picca Frédéric-Emmanuel wrote: > Here the messages I got when checking my guidata package (already in the > new queue) > > picca at grisette:~/Debian/guidata/guidata$ config-edit -application > dpkg-copyright -ui none > You should install Config::Model::TkUI o

Re: DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Picca Frédéric-Emmanuel
Le Thu, 13 Jan 2011 13:47:53 +0100, Dominique Dumont a écrit : > Hello Hello Here the messages I got when checking my guidata package (already in the new queue) picca at grisette:~/Debian/guidata/guidata$ config-edit -application dpkg-copyright -ui none You should install Config::Model::TkUI

DEP5 CANDIDATE parser/editor/validator/migrator is released in libconfig-model-perl

2011-01-13 Thread Dominique Dumont
Hello The new version of libconfig-model-perl 1.229 is now available in Sid. This new version provides a model of DEP-5 updated according to the CANDIDATE version. (Among other features [1], like editor/validator for debian/control). The DEP-5 editor must be run in the source package directory