Closed Bug 296507 Opened 19 years ago Closed 19 years ago

RDF resource examples shouldn't be built by default

Categories

(Core Graveyard :: RDF, defect)

Product:
Core Graveyard
Component:
RDF
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: sipaq, Assigned: sipaq)

References

(Blocks 1 open bug,
URL
)

Details

Attachments

(2 files)

Remove resources directory from makefile (checked in)
598 bytes, patch
axel
: review+
shaver
: superreview+
shaver
: approval1.8b3+
Details | Diff | Splinter Review
Remove rdf/resources from allmakefiles.sh also
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: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #185277 - Flags: review?(axel)
Attachment #185277 - Flags: approval1.8b3?
Attachment #185277 - Flags: superreview?(shaver)
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 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)
Fixed by checkin from CTho. Thanks Chris.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
*** Bug 179414 has been marked as a duplicate of this bug. ***
Attachment #185402 - Flags: superreview?(dmose)
Attachment #185402 - Flags: review?(axel)
Comment on attachment 185402 [details] [diff] [review] Remove rdf/resources from allmakefiles.sh also sr=dmose
Attachment #185402 - Flags: superreview?(dmose) → superreview+
Comment on attachment 185402 [details] [diff] [review] Remove rdf/resources from allmakefiles.sh also sr=dmose
Attachment #185402 - Flags: superreview?
Attachment #185402 - Flags: superreview+
Attachment #185402 - Flags: review?(axel)
Attachment #185402 - Flags: review+
Attachment #185402 - Flags: approval1.8b3?
Attachment #185402 - Flags: approval1.8b3? → approval1.8b3+
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+
second patch checked in Checking in allmakefiles.sh; /cvsroot/mozilla/allmakefiles.sh,v <-- allmakefiles.sh new revision: 1.565; previous revision: 1.564 done
I actually cvs removed those files now, a=shaver on irc.
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: