Closed Bug 381900 Opened 13 years ago Closed 12 years ago

Make a checkout target for the build system

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: bent.mozilla, Unassigned)

References

Details

Attachments

(1 file)

Spun off from bug 380846.

XULRunner apps that have a libxul-sdk shouldn't have to check out all the source code just to use the build system. We need to make a checkout target for just the build system and its dependencies.
We should factor that requirement with the l10n-checkout target in client.mk.
Attached file file list
With this list of files, I can build, package and install my XR app on:
* linux (debian/unstable);
* w32 (msvc 2005 Express + PSDK 2003 R2).
I doubt that many of those files are necessary, in particular the version.txt files and the nspr ones. You may get warnings or errors if those are missing, but that shouldn't be tragic.

I'm using a smaller list of files for l10n-checkout, http://mxr.mozilla.org/mozilla/source/client.mk#905, though I think it'd make sense to add mozilla/Makefile.in, too.
(In reply to comment #3)
> I doubt that many of those files are necessary, in particular the version.txt
> files and the nspr ones. You may get warnings or errors if those are missing,
> but that shouldn't be tragic.
> 
> I'm using a smaller list of files for l10n-checkout,
> http://mxr.mozilla.org/mozilla/source/client.mk#905, though I think it'd make
> sense to add mozilla/Makefile.in, too.

I agree that list is not minimal, but these 4 are critical:
./toolkit/mozapps/installer/package-name.mk
./toolkit/mozapps/installer/packager.mk
./xpcom/xpcom-config.h.in
./xpcom/xpcom-private.h.in

And not having nsprpub/configure means heavy workarounds like --with-system-nspr.
Blocks: 468835
The build system doesn't have support for checking anything out anymore.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.