Opened 20 years ago

Closed 19 years ago

#2875 closed defect (fixed)

Generated Installer pkgs should not have '&' in IFPkgDescriptionDescription

Reported by: pucklock@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: base Version: 1.0
Keywords: Cc:
Port:

Description

Portfiles must exclude the use of ampersand (&) in description field because Installer.app packages become corrupted and un-installable. The unrar portfile is an example (but there was 1 other a while ago, I just didn't know the fix, and can't remember):

port pkg unrar -v

will compile and package up just fine, but you will get "Welcome to %@ installer" in Installer.app - and you have to quit out, you are locked out of pressing any buttons. The portfile description field gets embedded into the packages some_package.pkgContents/Resources/Description.plist in the IFPkgDescriptionDescription key. It is this key that causes the problem. Changing the "&" there to "and" fixes the package - so changing the portfile will do/does the same.

fake, pseudo diff -u archivers/unrar :


# $Id: Portfile,v 1.8 2005/04/04 13:48:36 toby Exp $

PortSystem 1.0 name unrar version 3.5.1 categories archivers maintainers max@… -description Extract, view & test RAR archives +description Extract, view and test RAR archives

ps, I emailed toby the day after he made this portfile, and I posted this in the irc channel.

Attachments (2)

xml.patch (2.7 KB) - added by cjr@… 19 years ago.
Patch for portpkg.tcl and portmpkg.tcl
xml.2.patch (2.7 KB) - added by cjr@… 19 years ago.
Revised patch to base

Download all attachments as: .zip

Change History (8)

comment:1 Changed 20 years ago by toby@…

Why not fix the packaging stuff? & is a total valid character for Portfiles.

comment:2 Changed 20 years ago by pucklock@…

(In reply to comment #1)

Why not fix the packaging stuff? & is a total valid character for Portfiles.

& may be valid for portfiles, but it is NOT valid in the IFPkgDescriptionDescription key of ANY package. Go add "&" to that key into a non-Darwinports package & see what happens - its an Apple Installer.app limitation.

comment:3 Changed 19 years ago by snu@…

Component: dportsbase
Summary: portfile "description" field should never have the "&" charGenerated Installer pkgs should not have '&' in IFPkgDescriptionDescription

This is surely a base bug.

Changed 19 years ago by cjr@…

Attachment: xml.patch added

Patch for portpkg.tcl and portmpkg.tcl

Changed 19 years ago by cjr@…

Attachment: xml.2.patch added

Revised patch to base

comment:4 Changed 19 years ago by cjr@…

attachments.isobsolete: 01

comment:5 Changed 19 years ago by cjr@…

Can anyone with a base commit bit please review and (hopefully) apply this?

comment:6 Changed 19 years ago by olegb@…

Resolution: fixed
Status: newclosed

Thanks Chris ...

Note: See TracTickets for help on using tickets.