Closed Bug 673072 Opened 9 years ago Closed 9 years ago
Firefox doesn't build with libcurl >= 7
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20100101 Firefox/5.0 Build ID: 20110619060346 Steps to reproduce: Tried to build Firefox: make -f client.mk Actual results: The build failed with the following error: In file included from /home/evan/projects/mozilla-central/toolkit/crashreporter/google-breakpad/src/common/linux/http_upload.cc:36:0: ../../../../../../dist/system_wrappers/curl/types.h:3:29: fatal error: curl/types.h: No such file or directory compilation terminated. This header no longer exists in libcurl 7.21.7. It has been deprecated and empty since curl 7.12.0 (released June 2, 2004). See these commits: curl/types.h is deprecated and stubbed out: https://github.com/bagder/curl/commit/ffc5fa3a2be787c8198eb68836a45c440876c1bd curl/types.h is removed: https://github.com/bagder/curl/commit/2ef7a28a71f819343afefa721e5497119fb2cee0 Expected results: The build succeeds.
The attached patch removes references to curl/types.h. If we still need to be able to build against libcurl versions older than 7.12.0, then I can try a different approach.
Attachment #547353 - Flags: review?
Attachment #547353 - Flags: review? → review?(ted.mielczarek)
Comment on attachment 547353 [details] [diff] [review] First proposed patch - removes references to curl/types.h This should be fine. Breakpad only dates back to 2006 anyway. I'll land the Breakpad changes upstream in the Breakpad SVN repository for you.
Attachment #547353 - Flags: review?(ted.mielczarek) → review+
Component: Build Config → Breakpad Integration
Product: Firefox → Toolkit
QA Contact: build.config → breakpad.integration
I pushed this change to the build-system branch: http://hg.mozilla.org/projects/build-system/rev/aa5bfc851c35 We periodically merge with the mozilla-central branch, someone will update this bug to mark it FIXED when that happens. Thanks for the patch!
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Pushed to upstream: http://code.google.com/p/google-breakpad/source/detail?r=814 We're far enough out of sync with upstream that only one removal applied, upstream has imported its own local copy of curl headers (I think because curl headers are arch-specific for some reason).
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.