Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 550860 - [arch-tools] maintainer-timeout.py still CCs non-stable arches
Summary: [arch-tools] maintainer-timeout.py still CCs non-stable arches
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Agostino Sarubbo
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-31 08:27 UTC by Pacho Ramos
Modified: 2019-04-29 14:08 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
maintainer-timeout.new (maintainer-timeout.new,3.17 KB, text/plain)
2015-06-04 09:42 UTC, Pacho Ramos
Details
1.patch (1.patch,806 bytes, patch)
2015-06-05 18:00 UTC, Pacho Ramos
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pacho Ramos gentoo-dev 2015-05-31 08:27:49 UTC
The script should not CC s390@gentoo.org, m68k@gentoo.org and sh@gentoo.org

Maybe adding something like:
                bugzilla.modify(
                        bug.id_number(),
                        comment='Maintainer timeout (%d days). Arches please go ahead.' % options.days,
                        if "s390@gentoo.org" in params['cc']['add']:
                                params['cc']['add'].remove("s390@gentoo.org"),
                        if "m68k@gentoo.org" in params['cc']['add']:
                                params['cc']['add'].remove("m68k@gentoo.org"),
                        if "sh@gentoo.org" in params['cc']['add']:
                                params['cc']['add'].remove("sh@gentoo.org"),
                        add_cc=['%s@gentoo.org' % k for k in target_keywords],
                        keywords='STABLEREQ')

would work (it worked on a older script version :/)
Comment 1 Pacho Ramos gentoo-dev 2015-06-02 13:33:35 UTC
It should also skip arm64
Comment 2 Pacho Ramos gentoo-dev 2015-06-04 09:42:11 UTC
Created attachment 404572 [details]
maintainer-timeout.new

This works... but I cannot make the diff as I lost the original one and, for some reason, I cannot access the repo now :S
Comment 3 Agostino Sarubbo gentoo-dev 2015-06-05 14:51:39 UTC
(In reply to Pacho Ramos from comment #2)
> Created attachment 404572 [details]
> maintainer-timeout.new
> 
> This works... but I cannot make the diff as I lost the original one and, for
> some reason, I cannot access the repo now :S

nios2 joins also the keywords recently.

https://cgit.gentoo.org/proj/arch-tools.git/tree/maintainer-timeout.py that's what you need ;)
Comment 4 Pacho Ramos gentoo-dev 2015-06-05 18:00:40 UTC
Created attachment 404624 [details, diff]
1.patch

Thanks :D

This is the patch with the needed changes
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-12-18 09:54:09 UTC
(reassigning to the first person on the ACL, CC-ing everybody else)
Comment 6 Agostino Sarubbo gentoo-dev 2019-04-29 14:08:29 UTC
arch_tools is a project started by phajdan.jr which atm is a retired developer.
I made an ebuild that is is my overlay but atm I have nothing to do with this project because I'm not using those scripts to commit.
I asked mgorny to grant permission for all developers. If someone want to improve the project, it is on our git repo: https://gitweb.gentoo.org/proj/arch-tools.git/
For now, I'm closing it as WONTFIX