Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove end-of-life spidermonkey versions (78, 91, 102) #157874

Open
mweinelt opened this issue Feb 2, 2022 · 27 comments
Open

Remove end-of-life spidermonkey versions (78, 91, 102) #157874

mweinelt opened this issue Feb 2, 2022 · 27 comments
Labels
6.topic: old-versions Tickets pertaining to ongoing support of outdated versions of packages
Milestone

Comments

@mweinelt
Copy link
Member

mweinelt commented Feb 2, 2022

Both package have reached their end of life together with their respective firefox versions and should be removed before NixOS 22.05.

https://endoflife.date/firefox

@lostnet @abbradar @ajs124

@lostnet
Copy link
Contributor

lostnet commented Feb 11, 2022

Here's the other packages that I've been monitoring:

@mweinelt mweinelt added this to the 22.05 milestone Feb 14, 2022
@dasJ
Copy link
Member

dasJ commented May 21, 2022

I'd like to remind the people here that branch-off is scheduled for tomorrow and depending on how good the staging builds progress, we will branch off rather soon.

@lostnet
Copy link
Contributor

lostnet commented May 22, 2022 via email

@mweinelt
Copy link
Member Author

Unrealistic to achieve and postponed to the 22.11 release cycle. Thanks for your research and documentation on the topic!

@lostnet
Copy link
Contributor

lostnet commented Jul 17, 2022

libproxy could now be moved of 78 but cjs remains a problem as well as some other dependents.
So the refactor in #179822 still includes 78.

@ajs124
Copy link
Member

ajs124 commented Jul 18, 2022

libproxy could now be moved of 78

see #181917

@jtojnar
Copy link
Member

jtojnar commented Jul 18, 2022

Polkit done in #181264.

@ajs124
Copy link
Member

ajs124 commented Oct 4, 2022

I guess we can now remove spidermonkey_68 from the list, since that's gone, but need to add spidermonkey_91, used in couchdb and gjs, because support for that version of firefox has ended.

@jtojnar
Copy link
Member

jtojnar commented Oct 4, 2022

#182618 updates gjs

@mweinelt mweinelt changed the title Remove spidermonkey_78 and spidermonkey_68 Remove end-of-life spidermonkey versions (78, 91) Oct 4, 2022
@mweinelt
Copy link
Member Author

mweinelt commented Oct 4, 2022

@chvp
Copy link
Member

chvp commented Oct 4, 2022

It's not possible for 0ad unfortunately. The versions of spidermonkey have to match between clients, and we can't rely on other clients using the same version. AFAIK there are also no patches written yet for compatibility with 102.

@Artturin Artturin modified the milestones: 22.05, 23.05 Dec 31, 2022
@RaitoBezarius RaitoBezarius removed this from the 23.05 milestone May 31, 2023
@RaitoBezarius RaitoBezarius added this to the 23.11 milestone May 31, 2023
@bobby285271
Copy link
Member

Cinnamon 5.8 & CJS 5.8 bump (should use spidermonkey_102)

@mweinelt
Copy link
Member Author

mweinelt commented Jun 1, 2023

The 102 series will go EOL on 2023-08-29. The 115 series will be released in early July. https://whattrainisitnow.com/calendar/

@bobby285271
Copy link
Member

bobby285271 commented Aug 15, 2023

The next GJS (>= 1.77.2, as part of GNOME 45) will depend on spidermonkey 115 (not packaged yet unfortunately)

@RaitoBezarius
Copy link
Member

@chvp Please can you inline spidermonkey_78 inside 0ad? We will drop it officially but you can keep using it as the sole consumer of this package.

@puffnfresh Can you consider what it takes to move away from spidermonkey 102 in jsawk? Thanks!

@aforemny @jfrankenau Can you consider what it takes to move away from spidermonkey 102 in plowshare? Thanks!

I guess, we will have spidermonkey 102 for NixOS 23.11, but we can remove any public use of the previous versions IMHO by inlining 78 (0ad) & 91 (CouchDB 3.x) to their call site.

@mweinelt
Copy link
Member Author

I'm not convinced inlining is a good solution to this problem. It's not like we are getting new consumers of EOL spidermonkey versions regularly.

@RaitoBezarius
Copy link
Member

I'm not convinced inlining is a good solution to this problem. It's not like we are getting new consumers of EOL spidermonkey versions regularly.

It's not a permanent solution to the problem, it's a "let's erase them from the top-level" temporary solution.
For something like a game like 0ad, I'm not really sure we will see them disappear.

@ajs124
Copy link
Member

ajs124 commented Nov 4, 2023

@puffnfresh Can you consider what it takes to move away from spidermonkey 102 in jsawk? Thanks!

@aforemny @jfrankenau Can you consider what it takes to move away from spidermonkey 102 in plowshare? Thanks!

Both of these just call the js binary, which rarely ever changes. So just updating the package to a newer version and seeing if it still works as expected should be enough.

@mweinelt mweinelt changed the title Remove end-of-life spidermonkey versions (78, 91) Remove end-of-life spidermonkey versions (78, 91, 102) Mar 22, 2024
@mweinelt
Copy link
Member Author

Spidermonkey is likely affected by GHSA-gv5g-5832-j3rm, GHSA-cm37-53wc-mx6g, so please evaluate whether your applications could be affected by that.

I'm inclined to mark versions older than 115 as known vulnerable at some point.

@chvp
Copy link
Member

chvp commented Mar 22, 2024

0ad should not be affected, no remote JS is loaded.

@bobby285271
Copy link
Member

The patches are mozilla/gecko-dev@81806e7 and mozilla/gecko-dev@afbdf68 according to the bug ID I think.

It looks like MObjectKeysLength::computeRange is introduced in mozilla/gecko-dev@eec1c03 3 months ago, and mozilla/gecko-dev@afbdf68 does not touch the js folder 🤔

@aforemny aforemny mentioned this issue May 2, 2024
13 tasks
@aforemny

This comment was marked as outdated.

@tomodachi94 tomodachi94 added the 6.topic: old-versions Tickets pertaining to ongoing support of outdated versions of packages label May 12, 2024
@bobby285271
Copy link
Member

Cinnamon 6.2 and CJS 6.2 bump (should use spidermonkey_115)

@fpletz
Copy link
Member

fpletz commented Aug 6, 2024

For something like a game like 0ad, I'm not really sure we will see them disappear.

The next version of 0ad will support a newer version of spidermonkey - unfortunately only 91 but then we can drop 78: 0ad/0ad@9513e7e

@mweinelt
Copy link
Member Author

mweinelt commented Aug 6, 2024

That is so very cool of them, now that 128 ESR is released. 🤦

@fpletz
Copy link
Member

fpletz commented Aug 6, 2024

Next version of CouchDB will have support for QuickJS: apache/couchdb#4627

@bobby285271
Copy link
Member

#333028 packages spidermonkey_128. and to test mozjs128 based GJS, gjs.passthru.tests should build on 7740a53.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: old-versions Tickets pertaining to ongoing support of outdated versions of packages
Projects
Status: To do
Status: In progress
Development

No branches or pull requests