Re: DEP5 parser is available in Debians/Sid

2010-09-23 Thread Jonas Smedegaard

On Thu, Sep 23, 2010 at 06:49:50PM +0200, Dominique Dumont wrote:

Le mercredi 22 septembre 2010 18:23:39, Jonas Smedegaard a écrit :

>Configuration item 'Debian::Dep5 License id' has a wrong value:
>Unexpected key 'PGL-1'. Key must 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|C
>DDL|
>CPL|Eiffel|Expat|GPL|LGPL|GFDL|GFDL-NIV|LPPL|MIT|MPL|Perl|PSF|QPL|W3C-
>Software|ZLIB|Zope|other)[\d\.\-]*\+?$

Hmm - I believe you are being too restrictive above.

Syntax for license field says that "an arbitrary short name may be 
assigned" which I can only interpret to the following regex:


   ^\w+$


Hmm, I may have interpreted too literally the 'other' specification ;-)


Yes, standardized short names are encouraged, but not mandatory.

Yet another challenge for your tool ;-)


Well, it depends on what you want.

Just relaxing the restriction above is trivial.

But if you want a warning while allowing unknown licenses, then yes, 
I'll have to perform more extensive modifications to Config::Model( and 
on its various user interfaces to display properly the warnings...)


Sorry for being terse: Yes, you guessed correctly that my challenge to 
you is to not only support mandatory syntax but also recommended.



Would such a warning be a great addition or should we just provide the 
license list in the doc ?


...because yes, I would consider it cool if Config::Model would handle 
this.


I can easy imagine uses other than for DEP5.  Like deprecated options or 
syntax.


But I can imagine that it might require large code changes.


Should we perhaps stop cross-posting?

I have set Reply-To:[1] to only the SF list and myself.  Please keep 
me cc'ed as I am not suscribed there.


 - Jonas


[1] I don't know how to edit Mail-Followup-To: in Mutt, so haven't 
messed with that.


--
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private


signature.asc
Description: Digital signature


Re: DEP5 parser is available in Debians/Sid

2010-09-23 Thread Dominique Dumont
Le mercredi 22 septembre 2010 18:23:39, Jonas Smedegaard a écrit :
> >Configuration item 'Debian::Dep5 License id' has a wrong value:
> >Unexpected key 'PGL-1'. Key must 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|C
> >DDL|
> >CPL|Eiffel|Expat|GPL|LGPL|GFDL|GFDL-NIV|LPPL|MIT|MPL|Perl|PSF|QPL|W3C-
> >Software|ZLIB|Zope|other)[\d\.\-]*\+?$
> 
> Hmm - I believe you are being too restrictive above.
> 
> Syntax for license field says that "an arbitrary short name may be 
> assigned" which I can only interpret to the following regex:
> 
>^\w+$

Hmm, I may have interpreted too literally the 'other' specification ;-)

> Yes, standardized short names are encouraged, but not mandatory.
> 
> Yet another challenge for your tool ;-)

Well, it depends on what you want. 

Just relaxing the restriction above is trivial. 

But if you want a warning while allowing unknown licenses, then yes, I'll have 
to perform more extensive modifications to Config::Model( and on its various  
user interfaces to display properly the warnings...)

Would such a warning be a great addition or should we just provide the license 
list in the doc ?

All the best

Dominique
--
http://config-model.wiki.sourceforge.net/ -o- http://search.cpan.org/~ddumont/
http://www.ohloh.net/accounts/ddumont -o- http://ddumont.wordpress.com/


--
To UNSUBSCRIBE, email to debian-project-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201009231849.51504.domi.dum...@free.fr



Re: DEP5 parser is available in Debians/Sid

2010-09-22 Thread Jonas Smedegaard

Hi Dominique and others,

On Wed, Sep 22, 2010 at 02:05:53PM +0200, Dominique Dumont wrote:
Thanks to gregoa's sponsorship, the new parser for DEP5 is available in 
libconfig-model-perl 1.209.


Awesome!



Configuration item 'Debian::Dep5 License id' has a wrong value:
   Unexpected key 'PGL-1'. Key must 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|QPL|W3C-
Software|ZLIB|Zope|other)[\d\.\-]*\+?$


Hmm - I believe you are being too restrictive above.

Syntax for license field says that "an arbitrary short name may be 
assigned" which I can only interpret to the following regex:


  ^\w+$

Yes, standardized short names are encouraged, but not mandatory.

Yet another challenge for your tool ;-)


 - Jonas

--
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private


signature.asc
Description: Digital signature


DEP5 parser is available in Debians/Sid

2010-09-22 Thread Dominique Dumont
Hello

Thanks to gregoa's sponsorship, the new parser for DEP5 is available in 
libconfig-model-perl 1.209.

To run it, use the command config-edit-dep5. 

For instance:
$ dh-make-perl --cpan Dancer
[snip]
$ cd Dancer-1.1812/
$ config-edit-dep5 -ui none
$ sed -i 's/GPL/PGL/' debian/copyright 
$ config-edit-dep5 -ui none
Configuration item 'Debian::Dep5 License id' has a wrong value:
Unexpected key 'PGL-1'. Key must 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|QPL|W3C-
Software|ZLIB|Zope|other)[\d\.\-]*\+?$

'-ui none' option is specified to skip user interfaces.

Feel free to send feedback to improve this parser.

All the best


-- 
To UNSUBSCRIBE, email to debian-project-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201009221405.53690.domi.dum...@free.fr