Re: Bug#798476: Bug#870788: Extract recent uploaders from d/changelog

2017-08-05 Thread Adrian Bunk 
On Sat, Aug 05, 2017 at 03:19:29PM +, Holger Levsen wrote:
> On Sat, Aug 05, 2017 at 04:35:35PM +0200, Bill Allombert wrote:
> > > > Note that a prerequisite for such debian/changelog parsing would be
> > > > that policy sets strict syntax and semantics requirements.
> > > 
> > > No, we do not need to block such a feature that would work for 90% of
> > > packages until we have a policy about the [ name ] syntax.  It can begin
> > > as a useful heuristic.
> > 
> > How do you get that it would work 90% of package ?
> > Using [] for non-team members is very common.
> 
> for getting the _uploaders_ it's not even required to parse those fields, as
> each upload has one uploader which is semantically strict defined already.

Policy says that Uploaders contains the _co-maintainers_.

And that also matches what most teams do.

> cheers,
>   Holger

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

Re: Bug#798476: Bug#870788: Extract recent uploaders from d/changelog

2017-08-05 Thread Sean Whitton 
Hello,

[removing bug against -policy from CC]

On Sat, Aug 05 2017, Bill Allombert wrote:

> How do you get that it would work 90% of package ?  Using [] for
> non-team members is very common.

My intent was to discover who is working on a package.  It would be okay
for non-team members to be included in that.  Further, the tool can rank
contributors -- a non-team member who submits a few patches wouldn't be
ranked highly.

-- 
Sean Whitton


signature.asc
Description: PGP signature

Re: Bug#798476: Bug#870788: Extract recent uploaders from d/changelog

2017-08-05 Thread Holger Levsen 
On Sat, Aug 05, 2017 at 04:35:35PM +0200, Bill Allombert wrote:
> > > Note that a prerequisite for such debian/changelog parsing would be
> > > that policy sets strict syntax and semantics requirements.
> > 
> > No, we do not need to block such a feature that would work for 90% of
> > packages until we have a policy about the [ name ] syntax.  It can begin
> > as a useful heuristic.
> 
> How do you get that it would work 90% of package ?
> Using [] for non-team members is very common.

for getting the _uploaders_ it's not even required to parse those fields, as
each upload has one uploader which is semantically strict defined already.


-- 
cheers,
Holger


signature.asc
Description: Digital signature

Re: Bug#798476: Bug#870788: Extract recent uploaders from d/changelog

2017-08-05 Thread Bill Allombert 
On Sat, Aug 05, 2017 at 07:00:16AM -0700, Sean Whitton wrote:
> Hello,
> 
> On Sat, Aug 05 2017, Adrian Bunk wrote:
> 
> > I assume you are thinking of parsing the [ name ] syntax used by many
> > teams.
> 
> Yes.
> 
> > Note that a prerequisite for such debian/changelog parsing would be
> > that policy sets strict syntax and semantics requirements.
> 
> No, we do not need to block such a feature that would work for 90% of
> packages until we have a policy about the [ name ] syntax.  It can begin
> as a useful heuristic.

How do you get that it would work 90% of package ?
Using [] for non-team members is very common.

Cheers,
-- 
Bill. 

Imagine a large red swirl here.

Re: Bug#798476: Bug#870788: Extract recent uploaders from d/changelog

2017-08-05 Thread Sean Whitton 
Hello,

On Sat, Aug 05 2017, Adrian Bunk wrote:

> I assume you are thinking of parsing the [ name ] syntax used by many
> teams.

Yes.

> Note that a prerequisite for such debian/changelog parsing would be
> that policy sets strict syntax and semantics requirements.

No, we do not need to block such a feature that would work for 90% of
packages until we have a policy about the [ name ] syntax.  It can begin
as a useful heuristic.

-- 
Sean Whitton


signature.asc
Description: PGP signature