Opened 11 years ago

Last modified 11 years ago

#38229 new defect

birdfont: LSOpenURLsWithRole() failed with error -10810 for the file /Applications/MacPorts/BirdFont.app.

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: johan.mattsson@…
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc:
Port: birdfont

Description

Trying to open the birdfont application fails with:

$ open /Applications/MacPorts/BirdFont.app
LSOpenURLsWithRole() failed with error -10810 for the file /Applications/MacPorts/BirdFont.app.

This is because /Applications/MacPorts/BirdFont.app/Contents/MacOS/birdfont.sh does not have the execute bit set.

There is a bug in the install script, which runs:

	subprocess.check_call ('cp -r build/BirdFont.app ' + dest + '/Applications/', shell=True)

First, the use of the cp "-r" option is "strongly discouraged" according to the manpage. Use "-R" instead. Also, use "-p" to preserve permissions. (Don't use "-a", which combines -R and -P and -p, because although that's what you want, it doesn't exist on Tiger.) But this is not the cause of the problem in our case because in the MacPorts portfile we don't use that; we use a post-destroot block to copy the app manually. So the wrong permissions are already happening on that file during the build phase.

Attachments (2)

patch-portfile.diff (982 bytes) - added by johanmattssonm (Johan Mattsson) 11 years ago.
maintainer e-mail and chmod for birdfont.sh
patch-build_and_install.diff (10.6 KB) - added by johanmattssonm (Johan Mattsson) 11 years ago.
chmod for birdfont.sh

Download all attachments as: .zip

Change History (4)

comment:1 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Quick fix in r103514 but a better upstream fix should be made.

Changed 11 years ago by johanmattssonm (Johan Mattsson)

Attachment: patch-portfile.diff added

maintainer e-mail and chmod for birdfont.sh

Changed 11 years ago by johanmattssonm (Johan Mattsson)

chmod for birdfont.sh

comment:2 Changed 11 years ago by johanmattssonm (Johan Mattsson)

Thank you. A new fix is available upstream. patch-build_and_install.diff is the diff from HEAD in the git repository.

Last edited 11 years ago by johanmattssonm (Johan Mattsson) (previous) (diff)
Note: See TracTickets for help on using tickets.