search

Divested-Mobile / Mull-Fenix

Build scripts for a web browser built upon Mozilla technology
https://divestos.org/pages/our_apps#mull
GNU Affero General Public License v3.0
610 stars 16 forks source link

Remove deprecated / removed stuff from mull #15

Closed Gontier-Julien closed 3 years ago

Gontier-Julien commented 3 years ago

Those are all deprecated / removed entry from the current version of Firefox and still remain in Mull, there is no need or reason to still have them in there, so it a good idea to remove them.

SkewedZeppelin commented 3 years ago

Thank you for the effort, but userjs-arkenfox.js is maintained first upstream and then modified (for desktop browser) in my sister repository here in order to minimize the maintenance required. The Brace repo has ~30 changes onto arkenfox.

Mull then pulls from the Brace repo and adds a handful more changes.

Removing bits like this will fragment them and ultimately add more work necessary as they diverge.

Thorin-Oakenpants commented 3 years ago

Arkenfox keeps deprecated items for each ESR until the release after EOL of the that ESR, because some users like to use the latest user.js for ESR. The deprecated section is commented out by encapsulating them in /* and // ****/ so users can flip one char to enable them. Even if they were applied, deprecated prefs can't do any harm. The section is also handy for running prefsCleaner to clean them up.

SkewedZeppelin commented 3 years ago

In Brace I keep the deprecated section uncommented for use in distros Debian/CentOS that ship ESR.

All changes are marked with //MULL or //BRACE