Re: [PATCH 2/2] dmd.cgi: Show only packages with this maintainer in sid or experimental.
On 13/07/12 at 13:01 -0600, Tim Retout wrote:
> When a package is adopted by another person or team, the original
> maintainer will not want to see issues regarding that package. However,
> the Maintainer field will not be updated in stable or testing. Avoid
> showing those packages with the old address.
> ---
> web/inc/dmd-data.rb | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/web/inc/dmd-data.rb b/web/inc/dmd-data.rb
> index 3efd1c1..0ca714f 100755
> --- a/web/inc/dmd-data.rb
> +++ b/web/inc/dmd-data.rb
> @@ -40,7 +40,7 @@ class UDDData
> maint_emails = @emails.reject { |k, v| not v.include?(:maintainer) }.keys
> if not maint_emails.empty?
> q = <<-EOF
> - select distinct source, maintainer_email from sources_uniq where release in ('sid', 'experimental', 'wheezy', 'squeeze') and maintainer_email in (#{maint_emails.map { |e| quote(e) }.join(',')})
> + select distinct source, maintainer_email from sources_uniq where release in ('sid', 'experimental') and maintainer_email in (#{maint_emails.map { |e| quote(e) }.join(',')})
> EOF
> maint_rows = dbget(q)
> else
> @@ -50,7 +50,7 @@ class UDDData
> upload_emails = @emails.reject { |k, v| not v.include?(:uploader) }.keys
> if not upload_emails.empty?
> q = <<-EOF
> - select distinct source, email from uploaders where release in ('sid', 'experimental', 'wheezy', 'squeeze') and email in (#{upload_emails.map { |e| quote(e) }.join(',')})
> + select distinct source, email from uploaders where release in ('sid', 'experimental') and email in (#{upload_emails.map { |e| quote(e) }.join(',')})
> EOF
> upload_rows = dbget(q)
> else
Hi,
Note that it is not clear if it is really the responsibility of the new
maintainer to take care of stable bugs. Also, in the case of a removed package
that is still in stable, it is clearly the responsibility of the 'stable
maintainer' to take care of issues.
I would rather have a checkbox (disabled by default) that would say "also
include all related source packages, not just those maintained in sid
and experimental". This is part of the "inputs tuning" section of the
TODO list (http://debian.titanpad.com/DMD):
<----
[medium] inputs tuning:
- add possibility to use several emails at the same time
- checkboxs for maintainer: only or maintainer + uploader
- remember what was asked for, display that in the form instead of stupidly overriding with pkg-ruby-extras-maintainers@
- be able to include/exclude packages where the email is only listed in stable
- include other packages (using a list of packages)
[hard] add possibility to enter a PGP key, get packages uploaded with that key
----->
(I have limited internet connection for a few days, I will comment/apply the
other patches later)
Lucas
Reply to: