Closed Bug 687380 Opened 8 years ago Closed 8 years ago

Do not generate NSPR pkg/linux/* and pkg/solaris/* Makefiles on platforms where they will not be used


(NSPR :: NSPR, defect)

Not set


(Not tracked)



(Reporter: emorley, Assigned: emorley)



(1 file, 2 obsolete files)

Building Firefox locally using MSVC 2010 with

Output from NSPR configure:
creating pkg/linux/Makefile
creating pkg/solaris/Makefile
creating pkg/solaris/SUNWpr/Makefile
creating pkg/solaris/SUNWprd/Makefile

/nsprpub/pkg/ specifies the following:
ifeq ($(OS_TARGET),Linux)
DIRS = linux
ifeq ($(OS_TARGET),SunOS)
DIRS = solaris

So it would make sense to add something similar to configure:
Attached patch Patch v1 (obsolete) — Splinter Review
Adds the same platform ifdefs to the configure makefiles output generation for pkg/linux/* and pkg/solaris/* as those found in pkg/

Note: The patch was based off of mozilla-inbound, but this is intended for upstream NSPR. I decided against figuring out how to use CVS until after it's clear whether the patch will be r+'d. 

Thanks :-)
Attachment #560829 - Flags: review?(ted.mielczarek)
Attached patch Patch v1.1 (obsolete) — Splinter Review
As above, but with brain engaged.
Attachment #560829 - Attachment is obsolete: true
Attachment #560829 - Flags: review?(ted.mielczarek)
Attachment #560830 - Flags: review?(ted.mielczarek)
Attachment #560830 - Flags: review?(ted.mielczarek) → review+
As previously r+'d, except:
* Is now a CVS diff.
* Removes the dnl lines since either dupes or else the makefiles to which they refer are NPOTB.
* Whitespace/wrapping/indentation improvements:
    -Removes trailing whitespace whilst I'm there
    -Splits the |MAKEFILES="$MAKEFILES foo/bar/Makefile"| entries onto 
     multiple lines for readability (also allows for easier parsing of 
     the generated makefiles for the script in bug 696498)
    -Adjusts indentation for readability (happy to take input on this 
     one, if you don't feel it's appropriate)

Presuming you're ok with the post r+ changes, can you land it for me please Ted :-)
Attachment #560830 - Attachment is obsolete: true
Checking in configure;
/cvsroot/mozilla/nsprpub/configure,v  <--  configure
new revision: 1.317; previous revision: 1.316
Checking in;
/cvsroot/mozilla/nsprpub/,v  <--
new revision: 1.319; previous revision: 1.318
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.9
You need to log in before you can comment on or make changes to this bug.