Closed
Bug 296507
Opened 20 years ago
Closed 20 years ago
RDF resource examples shouldn't be built by default
Categories
(Core Graveyard :: RDF, defect)
Core Graveyard
RDF
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: sipaq, Assigned: sipaq)
References
()
Details
Attachments
(2 files)
|
598 bytes,
patch
|
axel
:
review+
shaver
:
superreview+
shaver
:
approval1.8b3+
|
Details | Diff | Splinter Review |
|
652 bytes,
patch
|
axel
:
review+
sipaq
:
superreview+
asa
:
approval1.8b3+
|
Details | Diff | Splinter Review |
Every mozilla application (FF, TB, XULrunner, Sunbird, Suite) packages the following files by default in the $application$/res/rdf/ directory: - folder-closed.gif - folder-open.gif - loading.gif - dom-test-4.css - ignore-test.xul This is unnecessary bloat and should only be done for a debug-build or when you have set --enable-tests.
| Assignee | ||
Comment 1•20 years ago
|
||
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #185277 -
Flags: review?(axel)
Attachment #185277 -
Flags: approval1.8b3?
| Assignee | ||
Updated•20 years ago
|
Attachment #185277 -
Flags: superreview?(shaver)
Comment 2•20 years ago
|
||
Comment on attachment 185277 [details] [diff] [review] Remove resources directory from makefile (checked in) To those wondering, this is just a test directory. I need to go hunting if those still work one day. The impl of nsRDFResource is in util. (yay)
Attachment #185277 -
Flags: review?(axel) → review+
Comment 3•20 years ago
|
||
Comment on attachment 185277 [details] [diff] [review] Remove resources directory from makefile (checked in) sr+a=shaver
Attachment #185277 -
Flags: superreview?(shaver)
Attachment #185277 -
Flags: superreview+
Attachment #185277 -
Flags: approval1.8b3?
Attachment #185277 -
Flags: approval1.8b3+
Attachment #185277 -
Attachment description: Remove resources directory from makefile → Remove resources directory from makefile (checked in)
| Assignee | ||
Comment 4•20 years ago
|
||
Fixed by checkin from CTho. Thanks Chris.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 5•20 years ago
|
||
*** Bug 179414 has been marked as a duplicate of this bug. ***
| Assignee | ||
Comment 6•20 years ago
|
||
| Assignee | ||
Updated•20 years ago
|
Attachment #185402 -
Flags: superreview?(dmose)
Attachment #185402 -
Flags: review?(axel)
Comment 7•20 years ago
|
||
Comment on attachment 185402 [details] [diff] [review] Remove rdf/resources from allmakefiles.sh also sr=dmose
Attachment #185402 -
Flags: superreview?(dmose) → superreview+
Comment 8•20 years ago
|
||
Comment on attachment 185402 [details] [diff] [review] Remove rdf/resources from allmakefiles.sh also sr=dmose
Updated•20 years ago
|
Attachment #185402 -
Flags: superreview?
Attachment #185402 -
Flags: superreview+
Attachment #185402 -
Flags: review?(axel)
Attachment #185402 -
Flags: review+
| Assignee | ||
Updated•20 years ago
|
Attachment #185402 -
Flags: approval1.8b3?
Updated•20 years ago
|
Attachment #185402 -
Flags: approval1.8b3? → approval1.8b3+
| Assignee | ||
Comment 9•20 years ago
|
||
Comment on attachment 185402 [details] [diff] [review] Remove rdf/resources from allmakefiles.sh also dmose already sr'ed this. It was probably overwritten when pike r'ed this patch.
Attachment #185402 -
Flags: superreview? → superreview+
Comment 10•20 years ago
|
||
second patch checked in Checking in allmakefiles.sh; /cvsroot/mozilla/allmakefiles.sh,v <-- allmakefiles.sh new revision: 1.565; previous revision: 1.564 done
Comment 11•20 years ago
|
||
I actually cvs removed those files now, a=shaver on irc.
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•