Closed
Bug 1229226
Opened 9 years ago
Closed 9 years ago
Some cleanup around RESOURCE_FILES
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox45 fixed)
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files)
14.40 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
1.42 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
No resource file is preprocessed in gecko, and no resource file is defined in comm-central.
Attachment #8693883 -
Flags: review?(gps)
Assignee | ||
Comment 2•9 years ago
|
||
Gecko only uses resource files in the GRE directory.
Attachment #8693885 -
Flags: review?(gps)
Updated•9 years ago
|
Attachment #8693883 -
Flags: review?(gps) → review+
Comment 3•9 years ago
|
||
Comment on attachment 8693885 [details] [diff] [review] Reject RESOURCE_FILES along DIST_SUBDIR or XPI_NAME Review of attachment 8693885 [details] [diff] [review]: ----------------------------------------------------------------- ::: python/mozbuild/mozbuild/frontend/emitter.py @@ +643,5 @@ > if resources: > + if context.get('DIST_SUBDIR') or context.get('XPI_NAME'): > + raise SandboxValidationError( > + 'RESOURCES_FILES is not allowed along DIST_SUBDIR or ' > + 'XPI_NAME.', context) This doesn't read very well to me. How about "RESOURCES_FILES cannot be used with DIST_SUBDIR or XPI_NAME."
Attachment #8693885 -
Flags: review?(gps) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/ff6fafc6b72a https://hg.mozilla.org/integration/mozilla-inbound/rev/133d8a4c9e5d
Comment 5•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ff6fafc6b72a https://hg.mozilla.org/mozilla-central/rev/133d8a4c9e5d
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•