[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #25543 [Applications/Tor Browser]: Rebase Tor Browser patches for ESR60
#25543: Rebase Tor Browser patches for ESR60
---------------------------------------------+-----------------------------
Reporter: gk | Owner:
| arthuredelstein
Type: task | Status: closed
Priority: Very High | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution: fixed
Keywords: TorBrowserTeam201805R, ff60-esr | Actual Points:
Parent ID: #25741 | Points:
Reviewer: | Sponsor:
---------------------------------------------+-----------------------------
Changes (by gk):
* status: needs_review => closed
* resolution: => fixed
Comment:
Replying to [comment:45 arthuredelstein]:
> > > > Why is `browser_permissions.js` suddenly deleted?
> > > > You are adding `^M` characters to `test_permmanager_defaults.js`
when doing changes.
> > >
> > > Fixed (I have rebased the existing Permissions patch for now.)
> >
> > I still see those characters. I think you can get rid of them by
converting that file using `dos2unix` or some similar tool.
>
> I'm not seeing the characters -- maybe I'm missing something? The new
commit is fe68460a72cddd936a5a313a2f986bafd9e7e7ef
I see those by doing `git show -p -1
extensions/cookie/test/unit/test_permmanager_defaults.js`. The problem
seems actually to be that the patch for bug 1421992 is causing this. Thus,
if Mozilla thinks this is okay, then be it so.
This looks fine to me now. I added just one fixup commit to remove a
superfluous newline that got added during the last revision and pushed
`tor-browser-60.0.1esr-8.0-1` to our `tor-browser` repo. \o/
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/25543#comment:46>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs