37355 - Minor cleanups for Mac pull/build scripts

Status: VERIFIED FIXED

(SeaMonkey :: Build Config, enhancement, P5)

Description

[Wan-Teh Chang]

I'm going to attach a patch that cleans up
the Mac pull/build scripts a little bit.
Feel free to throw away what you think is
wrong.

Comment 1

[Wan-Teh Chang]

Attachment: Proposed patch.

Comment 2

[Wan-Teh Chang]

Comments on the patch (id=8030).
1. BuildNGLayout.pl: I believe the NECKO build
   variable is obsolete.
2. BuildNGLayoutDebug.pl: it would be easier for
   me to override the values in the $build{} array
   for NSPR builds if the $build{} array entries were
   initialized to 0, because to do NSPR builds, I
   only need to set three entries to 1 (dist_runtime,
   stubs, and runtime).  Note that all the entries
   in $build{} are initialized to 0 in BuildNGLayout.pl.
3. PullNGLayout.pl: fixed comments and removed the
   setting of unused variable DEBUG.

Comment 3

[J.J. Enser]

Simon, feel free to reassign this to me if you want. I can (and should) take care 
of this, but I _just_ got added to the cc list. (thanks Jon)

Comment 4

[Simon Fraser [no longer active]]

Doing so

Comment 5

[J.J. Enser]

Didn't check this in yet cuz i have other related/similar changes in my tree.
Will do so after merging them all.

Comment 6

[J.J. Enser]

fixed. checked in with other minor adjustement between optimized/debug scripts.

Comment 7

[Jon Granrose]

verified.