Feature #7252

Migrate tor-launcher to iceweasel --app

Added by anonym 2014-05-15 02:51:13 . Updated 2014-10-16 08:10:36 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Tor configuration
Target version:
Start date:
2014-05-15
Due date:
% Done:

100%

Feature Branch:
feature/7252-no-xulrunner
Type of work:
Code
Blueprint:

Starter:
1
Affected tool:
Deliverable for:

Description

Since xulrunner is being deprecated in FF31, we need to start it with iceweasel --app instead of xulrunner, which will have identical results. Well, except that we need to pgrep for iceweasel --app ... (instead of xulrunner ...) in config/chroot_local-includes/etc/NetworkManager/dispatcher.d/60-tor-ready-notification.sh.


Files


Subtasks


History

#1 Updated by anonym 2014-05-15 02:54:49

Just to be clear, we can do this migration now, as `iceweasel —app` works with our current FF24-based Iceweasel (this was how I tested it in Bug #7237).

#2 Updated by intrigeri 2014-05-28 14:35:09

  • Priority changed from Normal to Elevated

#3 Updated by intrigeri 2014-07-10 15:41:29

  • Target version set to Tails_1.2

#4 Updated by joelanders 2014-08-03 18:09:02

Built an ISO and stuff still works.

#5 Updated by intrigeri 2014-08-04 09:42:54

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 10

#6 Updated by intrigeri 2014-08-04 09:47:50

  • Feature Branch set to feature/7252-no-xulrunner

Thanks. The patch looks good, building an ISO, will run the test suite on it.

Have you tried Tor Launcher (by opting-in in Tails Greeter for advanced Tor configuration) in an ISO built with this patch applied?

#7 Updated by intrigeri 2014-08-04 10:45:53

  • Status changed from In Progress to Fix committed
  • Assignee deleted (intrigeri)
  • % Done changed from 10 to 100
  • QA Check changed from Ready for QA to Pass

Passes these features of the automated test suite: features/firewall_leaks.feature, features/time_syncing.feature, features/torified_browsing.feature, features/unsafe_browser.feature, features/windows_camouflage.feature.

Manually tested bridges, works fine. Merged into the devel branch!

joelanders: next time, please follow our standard workflow to submit your changes. Thanks!

#8 Updated by anonym 2014-10-16 08:10:37

  • Status changed from Fix committed to Resolved