Old (1.9.0/1.8.1) Default Plugin.plugin Makefile uses non-portable "echo -n"

RESOLVED FIXED

Status

()

Core
Plug-ins
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Smokey Ardisson (offline for a while; not following bugs - do not email), Assigned: Smokey Ardisson (offline for a while; not following bugs - do not email))

Tracking

({verified1.8.1.24, verified1.9.0.16})

1.9.0 Branch
All
Mac OS X
verified1.8.1.24, verified1.9.0.16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 407892 [details] [diff] [review]
Replace "echo -n" with "printf"

The old Default Plugin.plugin's Makefile use the non-portable "echo -n" syntax in its Makefile to create its PkgInfo file; this results in a bogus string in PkgInfo when building on 10.5 (see bug 501457, where this behavior was first diagnosed).

Because of the possibility of Camino being in a position where we have to do our 1.9.0 and any remaining 1.8.1 releases on a 10.5 box, I'd like to get this trivial patch for Default Plugin.plugin in the tree on those branches.
Attachment #407892 - Flags: review?(joshmoz)

Updated

8 years ago
Attachment #407892 - Flags: review?(joshmoz) → review+
Comment on attachment 407892 [details] [diff] [review]
Replace "echo -n" with "printf"

Requesting approval for 1.9.0.16 and 1.8.1.next for this trivial patch to ensure the Mac Default Plugin builds correctly on 10.5.  While other apps build and ship the Default Plugin, PFS overrides it, so Camino is the only actual consumer of this plugin, and this ensures we'll be able to build proper releases on 10.5 tinderboxen on these branches (the output of the two commands is identical on 10.4; see bug 501457).
Attachment #407892 - Flags: approval1.9.0.16?
Attachment #407892 - Flags: approval1.8.1.next?
Comment on attachment 407892 [details] [diff] [review]
Replace "echo -n" with "printf"

Approved for 1.9.0.16 and 1.8.1.24. a=ss for this minor change.
Attachment #407892 - Flags: approval1.9.0.16?
Attachment #407892 - Flags: approval1.9.0.16+
Attachment #407892 - Flags: approval1.8.1.next?
Attachment #407892 - Flags: approval1.8.1.next+
Checking in modules/plugin/samples/default/mac/Makefile.in;
/cvsroot/mozilla/modules/plugin/samples/default/mac/Makefile.in,v  <--  Makefile.in
new revision: 1.23; previous revision: 1.22
done

Checking in modules/plugin/samples/default/mac/Makefile.in;
/cvsroot/mozilla/modules/plugin/samples/default/mac/Makefile.in,v  <--  Makefile.in
new revision: 1.13.4.2; previous revision: 1.13.4.1
done
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: fixed1.8.1.24, fixed1.9.0.16
Resolution: --- → FIXED
In the event anyone comes along and is concerned with verifying this fix, the way to do so is to grab tinderbox-builds for each branch from Camino's 10.5 tinderbox[1], and view the contents of Camino.app/Contents/MacOS/plugins/Default Plugin.plugin/Contents/PkgInfo (should be NSPLMOSS).  

For added verification, you could then compare to official nightlies (built on 10.4 boxen) for 1.8.1 (latest-Cm1.6-M1.8/) and 1.9.0 (latest-Cm2.1-M1.9/) and ensure they also still have the right value.

[1] http://ftp.mozilla.org/pub/mozilla.org/camino/tinderbox-builds/cb-xserve04-1.6-M1.8/
http://ftp.mozilla.org/pub/mozilla.org/camino/tinderbox-builds/cb-xserve04-2.1-M1.9/
Verified for 1.9.0.16 by looking at my 1.9.0.16 build on OS X.
Keywords: fixed1.9.0.16 → verified1.9.0.16
Verified for 1.8.1.24 using Camino builds (Version 1.6.11pre (1.8.1.24pre 2010022513)).
Keywords: fixed1.8.1.24 → verified1.8.1.24
You need to log in before you can comment on or make changes to this bug.