Bug 1061949 - Build with newer mozjs
Summary: Build with newer mozjs
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: polkit
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Miloslav Trmač
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 1103454 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-02-05 22:39 UTC by Kalev Lember
Modified: 2018-04-09 13:26 UTC (History)
7 users (show)

Fixed In Version: polkit-0.114-1.fc28
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-04-09 13:26:03 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Kalev Lember 2014-02-05 22:39:35 UTC
In F21, gnome-shell is switching over from mozjs17 to mozjs24. Would be great if polkit could switch over as well, in order to avoid shipping with two JS runtimes.

Comment 1 Miloslav Trmač 2014-06-30 17:29:10 UTC
*** Bug 1103454 has been marked as a duplicate of this bug. ***

Comment 2 Orion Poplawski 2014-10-22 15:49:06 UTC
This looks like an upstream issue to fix, as mozjs24 is C++ only.

Comment 3 Jaroslav Reznik 2015-03-03 15:27:16 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle.
Changing version to '22'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22

Comment 4 Peter Robinson 2015-03-20 18:49:56 UTC
Might be worth looking at mozjs31 now

Comment 5 Peter Robinson 2015-11-04 10:39:46 UTC
There's now also mozjs38

Comment 6 Peter Robinson 2016-06-14 15:19:36 UTC
Moving to rawhide, note there is now mozjs45

Comment 7 Jan Kurik 2016-07-26 04:45:48 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 25 development cycle.
Changing version to '25'.

Comment 8 Fedora Update System 2018-04-05 00:11:34 UTC
polkit-0.114-1.fc28 has been submitted as an update to Fedora 28. https://bodhi.fedoraproject.org/updates/FEDORA-2018-fbc8b4e877

Comment 9 Fedora Update System 2018-04-06 02:32:02 UTC
polkit-0.114-1.fc28 has been pushed to the Fedora 28 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2018-fbc8b4e877

Comment 10 Ali Akcaagac 2018-04-06 11:50:02 UTC
Hi,

I was just made aware of a recent polkit(-libs) update and like to give some feedback on it. This update works normally - and this is not the concern.

But I am curious why it now depends on mozjs52 which adds another 23mb of dependency to the system?

We use Fedora on a low consumer platform with things stripped away as much as possible (less than 750packages).

Every free megabyte counts on these small systems. Now you add 23mb of dependency to two packages (polkit and polkit-lib) that have just a sice of less than 250kb.

If it's not necessary, then please get rid of the mozjs52 dependency. Mozjs17 was just 3.8mb now replaced by a 23mb dependency.

A 6x times overhead...

Comment 11 Peter Robinson 2018-04-06 12:20:45 UTC
(In reply to Ali Akcaagac from comment #10)
> Hi,
> 
> I was just made aware of a recent polkit(-libs) update and like to give some
> feedback on it. This update works normally - and this is not the concern.
> 
> But I am curious why it now depends on mozjs52 which adds another 23mb of
> dependency to the system?
> 
> We use Fedora on a low consumer platform with things stripped away as much
> as possible (less than 750packages).
> 
> Every free megabyte counts on these small systems. Now you add 23mb of
> dependency to two packages (polkit and polkit-lib) that have just a sice of
> less than 250kb.
> 
> If it's not necessary, then please get rid of the mozjs52 dependency.
> Mozjs17 was just 3.8mb now replaced by a 23mb dependency.

Mozjs17 is unmaintained and being a JS engine means it has potential CVEs (with little doubt) so there is a need to move it forward because there's other implications of leaving it as it is.

Comment 12 Ali Akcaagac 2018-04-06 12:35:50 UTC
(In reply to Peter Robinson from comment #11)
> Mozjs17 is unmaintained and being a JS engine means it has potential CVEs
> (with little doubt) so there is a need to move it forward because there's
> other implications of leaving it as it is.

I fully understand this. I also noticed the huge cleanup in the spec file and all the patches that got dropped. I only wished polkit wouldn't depend on that big thing called mozjs52. Anyways! Thank you for the feedback.

Comment 13 Fedora Update System 2018-04-09 13:26:03 UTC
polkit-0.114-1.fc28 has been pushed to the Fedora 28 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.