[PATCH 0/4] Pull polkit forward to mozjs24
Jeremy Linton
jeremy.linton at arm.com
Thu Aug 18 15:38:53 UTC 2016
On 08/17/2016 03:57 PM, Colin Walters wrote:
>
>
> On Thu, Aug 4, 2016, at 12:57 PM, Jeremy Linton wrote:
>> Polkit is out of date with respect to supported versions of mozjs. These
>> patches start moving polkit forward again. Right now they are just to
>> mozjs24 which is one of the first C++ only JSAPI versions of spidermonkey
>> and is a good starting point for evolving polkit. Moving it forward to 38 or
>> 45 involves further effort.
>
> This came up before; see
> https://lists.freedesktop.org/archives/polkit-devel/2014-April/000401.html
> and
> https://lists.freedesktop.org/archives/polkit-devel/2014-June/000404.html
>
> It might be worth looking at kalev's work to see what differs.
Are the patches visible anywhere? I don't see them in those threads.
>
> I think my vote is actually for doing 24 only. Does anyone object to that?
>
> Aside from the additional #ifdefs, your patches look clean and good to me at first glance.
The #ifdefs definitely restrict the code flow. The more recent versions
of JSAPI lean heavily on a RAII/construction model for rooting objects
in the GC. So, removing them helps clear the way for reworking the code
flow to fit well with mozjs45+.
I will re-post with mozjs1.85 and 17 removed. Assuming everyone is happy
with this. Do all the distro's have mozjs24?
More information about the polkit-devel
mailing list