packages-static should be auto-generated

NEW
Unassigned

Status

()

Core
Build Config
9 years ago
2 months ago

People

(Reporter: sdwilsh, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 years ago
We should be building packages-static automatically with data from things like EXTRA_JS_MODULES, EXTRA_COMPONENTS, etc.

Far too often developers forget to add to packages-static and nightlies don't work.

Comment 1

9 years ago
If we do this, why do we need packages-static at all? Why not just make the file list from whatever shows up in dist/bin?
(In reply to comment #1)
> If we do this, why do we need packages-static at all? Why not just make the
> file list from whatever shows up in dist/bin?

The main use I've seen for packages-static is to bundle the xpt files up into a few files. We'd have to change that just to bundle every xpt in components/ into a single file.

There would also have to be verification that what is currently being pushed into the dist/bin directory is the same as packages-static before it gets removed (but IMHO that's a good thing).

My only concern would be that the default is for enabling tests, which would mean that all the test files would get packaged as well (it would certainly be nice to run tests against the builds we actually ship, and this wouldn't help that).

Comment 3

9 years ago
For the most part I don't think we should be shipping test files into dist/bin... but there's not currently a way to run the binary-only tests unless you put them in dist/bin.
(Reporter)

Comment 4

9 years ago
would disabling tests for release builds accomplish that?
(In reply to comment #4)
> would disabling tests for release builds accomplish that?
> 
Yes, but I'd like to see release builds have the tests run against them. Look at the nightlies - we ship them, even though all the unit test boxes could be burning away with critical errors.
I would definitely like to stop putting test cruft in dist/bin. I think on Mac/Linux run-mozilla.sh should make this work, but we'd probably be broken on Windows. According to this document, however:
http://msdn.microsoft.com/en-us/library/7d83bc18(VS.80).aspx

The current directory and all the directories in PATH are both searched, so we could either run the test programs with CWD=dist/bin, or stick dist/bin into PATH for the test run.
You need to log in before you can comment on or make changes to this bug.