On Wed, Apr 18, 2018 at 09:52:18PM +0500, Andrey Rahmatullin wrote:
> On Wed, Apr 18, 2018 at 04:00:51PM +0100, Ian Jackson wrote:
> > Instead, tools grew to tolerate commas here rather than treat them as
> > separators (because they would mishandle the erroneous packages).
> Is this the main problem with fixing the Policy? Does someone have a plan
> with this?

I checked UDD for real cases:

udd=# select distinct maintainer from packages where maintainer like '%,%' 
order by maintainer;
                                                  maintainer                    
                              
--------------------------------------------------------------------------------------------------------------
 "Adam C. Powell, IV" <[email protected]>
 Adam C. Powell, IV <[email protected]>
 Debian GNOME Maintainers <[email protected]>, 
Sebastian Dröge <[email protected]>
 John H. Robinson, IV <[email protected]>
 "Natural Language Processing, Japanese" 
<[email protected]>
 Natural Language Processing, Japanese 
<[email protected]>
 Rogério Brito <[email protected]>,                                             
                             +
  Holger Levsen <[email protected]>
 Thomas Bushnell, BSG <[email protected]>
 TransNexus, Inc. <[email protected]>
(9 rows)

>From my understanding the names in quotes should be parsed correctly, right?

This leaves 7 maintainer names

udd=# select distinct maintainer from packages where maintainer like '%,%' and 
maintainer not like '"%' order by maintainer;
                                                  maintainer                    
                               
---------------------------------------------------------------------------------------------------------------
 Adam C. Powell, IV <[email protected]>
 Debian GNOME Maintainers <[email protected]>, 
Sebastian Dröge <[email protected]>
 John H. Robinson, IV <[email protected]>
 Natural Language Processing, Japanese 
<[email protected]>
 Rogério Brito <[email protected]>,                                             
                             +
  Holger Levsen <[email protected]>
 Thomas Bushnell, BSG <[email protected]>
 TransNexus, Inc. <[email protected]>
(7 rows)

Out of these there are clearly two bugs that violate our current

udd=# select distinct package, maintainer from packages where maintainer like 
'%>%,%'order by maintainer;
     package      |                                                  maintainer 
                                                  
------------------+---------------------------------------------------------------------------------------------------------------
 gir1.0-gdata-0.0 | Debian GNOME Maintainers 
<[email protected]>, Sebastian Dröge 
<[email protected]>
 libgdata7        | Debian GNOME Maintainers 
<[email protected]>, Sebastian Dröge 
<[email protected]>
 libgdata-common  | Debian GNOME Maintainers 
<[email protected]>, Sebastian Dröge 
<[email protected]>
 libgdata-dev     | Debian GNOME Maintainers 
<[email protected]>, Sebastian Dröge 
<[email protected]>
 libgdata-doc     | Debian GNOME Maintainers 
<[email protected]>, Sebastian Dröge 
<[email protected]>
 youtube-dl       | Rogério Brito <[email protected]>,                          
                                                +
                  |  Holger Levsen <[email protected]>
(6 rows)


I think we should start filing bug reports against packages 
that do not match our current understanding of that field
(and lintian should throw an error about this).

Currently that definitely fits the last query but if we
intend to enhance the maintainer field to some later point
in time we should also ask the other 5 maintainers above
to add quotes around their names.

What do you think?

Kind regards

       Andreas.

-- 
http://fam-tille.de

Reply via email to