All users were logged out of Bugzilla on October 13th, 2018

mozilla-download never thinks $GAIA/b2g is up-to-date

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
9 months ago

People

(Reporter: gaye, Assigned: gaye)

Tracking

Details

Attachments

(2 attachments)

(Assignee)

Description

4 years ago
The issue is that if you have an out-of-date $GAIA/b2g we'll download new stuff and copy it into the directory instead of clobbering the existing b2g dir which doesn't update the b2g dir's timestamp (only the timestamp of its children) so make gets tripped up.
(Assignee)

Updated

4 years ago
Assignee: nobody → gaye
Created attachment 8593018 [details] [review]
[gaia] gaye:bug-1154901 > mozilla-b2g:master
(Assignee)

Comment 2

4 years ago
Created attachment 8593019 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/29529

Just a little patch to make sure that $GAIA/b2g timestamp gets updated when we run |make b2g|
Attachment #8593019 - Flags: review?(poirot.alex)
Comment on attachment 8593019 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/29529

I'm bit overloaded these days and Ricky has a better knowledge of the build system now.
Attachment #8593019 - Flags: review?(poirot.alex) → review?(ricky060709)
Attachment #8593019 - Flags: review?(ricky060709) → review+
Patch looks fine to me but I don't totally understand why we have to update b2g's timestamp?
(In reply to Ricky Chien [:rickychien] from comment #4)
> Patch looks fine to me but I don't totally understand why we have to update
> b2g's timestamp?

Make determines whether a target needs to be re-made based on the target's timestamps and the timestamps of its dependencies.  Without this fix, on OS X, the timestamp of b2g will never be updated after it's initially created.  However, we update node_modules all the time, so node_modules (as well as the current dep "node_modules/.bin/mozilla-download" that's being simplified to node_modules) can easily end up with more recent timestamps.  Once that happens, OS X users will find that every time "make b2g" is invoked (or anything depending on the "b2g" target), make will decide that it needs to rebuild the "b2g" target and re-run its rules which means re-downloading b2g.
(Assignee)

Comment 6

4 years ago
Thanks for explanation :asuth! https://github.com/mozilla-b2g/gaia/commit/3cd0a9facce26c2acc7be3755a17131a6358e33f landed on master
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Updated

9 months ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.