Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 833122 - sys-auth/polkit-0.120-r2 mozjs91 patch
Summary: sys-auth/polkit-0.120-r2 mozjs91 patch
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Freedesktop bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-02-11 11:27 UTC by Andrea Postiglione
Modified: 2022-02-19 02:04 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---

Attachments
mozjs 91 patch (mozjs-91.patch,2.02 KB, patch)
2022-02-11 11:27 UTC, Andrea Postiglione 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrea Postiglione 2022-02-11 11:27:17 UTC 
Created attachment 764829 [details, diff]
mozjs 91 patch

mozjs91 patch
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-02-11 11:30:51 UTC 
I know where this patch comes from but please include links to them in future (and now ideally).

But I'm not sure we want to apply this proactively. Doesn't most stuff (all?) still need :78, so this will result in some people needing to build a later slot just for Polkit?
Comment 3 Andrea Postiglione 2022-02-11 11:39:31 UTC 
this patch is for not having two versions of spidermonkey
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-02-11 11:41:39 UTC 
(In reply to Andrea Postiglione from comment #3)
> this patch is for not having two versions of spidermonkey

But this means it'll only support 91. Does something else common support 91?
Comment 5 Andrea Postiglione 2022-02-11 11:43:51 UTC 
(In reply to Sam James from comment #4)
> (In reply to Andrea Postiglione from comment #3)
> > this patch is for not having two versions of spidermonkey
> 
> But this means it'll only support 91. Does something else common support 91?

at the moment only polkit uses spidermonkey 78
Comment 6 Mike Gilbert gentoo-dev 2022-02-11 17:36:31 UTC 
I think we will either add a new polkit snapshot, or wait for the next release to pick this up.

Also, it is likely we will just switch to duktape and disable mozjs.
Comment 7 Larry the Git Cow gentoo-dev 2022-02-19 02:04:11 UTC 
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a5f0be7ceb05007e3a309487badb9a098b877952

commit a5f0be7ceb05007e3a309487badb9a098b877952
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-02-19 01:56:05 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-02-19 02:03:44 +0000

    sys-auth/polkit: add 0.120_p20220127
    
    Now with Duktape!
    
    Closes: https://bugs.gentoo.org/833122
    Closes: https://bugs.gentoo.org/734326
    Signed-off-by: Sam James <sam@gentoo.org>

 profiles/arch/alpha/package.use.force         |   6 +-
 profiles/arch/hppa/package.use.force          |   6 ++
 profiles/arch/ia64/package.use.force          |   6 +-
 profiles/arch/powerpc/ppc32/package.use.force |   6 ++
 profiles/arch/sparc/package.use.force         |   6 +-
 sys-auth/polkit/Manifest                      |   1 +
 sys-auth/polkit/metadata.xml                  |   1 +
 sys-auth/polkit/polkit-0.120_p20220127.ebuild | 123 ++++++++++++++++++++++++++
 8 files changed, 152 insertions(+), 3 deletions(-)