Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Firefox doesn't build with libcurl >= 7.21.7

RESOLVED FIXED

Status

()

Toolkit
Crash Reporting
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Evan Shaw, Assigned: Evan Shaw)

Tracking

Trunk
All
Other
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-bs)

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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.
(Assignee)

Comment 1

6 years ago
Created attachment 547353 [details] [diff] [review]
First proposed patch - removes references to curl/types.h

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)
Assignee: nobody → edsrzf
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
Whiteboard: fixed-in-bs
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).
http://hg.mozilla.org/mozilla-central/rev/aa5bfc851c35
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Duplicate of this bug: 680370
You need to log in before you can comment on or make changes to this bug.