Bug 227147 - Remove deprecated keywords: needs-staging, et al
Summary: Remove deprecated keywords: needs-staging, et al
Status: Open
Alias: None
Product: Services
Classification: Unclassified
Component: Bug Tracker (show other bugs)
Version: unspecified
Hardware: Any Any
: --- Affects Only Me
Assignee: Kubilay Kocak
URL:
Keywords: easy, needs-qa
Depends on:
Blocks:
 
Reported: 2018-03-31 22:12 UTC by Mateusz Piotrowski
Modified: 2018-04-09 10:20 UTC (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mateusz Piotrowski freebsd_committer 2018-03-31 22:12:30 UTC
The link[1] in the description of "needs-staging" in the Bugzilla keywords list[3] to is broken as it was recently renamed[2].

[1]: https://wiki.freebsd.org/ports/StageDir
[2]: https://wiki.freebsd.org/Ports/StageDir
[3]: https://bugs.freebsd.org/bugzilla/describekeywords.cgi
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2018-04-01 00:07:28 UTC
(In reply to Mateusz Piotrowski from comment #0)

I've worked around it on the wiki side by using a page redirect.  (I learned something new today.)
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2018-04-01 02:23:58 UTC
The needs-staging keyword is deprecated, mostly cause all ports are already staged, and all new ports must be stagedir compliant. 

I had not removed it earlier are there are other keywords which need to change/remove/be added, and sometimes functionality (auto-assigner) can rely on or use/add keywords in certain cases, and may not resiliently handle their non-presence.

I'll work with @gonzo to verify the latter is not the case, and remove it once that has been identified.
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2018-04-01 02:26:07 UTC
In the meantime, I've fixed the description link, added deprecated messaging to it's description, and will delete the redirected page (after seeing if any other pages in the wiki depend on it)
Comment 4 Oleksandr Tymoshenko freebsd_committer freebsd_triage 2018-04-09 07:34:00 UTC
(In reply to Kubilay Kocak from comment #2)

Needs staging is not used in auto-assigner, so it's safe to remove it.
Comment 5 Kubilay Kocak freebsd_committer freebsd_triage 2018-04-09 10:20:00 UTC
(In reply to Oleksandr Tymoshenko from comment #4)

Thanks.

the auto assignment of keyword:patch based on [tags] in summary has now been removed iirc?

Beyond that, are there additions/removals of any keywords in the AA code left at this stage?