'chuck' failed to build in 10.10

[edmz]

Relevant gist at https://gist.github.com/anonymous/fd10b7a51558a050979e

Seems, to my inexperienced eye, like its trying to compile it with some kind of multitouch support:

Undefined symbols for architecture x86_64:
"_MTDeviceCreateList", referenced from:
MultiTouchDevice_init() in util_hid.o
"_MTDeviceStart", referenced from:
MultiTouchDevice_open(int) in util_hid.o
"_MTDeviceStop", referenced from:
MultiTouchDevice_close(int) in util_hid.o
"_MTRegisterContactFrameCallback", referenced from:
MultiTouchDevice_open(int) in util_hid.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

[mistydemeo]

Those functions come from the private framework MultitouchSupport, but it doesn't appear to have tried to link it.

Looks like a bad version regex is causing it to think it's on OS X 10.4:

ifneq ($(shell sw_vers -productVersion | egrep '10\.[6789](\.[0-9]+)?'),)
...
LINK_EXTRAS=-F/System/Library/PrivateFrameworks \
    -weak_framework MultitouchSupport
else
ISYSROOT=-isysroot /Developer/SDKs/MacOSX10.4u.sdk
LINK_EXTRAS=
endif

Is evaluating to false for 10.10. Explains why it's trying to use the 10.4u SDK.

[edmz]

Great catch! I missed that!

is there a way for me to modify the recipe locally and give it a try?

On Fri, Nov 14, 2014 at 10:32 AM, Misty De Meo notifications@github.com
wrote:

Those functions come from the private framework MultitouchSupport, but it
doesn't appear to have tried to link it.

Looks like a bad version regex is causing it to think it's on OS X 10.4:

ifneq ($(shell sw_vers -productVersion | egrep '10.6789?'),)
...LINK_EXTRAS=-F/System/Library/PrivateFrameworks
-weak_framework MultitouchSupportelseISYSROOT=-isysroot /Developer/SDKs/MacOSX10.4u.sdkLINK_EXTRAS=endif

Is evaluating to false for 10.10. Explains why it's trying to use the
10.4u SDK.

—
Reply to this email directly or view it on GitHub
#34189 (comment).

Eduardo

[edmz]

Here is an updated regex that should work for 10.10:

10.([6789]|1[0-9]+)(.[0-9]+)?

Relevant Rubular test case:

http://rubular.com/r/qYqp8gSlQ5

On Fri, Nov 14, 2014 at 10:34 AM, Eduardo Dominguez eduardodmz@gmail.com
wrote:

Great catch! I missed that!

is there a way for me to modify the recipe locally and give it a try?

On Fri, Nov 14, 2014 at 10:32 AM, Misty De Meo notifications@github.com
wrote:

Those functions come from the private framework MultitouchSupport, but it
doesn't appear to have tried to link it.

Looks like a bad version regex is causing it to think it's on OS X 10.4:

ifneq ($(shell sw_vers -productVersion | egrep '10.6789?'),)
...LINK_EXTRAS=-F/System/Library/PrivateFrameworks
-weak_framework MultitouchSupportelseISYSROOT=-isysroot /Developer/SDKs/MacOSX10.4u.sdkLINK_EXTRAS=endif

Is evaluating to false for 10.10. Explains why it's trying to use the
10.4u SDK.

—
Reply to this email directly or view it on GitHub
#34189 (comment)
.

Eduardo

Eduardo

[mistydemeo]

Ignore that comment, this is compatible!

[edmz]

I forgot to mention that I tried it in the command line :)

On Fri, Nov 14, 2014 at 10:46 AM, Misty De Meo notifications@github.com
wrote:

Ignore that comment, this is compatible!

—
Reply to this email directly or view it on GitHub
#34189 (comment).

Eduardo

[edmz]

Here is a modified and confirmed working version of the patch found at the
bottom of the recipe:

@@ -1,9 +1,7 @@

uncomment the following to force 32-bit compilation

FORCE_M32=-m32

-ifneq ($(shell sw_vers -productVersion | egrep '10.6789?'),)
-SDK=$(shell xcodebuild -sdk macosx -version | grep '^Path:' | sed 's/Path:
(.*)/\1/')
-ISYSROOT=-isysroot $(SDK)
+ifneq ($(shell sw_vers -productVersion | egrep
'10.([6789]|1[0-9]+)(.[0-9]+)?'),)
LINK_EXTRAS=-F/System/Library/PrivateFrameworks
-weak_framework MultitouchSupport
else

Should I create a PR?

On Fri, Nov 14, 2014 at 10:46 AM, Eduardo Dominguez eduardodmz@gmail.com
wrote:

I forgot to mention that I tried it in the command line :)

On Fri, Nov 14, 2014 at 10:46 AM, Misty De Meo notifications@github.com
wrote:

Ignore that comment, this is compatible!

—
Reply to this email directly or view it on GitHub
#34189 (comment)
.

Eduardo

Eduardo