Bug 1376560 - libproxy is using js185 which is very dated, please pull it forward to mozjs24
Summary: libproxy is using js185 which is very dated, please pull it forward to mozjs24
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: libproxy
Version: rawhide
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: David King
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-15 17:37 UTC by Jeremy Linton
Modified: 2017-01-09 16:23 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-01-09 16:23:32 UTC
Type: Bug

Attachments (Terms of Use)
Pulls libproxy to mozjs24 and fixes an intermittent unit test failure (9.36 KB, patch)
2016-09-19 22:51 UTC, Jeremy Linton 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)


Links
System ID Private Priority Status Summary Last Updated
Github libproxy libproxy issues 32 0 None None None 2016-09-15 21:15:34 UTC

Description Jeremy Linton 2016-09-15 17:37:38 UTC 
Description of problem:
libproxy in fedora is using mozjs 1.8.5. Js185 is getting rather old. Lets pull libproxy forward to a more recent version of mozjs in order to better support it. 


Thanks,
Comment 1 Jeremy Linton 2016-09-19 22:49:03 UTC 
Koji build:

http://koji.fedoraproject.org/koji/taskinfo?taskID=15710944
Comment 2 Jeremy Linton 2016-09-19 22:51:46 UTC 
Created attachment 1202640 [details]
Pulls libproxy to mozjs24 and fixes an intermittent unit test failure
Comment 3 David King 2017-01-09 16:23:32 UTC 
The get-pac-test failure has been fixed upstream for 0.4.14 (now in Rawhide and working its way to F25 as well). The JS implementation is not that important from my point of view, as several other packages depend on mozjs 1.8.5. It is likely that libproxy in Fedora will follow whatever JS implementation upstream libproxy uses, unless there is a compelling reason to do otherwise.
Note You need to log in before you can comment on or make changes to this bug.