Closed Bug 968806 Opened 10 years ago Closed 10 years ago

Touching WebIDL.py, Codegen.py or Configuration.py does not cause the WebIDL bindings to be regenerated

Categories

(Firefox Build System :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: ehsan.akhgari, Unassigned)

Details

STR:

./mach build
touch dom/bindings/parser/WebIDL.py dom/bindings/Codegen.py dom/bindings/Configuration.py
./mach build

Found this out when working on bug 968643.
Did you just touch them, or change something?  I think the setup gps put in place in bug 928195 uses file hashes, not file dates, to decide whether the files changed...
No-op is no-op:

gps@gps-mbp:~/src/firefox$ mach build -v dom/bindings/export
 0:00.08 /usr/bin/make -C /Users/gps/src/firefox/obj-firefox.noindex -j8 -s backend.RecursiveMakeBackend
 0:00.51 /usr/bin/make -C dom/bindings -j8 export
 0:00.53 /Applications/Xcode.app/Contents/Developer/usr/bin/make -C test export
 0:00.53 make[1]: Nothing to be done for `export'.
Your build was successful!

touching a .py file invokes WebIDL processing:

gps@gps-mbp:~/src/firefox$ touch dom/bindings/parser/WebIDL.py
gps@gps-mbp:~/src/firefox$ mach build -v dom/bindings/export
 0:00.09 /usr/bin/make -C /Users/gps/src/firefox/obj-firefox.noindex -j8 -s backend.RecursiveMakeBackend
 0:00.51 /usr/bin/make -C dom/bindings -j8 export
 0:00.53 /Users/gps/src/firefox/obj-firefox.noindex/_virtualenv/bin/python -m mozbuild.action.webidl /Users/gps/src/firefox/dom/bindings
 0:03.01 /Applications/Xcode.app/Contents/Developer/usr/bin/make -C test export
 0:03.02 make[1]: Nothing to be done for `export'.
Your build was successful!

I'm unable to reproduce the bug.

Please provide more details about what isn't working.
Flags: needinfo?(ehsan)
Ah, I was fooled by this setup it seems.  Sorry for the false alarm.
No longer blocks: clobber
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(ehsan)
Resolution: --- → INVALID
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.