Opened 13 years ago

Closed 5 years ago

#2840 defect closed fixed (fixed)

fails to build on Mac OS X with QuickTime 6 headers

Reported by: zooko Owned by:
Priority: low Milestone:
Component: release management Keywords:
Cc: Thijs Triemstra Branch:
Author:

Description

Darius Bacon has a Mac OS X machine that doesn't have QuickTime 7 headers installed -- instead it has QuickTime 6 headers. As far as I know, failure to build cfsupport shouldn't be fatal -- the install should continue without cfsupport.

ygbm:~/net/Twisted-2.5.0 darius$ python setup.py install
running install
running build
running build_py
running build_ext
Checking if C extensions can be compiled, don't be alarmed if a few compile errors are printed.
gcc -fno-strict-aliasing -Wno-long-double -no-cpp-precomp -mno-fused-madd -fno-common -fno-common -dynamic -DNDEBUG -g -O3 -Wall -Wstrict-prototypes -I/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4 -I/opt/local/include/python2.4 -c conftest.c -o conftest.o
gcc -fno-strict-aliasing -Wno-long-double -no-cpp-precomp -mno-fused-madd -fno-common -fno-common -dynamic -DNDEBUG -g -O3 -Wall -Wstrict-prototypes -I/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4 -I/opt/local/include/python2.4 -c conftest.c -o conftest.o
conftest.c:1:23: sys/epoll.h: No such file or directory
building 'twisted.internet.cfsupport' extension
gcc -fno-strict-aliasing -Wno-long-double -no-cpp-precomp -mno-fused-madd -fno-common -fno-common -dynamic -DNDEBUG -g -O3 -Wall -Wstrict-prototypes -I/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4 -I/opt/local/include/python2.4 -c twisted/internet/cfsupport/cfsupport.c -o build/temp.darwin-8.9.0-Power_Macintosh-2.4/twisted/internet/cfsupport/cfsupport.o -w
In file included from /opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:10,
                 from twisted/internet/cfsupport/cfsupport.c:8:
/System/Library/Frameworks/Carbon.framework/Headers/Carbon.h:33:37: CarbonSound/CarbonSound.h: No such file or directory
In file included from /opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:11,
                 from twisted/internet/cfsupport/cfsupport.c:8:
/System/Library/Frameworks/QuickTime.framework/Headers/QuickTime.h:16:2: #error "Your QuickTime Headers are out of date.  You may already have new headers available here: /Developer/SDKs/QuickTime7Headers.pkg .  Otherwise you can get the latest headers via Software Update or in the latest QuickTime SDK available from http://developer.apple.com/quicktime ."
In file included from twisted/internet/cfsupport/cfsupport.c:8:
/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:138: error: parse error before "Track"
/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:140: error: parse error before "Movie"
/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:142: error: parse error before "MovieController"
/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:146: error: parse error before "UserData"
/opt/local/Library/Frameworks/Python.framework/Versions/2.4/include/python2.4/pymactoolbox.h:148: error: parse error before "Media"
error: command 'gcc' failed with exit status 1
Error: Subprocess exited with result 1 for project core

Change History (8)

comment:1 Changed 13 years ago by Jean-Paul Calderone

Component: corerelease management
Owner: changed from Glyph to radix
Priority: normallow

Build failures on every other platform are fatal, as far as I know. Not sure why one would expect OS X to be special.

comment:2 Changed 13 years ago by zooko

I guess I was thinking of that because of "Checking if C extensions can be compiled, don't be alarmed if a few compile errors are printed.", and because the thing that I wanted Darius to try (allmydata.org tahoe) doesn't need cfsupport.

So, do I understand correctly that it isn't normal for the build process to consider some extensions as optional?

comment:3 Changed 13 years ago by Jean-Paul Calderone

The output is misleading. The message you mention only refers to compilation of conftest.c files. The error which prevented your build from succeeding came from cfsupport.c. It might be useful for the conftest section to be delimited, rather than only having a header. Hopefully this also clarifies what "optional" means - anything which is detected as not working by a conftest will be skipped. Apparently this QuickTime 6 problem is not noticed during that phase, so when the actual compilation starts it fails and terminates the entire build. Perhaps there should be more conftest checks for the cfsupport extension.

Although, as a related matter, I don't know what cfsupport does or is for. As far as I know it is unmaintained. It has no coverage on any buildslave, in any case. It probably shouldn't be allowed to cause a failure this severe.

comment:4 Changed 13 years ago by zooko

So if I were to submit a patch that makes Darius's build proceed, that patch ought to work by adding a check to conftest and skipping cfsupport build, rather than by ignoring the failure of cfsupport build?

comment:5 Changed 13 years ago by Jean-Paul Calderone

That would be consistent with how other extentions are treated.

comment:6 Changed 11 years ago by Thijs Triemstra

Cc: Thijs Triemstra added

comment:7 Changed 10 years ago by <automation>

Owner: radix deleted

comment:8 Changed 5 years ago by hawkowl

Resolution: fixed
Status: newclosed

twisted/internet/cfsupport doesn't exist anymore.

Note: See TracTickets for help on using tickets.