Closed Bug 494490 Opened 15 years ago Closed 3 years ago

make install on linux installs the system_wrappers{_js} in includedir

Categories

(Firefox Build System :: General, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE
mozilla1.9.2a1

People

(Reporter: wuno, Assigned: wuno)

References

Details

Attachments

(2 files, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2a1pre) Gecko/20090522 Minefield/3.6a1pre
Build Identifier: 

after bug488175 the system_wrappers get installed in the includedir. Should they be added to removed-files or should be added --exclude=system_wrappers* to the respective tar command in toolkit/mozapps/install/packager.mk?

Reproducible: Always
I'd actually like to move system_wrappers out of dist/include into, say dist/system_wrappers
Assignee: nobody → wuno
Status: NEW → ASSIGNED
Attachment #379288 - Flags: review?(benjamin)
Attached patch patch for nspr (obsolete) — Splinter Review
probably Wan-Teh has to r as well
Attachment #379289 - Flags: review?(benjamin)
Attachment #379288 - Flags: review?(benjamin) → review+
Comment on attachment 379288 [details] [diff] [review]
patch for M-C and js
[Checkin: Comment 8]

Are you sure the NSPR patch is necessary?
Assignee: wuno → nobody
Product: Toolkit → Core
Attachment #379289 - Flags: review?(benjamin) → review?(wtc)
Attachment #379289 - Flags: review?(wtc)
Comment on attachment 379289 [details] [diff] [review]
patch for nspr

(In reply to comment #4)
> (From update of attachment 379288 [details] [diff] [review])
> Are you sure the NSPR patch is necessary?

Tried w/o the NSPR patch w/ and w/o --system-nspr
works w/o the nspr patch
Keywords: checkin-needed
Version: unspecified → Trunk
Attachment #379289 - Flags: review+
Comment on attachment 379289 [details] [diff] [review]
patch for nspr

r=wtc.  Since NSPR isn't using the system header wrappers,
this patch is modifying dead code.  Alternatively, we can
just delete the dead code.
(In reply to comment #6)
> Alternatively, we can
> just delete the dead code.
Well, then do it, couldn't then nsprpub/config/make-system-wrappers.pl cvs removed?
Attachment #380517 - Flags: review?(wtc)
Attachment #379289 - Attachment is obsolete: true
Assignee: nobody → wuno
Comment on attachment 379288 [details] [diff] [review]
patch for M-C and js
[Checkin: Comment 8]


http://hg.mozilla.org/mozilla-central/rev/5f14dd5ec14a
Attachment #379288 - Attachment description: patch for M-C and js → patch for M-C and js [Checkin: Comment 8]
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla1.9.2a1
Depends on: 488175
Wan-Teh,
do you have any preferences modification (https://bugzilla.mozilla.org/attachment.cgi?id=379289) vs. removal (https://bugzilla.mozilla.org/attachment.cgi?id=380517) of the dead code?
Product: Core → Firefox Build System

This is 12 years old, and I'm not sure if the context still applies. Closing :)

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.