Issue 111498

Summary: python: Build framework for internal python on Mac (create python_mac module?)
Product: Build Tools Reporter: lohmaier
Component: external prerequisitesAssignee: AOO issues mailing list <issues>
Status: CONFIRMED --- QA Contact:
Severity: Trivial    
Priority: P3 CC: issues
Version: 4.0.0-dev   
Target Milestone: ---   
Hardware: All   
OS: Mac OS X, all   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on: 111501    
Issue Blocks: 111496, 111500    
Attachments:
Description Flags
patch to build a python framework for use with Mac OSX
none
d.lst entries needed for mac - no patch, but replacement for better overview.
none
revised patch, uses install_name_tool, doesn't deliver unnecessary stuff to solver (on Mac)
none
remove reference to buildinstalldir from scripts, honor build-verbosity in rules none

Description lohmaier 2010-05-09 21:32:10 UTC
Building internal python for Mac requires building python as framework (at least 
I didn't manage to get it working with non-framework configuration, and python's 
readme reads: 

"…A framework install is probably needed if you want to use any Aqua-based GUI 
toolkit (whether Tkinter, wxPython, Carbon, Cocoa or anything else).…"

The "problem" now is that the framework will have a different structure than a 
"standard" build, resulting in the need for massive additions/changes to 
python/prj/d.lst

And as there is no recursive copy action and no conditionals in d.lst files 
(AFAIK), I wonder whether an additional python_mac module would be more 
appropriate.
Comment 1 lohmaier 2010-05-09 22:30:11 UTC
Created attachment 69380 [details]
patch to build a python framework for use with Mac OSX
Comment 2 lohmaier 2010-05-09 22:35:49 UTC
The attached patch contains the modifications excluding those of prj/d.lst (will 
be seperate attachment)

Note that I moved the generation of the python zip (for building the installset) 
from pyuno to python, as it seems more naturally to generate it straight from 
the directory where it is created, than to copy it from solver to pyuno just to 
zip it there, and copy it back to solver.

Also note that the default patch was modified to not only try the 
_POSIX_SEMAPHORES test on FreeBSD, as Mac OSX (10.4) also needs the 
HAVE_BROKEN_POSIX_SEMAPHORES (has the _POSIX_SEMAPHORES defined to -1)

Last note: I use a custom name for the framework (OOoPython instead of Python) - 
python on Mac is a story full of frustration because the different versions 
aren't clearly seperated, and it's easy to create python stuff that is linked 
agains the wrong version. Using a dedicated name solves this problem and eases 
troubleshooting. (has no impact on the user, user will just continue to run 
"python" executable/launcher script.
Comment 3 lohmaier 2010-05-09 22:42:03 UTC
Created attachment 69381 [details]
d.lst entries needed for mac - no patch, but replacement for better overview.
Comment 4 lohmaier 2010-05-10 18:48:12 UTC
Created attachment 69402 [details]
revised patch, uses install_name_tool, doesn't deliver unnecessary stuff to solver (on Mac)
Comment 5 lohmaier 2010-05-12 12:56:11 UTC
Created attachment 69430 [details]
remove reference to buildinstalldir from scripts, honor build-verbosity in rules
Comment 6 lohmaier 2010-05-12 12:57:16 UTC
type → PATCH
Comment 7 Rob Weir 2013-03-11 15:02:48 UTC
I'm adding this comment to all open issues with Issue Type == PATCH.  We have 220 such issues, many of them quite old.  I apologize for that.  

We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0.

If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know.

On the other hand, if the patch is no longer relevant, please let us know that as well.

If you have any general questions or want to discuss this further, please send a note to our dev mailing list:  dev@openoffice.apache.org

Thanks!

-Rob
Comment 8 Marcus 2017-05-20 11:27:59 UTC
Reset assigne to the default "issues@openoffice.apache.org".