Closed Bug 752917 Opened 12 years ago Closed 11 years ago

"make export" shouldn't copy IDL files/exports unless necessary

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 850380

People

(Reporter: joey, Unassigned)

Details

Add makefile deps to only install content when sources are modified.
$obj/mobile/android/base/components will do work on subsequent calls.

gmake -C components export
gmake[2]: Entering directory `/local/mozilla/bugs/clean/objdir-droid/mobile/android/components'
/local/mozilla/bugs/clean/objdir-droid/config/nsinstall -R -m 644 /local/mozilla/bugs/clean/mobile/android/components/SessionStore.idl ../../../dist/idl
/local/mozilla/bugs/clean/objdir-droid/config/nsinstall -R -m 644 _xpidlgen/SessionStore.h ../../../dist/include
gmake[2]: Leaving directory `/local/mozilla/bugs/clean/objdir-droid/mobile/android/components'
This is just a general problem with the build system, not specific to mobile builds, FWIW. I don't think it's worth tackling as part of the mobile build effort, but it would be nice to fix eventually. (Also it's possible there's already a bug filed on this, I didn't look.)

http://mxr.mozilla.org/mozilla-central/source/config/rules.mk#1488
Summary: mobile/android/components: dependency build should be a nop → "make export" shouldn't copy IDL files/exports unless necessary
I'm pretty sure the patch in bug 850380 addresses this.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.