Closed Bug 840202 Opened 8 years ago Closed 8 years ago

remove accessibility branch

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aki, Assigned: aki)

References

Details

Attachments

(5 files)

Not used since April last year.
Pinged #accessibility; seems like they don't use it.

We should get a sign-off here before doing this.

Marking as blocking bug 747382 since the addSuite() and loadCustomUnittestSuite() methods would be complex to port, and are only used for the [currently unused] accessibility branch.

[11:40]	<aki>	i'm asking if https://tbpl.mozilla.org/?tree=Accessibility is needed, since it hasn't been used since april last year
[11:40]	<aki>	if it is needed, do we need to special case snow leopard on that branch anymore
[11:40]	<hub>	I don't use it either
[11:41]	<aki>	davidb: ok to remove it?
[12:52]	<davidb>	aki: yes
[12:52]	<davidb>	aki: planned to use it but didn't
(In reply to Aki Sasaki [:aki] from comment #0)
> Not used since April last year.
> Pinged #accessibility; seems like they don't use it.
> 
> We should get a sign-off here before doing this.
> 
> Marking as blocking bug 747382 since the addSuite() and
> loadCustomUnittestSuite() methods would be complex to port, and are only
> used for the [currently unused] accessibility branch.

Hopefully if we need something like this in the future it will be simpler to add.
(In reply to David Bolter [:davidb] from comment #3)
> (In reply to Aki Sasaki [:aki] from comment #0)
> > Not used since April last year.
> > Pinged #accessibility; seems like they don't use it.
> > 
> > We should get a sign-off here before doing this.
> > 
> > Marking as blocking bug 747382 since the addSuite() and
> > loadCustomUnittestSuite() methods would be complex to port, and are only
> > used for the [currently unused] accessibility branch.
> 
> Hopefully if we need something like this in the future it will be simpler to
> add.

Yeah, I think we can add it lower in the config, as branch-specific configs, rather than the way it was currently implemented.
Attachment #712595 - Flags: review?(armenzg)
Attachment #712595 - Flags: review?(armenzg) → review+
Attachment #712664 - Flags: review?(philringnalda)
Comment on attachment 712664 [details] [diff] [review]
(tbpl) remove accessibility branch

The // deathwatch comment actually refers to devtools and jaegermonkey as well, r=me with it left in there (or, a far better thing, them killed too).
Attachment #712664 - Flags: review?(philringnalda) → review+
Bunch of mozconfigs can be deleted from buildbot-configs too iirc
Comment on attachment 712664 [details] [diff] [review]
(tbpl) remove accessibility branch

Left the comment in.
https://hg.mozilla.org/webtools/tbpl/rev/3e56cfef78ce
Attachment #712664 - Flags: checked-in+
Attachment #712678 - Flags: review?(emorley) → review+
Also spotted entries in:
buildbot-configs/mozilla/b2g_project_branches.py
tools/buildfarm/maintenance/production-branches.json
Comment on attachment 712678 [details] [diff] [review]
hg rm */accessibility in buildbot-configs/mozilla2/

http://hg.mozilla.org/build/buildbot-configs/rev/e46693ead871
Attachment #712678 - Flags: checked-in+
Attachment #712709 - Flags: review?(emorley)
Attachment #712710 - Flags: review?(emorley)
Attachment #712709 - Flags: review?(emorley) → review+
Attachment #712710 - Flags: review?(emorley) → review+
In production.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Depends on: 842323
/pub/mozilla.org/firefox/nightly/latest-accessibility removed from ftp.m.o
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.