provide commit message hook to trigger clobber builds

RESOLVED WONTFIX

Status

()

Core
Build Config
P5
normal
RESOLVED WONTFIX
7 years ago
6 years ago

People

(Reporter: blassey, Unassigned)

Tracking

Other Branch
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [automation][clobber])

Sometimes you push knowing that your changes will need a clobber. The most common for me are changes to nspr, but I'm sure there are others. Self service clobbers can help, but when those changes get merged to other branches it still causes pain. So it would be useful if the commit message could indicate to the builders that a clobber will be necessary.
We already have https://build.mozilla.org/clobberer/ which should serve this need.
(In reply to comment #1)
> We already have https://build.mozilla.org/clobberer/ which should serve this
> need.

That's what I meant bey self-service clobbers, and while that is great (thanks!) I explained in the description why that isn't entirely sufficient.
Sorry, I'm a dope who didn't read the summary! This stands as a valid bug, then.

Updated

7 years ago
Priority: -- → P5
Whiteboard: [automation][clobber]
I would rather move this into the build system itself so that developers that pick up these changes get a clobber too.

I think we could accomplish this by making config.status depend on a "CLOBBER" file that we changed when necessary and a special rule to blow away the object directory.
That sounds like a better solution to me too.
ditto
Component: Release Engineering → Build Config
Product: mozilla.org → Core
QA Contact: release → build-config
Version: other → Other Branch
Blocks: 661800
No longer blocks: 661800

Comment 7

6 years ago
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #4)
> I would rather move this into the build system itself so that developers
> that pick up these changes get a clobber too.
> 
> I think we could accomplish this by making config.status depend on a
> "CLOBBER" file that we changed when necessary and a special rule to blow
> away the object directory.

Bug 717372 has been filed for this.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.