Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#21264 closed defect (fixed)

torlauncher has no readme file

Reported by: arma Owned by: brade
Priority: Medium Milestone:
Component: Applications/Tor Launcher Version:
Severity: Normal Keywords: TorBrowserTeam201701
Cc: mcs, gk, arthuredelstein Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

https://gitweb.torproject.org/tor-launcher.git/tree/
doesn't give me any place to start in terms of building or using the extension.

It would probably be helpful to teach people at least these few things:

  • Tor launcher is intended only for use in Tor Browser, not for putting in your own Firefox. (But also it's supported for use in Tor Messenger, and maybe others?)
  • Here's what you need to do to make your own xpi, e.g. if you've changed the translations and you want to rebuild it for testing.
  • Some pointer to the Tor Browser build docs, so people can learn where Tor Launcher fits into the bigger picture.

And maybe more?

Child Tickets

Change History (5)

comment:1 Changed 2 years ago by mcs

Cc: mcs added

comment:2 Changed 2 years ago by mcs

Cc: gk arthuredelstein added
Keywords: TorBrowserTeam201701R added
Status: newneeds_review

comment:3 Changed 2 years ago by gk

Let's go with that one, thanks. I think there is one typo in "the browser or other application". I guess it should have been "applications"?

comment:4 in reply to:  3 ; Changed 2 years ago by mcs

Keywords: TorBrowserTeam201701 added; TorBrowserTeam201701R removed
Resolution: fixed
Status: needs_reviewclosed

Replying to gk:

Let's go with that one, thanks. I think there is one typo in "the browser or other application". I guess it should have been "applications"?

Thanks. Kathy and I also replaced "first screen that you see" with "first window that you see" and we fixed some whitespace issues. This is fd0ae33ec9af248ebbcea8774a0349a2aa1413ca on master.

Should this be added on the maintenance branches as well?

comment:5 in reply to:  4 Changed 2 years ago by gk

Replying to mcs:

Replying to gk:

Let's go with that one, thanks. I think there is one typo in "the browser or other application". I guess it should have been "applications"?

Thanks. Kathy and I also replaced "first screen that you see" with "first window that you see" and we fixed some whitespace issues. This is fd0ae33ec9af248ebbcea8774a0349a2aa1413ca on master.

Should this be added on the maintenance branches as well?

Why not. Cherry-picked to maint-0.2.10 with commit e16b92f0db218f9441cdf5ae007a5807da520615.

Note: See TracTickets for help on using tickets.