Skip to content
This repository has been archived by the owner on Nov 13, 2023. It is now read-only.

Initial Lion Support #3

Closed
wants to merge 1 commit into from
Closed

Initial Lion Support #3

wants to merge 1 commit into from

Conversation

VxJasonxV
Copy link

I'm ultimately not sure where the use of the ARCH variable came from, because it didn't exist on my systems. Looking around an Intel Mac mini still on 10.6 and the Intel iMac on 10.7 I'm currently using, I found that the HOSTTYPE variable is set appropriate in the environment to either i386 or x86_64. I initially tried to use the arch command's output, but that appears to return i386 no matter what, otherwise ppc if that is the processor architecture in use. Pending some testing, this change appears to work well for any architecture, on Snow Lepord, or Lion, and has the chance to work very well outside of those two OSes.

…6 and 10.7

I'm ultimately not sure where the use of the ARCH variable came from, because it didn't exist on my systems. Looking around an Intel Mac mini still on 10.6 and the Intel iMac on 10.7 I'm currently using, I found that the HOSTTYPE variable is set appropriate in the environment to either i386 or x86_64. I initially tried to use the arch command's output, but that appears to return i386 no matter what, otherwise ppc if that is the processor architecture in use. Pending some testing, this change appears to work well for any architecture, on Snow Lepord, or Lion, and has the chance to work very well outside of those two OSes.
@VxJasonxV VxJasonxV mentioned this pull request Oct 31, 2011
@timcharper
Copy link
Owner

Sorry VxJasonxV... ARCH is defined by the build_test_and_package script.. it loops through 32 bit and 64 bit, producing both packages.

I currently run this on Lion. It does work on Lion.

I have recently updated the script to build universal binaries (but forgot to push the changes... sorry!). Try it now?

@timcharper timcharper closed this Oct 31, 2011
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants