[Python-bugs-list] [ python-Bugs-688007 ] Bundlebuilder needs to pre-convert resource files

SourceForge.net noreply@sourceforge.net
Tue, 18 Feb 2003 07:08:02 -0800


Bugs item #688007, was opened at 2003-02-17 15:21
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=688007&group_id=5470

Category: Macintosh
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Jack Jansen (jackjansen)
Assigned to: Just van Rossum (jvr)
Summary: Bundlebuilder needs to pre-convert resource files

Initial Comment:
If bundlebuilder encounters an applesingle-encoded resource file it should pre-convert this to a datafork-based resource file.

At the moment the file is copied to the Resources directory verbatim, and this can cause problems if you use "sudo make frameworkinstall" to install Python: the resource folder will be mode 755 owned by root, and any normal user trying to run the applet will get a crash when the applet tries to unpack the applesingle .rsrc file into the datafork resource file .rsrc.df.rsrc.

----------------------------------------------------------------------

>Comment By: Just van Rossum (jvr)
Date: 2003-02-18 16:08

Message:
Logged In: YES 
user_id=92689

Aren't datafork res files supported in OS9? If so, this might be an opportunity to get rid of dataforks (as well as macresource.py) altogether...

----------------------------------------------------------------------

Comment By: Jack Jansen (jackjansen)
Date: 2003-02-18 15:04

Message:
Logged In: YES 
user_id=45365

I agree on a theoretical level, but not on a practical one. The .rsrc files in the CVS repository are AppleSingle encoded, and I can't think of another way to keep them useable on both MacOS9 and MacOSX.

Any suggestions?

----------------------------------------------------------------------

Comment By: Just van Rossum (jvr)
Date: 2003-02-18 13:32

Message:
Logged In: YES 
user_id=92689

Yes, it must be compatible with 2.2 (it also ships with PyObjC). bundlebuilder.py was deliberately written to have very few dependencies: plistlib.py is the only dependency outside the std lib.

I'm not sure I like this patch to begin with: for one it seems it's doing this resource processing at the wrong abstraction level, it adds a special case for something that I don't like being treated specially within bundlebuilder. It would be much better if this preprocessing was done elsewhere.

In other words: I think the bug is in attempting to write the converted file to begin with, not with bundlebuilder not doing the conversion.

----------------------------------------------------------------------

Comment By: Jack Jansen (jackjansen)
Date: 2003-02-17 17:56

Message:
Logged In: YES 
user_id=45365

Fixed in bundlebuilder.py rev 1.8, macresource rev. 1.3.

Just, I'm assigning this to you because I'm not sure whether you still keep bundlebuilder 2.2-compatible. If so you may need to disable this bit of magic if macresource.install isn't available. Please close the report when you're happy.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=688007&group_id=5470