Closed Bug 621226 Opened 9 years ago Closed 9 years ago

extern "C" necessary for older libproxy iirc

Categories

(Core :: Networking, defect)

Other
Other
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: reportbase, Unassigned)

References

()

Details

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.12) Gecko/20101027 Ubuntu/10.04 (lucid) Firefox/3.6.12
Build Identifier: 

Upstream patch.  More details to follow.

Reproducible: Always
Attached patch Upstream patch (obsolete) — Splinter Review
Attachment #499583 - Attachment is obsolete: true
Attachment #499583 - Attachment is patch: true
Attachment #499583 - Attachment mime type: application/octet-stream → text/plain
...the extern "C" turned out to be necessary for older libproxy iirc.
Component: General → Networking
Product: Fennec → Core
QA Contact: general → networking
Depends on: 619056
Summary: proxy qt changes → extern "C" necessary for older libproxy iirc
Attachment #499600 - Flags: review?(doug.turner)
Should I ask you or ted for reviewing this patch?
Blocks: 517655
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 499600 [details]
Upstream patch from harmattan

Safe enough, and npodb
Attachment #499600 - Flags: approval2.0?
Comment on attachment 499600 [details]
Upstream patch from harmattan

r+
a=npodb
Attachment #499600 - Flags: review?(doug.turner)
Attachment #499600 - Flags: review+
Attachment #499600 - Flags: approval2.0?
Attachment #499600 - Flags: approval2.0+
Blocks: 619056
No longer depends on: 619056
http://hg.mozilla.org/mozilla-central/rev/a2fc46898518
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.