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

Improve the makefile #256

Open
2 of 7 tasks
pyllyukko opened this issue Apr 4, 2017 · 11 comments
Open
2 of 7 tasks

Improve the makefile #256

pyllyukko opened this issue Apr 4, 2017 · 11 comments

Comments

@pyllyukko
Copy link
Owner

pyllyukko commented Apr 4, 2017

@nodiscc
Copy link
Contributor

nodiscc commented Apr 26, 2017

  • add a target to compare added/removed preferences between 2 firefox versions
  • add a target to compare preferences and their values against an arbitrary user.js file/URL
  • add a target to compare this user.js against Tor Browser Bundle This project and Tor Browser without Tor #316

@pyllyukko
Copy link
Owner Author

add a target to compare preferences and their values against an arbitrary user.js file/URL

I think compare-user.js does a decent job regarding this.

@nodiscc
Copy link
Contributor

nodiscc commented May 6, 2017

My main grief with compare-user.js is that it requires a full node.js stack for something relatively simple.

@nodiscc
Copy link
Contributor

nodiscc commented May 22, 2017

  • Improve gen-readme.sh to detect paragraphs where all lines are commented out (hence a disabled pref), and not output README entries when the pref is disabled. This would prevent having ot use workarounds like https://github.com/pyllyukko/user.js/pull/296/files and will make diffs easier to read when master/relaxed branch start to diverge more, as well as comparing against other forks.

@pyllyukko
Copy link
Owner Author

We could try to utilize Awk's records.

Something like:

$ gawk 'BEGIN { RS = "\n// PREF: " }NR>1{ print "Record #", NR-1; print}' user.js
Record # 1
Disable Service Workers
// https://developer.mozilla.org/en-US/docs/Web/API/Worker
// https://developer.mozilla.org/en-US/docs/Web/API/ServiceWorker_API
// https://wiki.mozilla.org/Firefox/Push_Notifications#Service_Workers
// NOTICE: Disabling ServiceWorkers breaks functionality on some sites (Google Street View...)
// Unknown security implications
// CVE-2016-5259, CVE-2016-2812, CVE-2016-1949, CVE-2016-5287 (fixed)
user_pref("dom.serviceWorkers.enabled",				false);

Record # 2
Disable Web Workers
// https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Using_web_workers
// https://www.w3schools.com/html/html5_webworkers.asp
user_pref("dom.workers.enabled",					false);

...

@pyllyukko
Copy link
Owner Author

I think we should also check, that the values are of correct type to avoid bugs like #30.

@pyllyukko
Copy link
Owner Author

pyllyukko commented Oct 7, 2017

add a target to compare this user.js against Tor Browser Bundle #316

d0b7859

@nodiscc
Copy link
Contributor

nodiscc commented Dec 18, 2017

  • The new tbb-diff target should probably discard preferences that have been ignored on purpose in ignore.list. What do you think?

Very nice feature otherwise, and it could probably be extended to cover other user.js variants. I'd like to

@pyllyukko
Copy link
Owner Author

The new tbb-diff target should probably discard preferences that have been ignored on purpose in ignore.list. What do you think?

True that. Need to think about the best way to do it.

Very nice feature otherwise, and it could probably be extended to cover other user.js variants. I'd like to

We could start by implementing functionality, that takes the other user.js as a parameter.

@nodiscc
Copy link
Contributor

nodiscc commented Feb 10, 2024

I think the makefile-based solution will start being hard to maintain if we want to add more sources for comparison. I'd like to work on something python-based in the few coming months. I'll keep this issue updated.

@pyllyukko
Copy link
Owner Author

I'd like to work on something python-based in the few coming months. I'll keep this issue updated.

Sound good. Thanks in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants