Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 302144

Summary: Darwin (OS X 10.5) fix for dev-lang/parrot
Product: Gentoo/Alt Reporter: calculuspenguin
Component: Prefix SupportAssignee: Gentoo Prefix <prefix>
Status: RESOLVED FIXED    
Severity: normal    
Priority: High    
Version: unspecified   
Hardware: All   
OS: OS X   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: OS X 10.5 patch

Description calculuspenguin 2010-01-25 06:37:49 UTC
Parrot fails to configure on Mac OS X 10.5 due to passing -Wno-long-double when creating a test program.  The flag -Wno-long-double does not appear in the gcc man page and results in an "unrecognized command line option" error when used.

Also, parrot does not support staged installs properly.  Its dynamically linked libraries are in the wrong location when parrot is installed in the final system.  Once patched, consider adding ~x86-macos to the keyword list and possibly whitelisting this package (only differences are using $EPREFIX during configure and install)
Comment 1 calculuspenguin 2010-01-25 06:39:39 UTC
Created attachment 217366 [details]
OS X 10.5 patch

Attached patch to fix above mentioned issues (configure flag and staged installs).
Comment 2 Fabian Groffen gentoo-dev 2010-01-25 20:38:28 UTC
Slightly adapted the patch to always apply the -Wno-double removal (since we all use 4.2.1) and to skip looking for fink and macports crap.

Did you grab the patch from somewhere else?  Otherwise, the staged stuff is that worth reporting upstream?
Comment 3 calculuspenguin 2010-01-25 20:48:32 UTC
I created the patch myself after several hours of looking through the makefiles and other configure related files to see what needed changing.  I decided to take this approach instead of using install_name_tool after everything was already built.

For reference, the parrot build available from MacPorts uses install_name_tool.
Comment 4 Fabian Groffen gentoo-dev 2010-01-25 21:03:38 UTC
very good then, think you should bring it upstream and give us the reference so we can put it in the patch :)