Unable to run xpcshell tests with tup build backend enabled

RESOLVED FIXED in Firefox 63

Status

defect
RESOLVED FIXED
11 months ago
10 months ago

People

(Reporter: ato, Assigned: chmanchester)

Tracking

(Blocks 1 bug)

Trunk
mozilla63
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

11 months ago
Trying to run xpcshell tests with the tup build backend enabled, I
get an error saying the CLOBBER file has been updated:

> % ./mach xpcshell-test --sequential testing/marionette/test/unit/test_sync.js
> 
> The build system was unable to install tests because the CLOBBER file
> has been updated. This means if you edited any test files, your
> changes may not be picked up until a full/clobber build is performed.
> 
> The easiest and fastest way to perform a clobber build is to run:
> 
>  $ mach clobber
>  $ mach build
> 
> If you did not modify any test files, it is safe to ignore this
> message and proceed with running tests. To do this run:
> 
>  $ touch /home/ato/src/gecko/obj-x86_64-pc-linux-gnu/CLOBBER
Reporter

Updated

11 months ago
Blocks: buildtup
Reporter

Updated

11 months ago
Summary: Unabel to run xpcshell tests with tup build backend enabled → Unable to run xpcshell tests with tup build backend enabled
Assignee

Comment 1

11 months ago
This is related to bug 1478798. This should work with the proposed patch there and an additional provision to re-run configure when CLOBBER changes, which we should probably be doing anyway.
Assignee

Comment 2

11 months ago
In the meantime, touching CLOBBER before proceeding is probably safe.
Assignee

Updated

10 months ago
Assignee: nobody → cmanchester
Comment on attachment 8998024 [details]
Bug 1480771 - Take CLOBBER into account when deciding whether to re-run configure.

Gregory Szorc [:gps] has approved the revision.

https://phabricator.services.mozilla.com/D2802
Attachment #8998024 - Flags: review+

Comment 5

10 months ago
Pushed by cmanchester@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/536829999219
Take CLOBBER into account when deciding whether to re-run configure. r=gps,firefox-build-system-reviewers
Assignee

Comment 6

10 months ago
To give some more context to this fix: bug 1478798 along with this change should keep the clobber file from getting in peoples way at all when building with tup, but in the meantime this change means ./mach build followed by running tests will no longer cause this problem, ./mach build will instead manage and keep the clobber file up to date.

Comment 7

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/536829999219
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.