Re: [Tails-dev] Please review'n'merge feature/dont_autostart…

Delete this message

Reply to this message
Autore: intrigeri
Data:  
To: The Tails public development discussion list
Oggetto: Re: [Tails-dev] Please review'n'merge feature/dont_autostart_iceweasel + test/dont_autostart_iceweasel
sajolida@??? wrote (09 Jan 2014 10:14:01 GMT) :
> I tested that between yesterday and this morning. I'm glad I lunched out
> yesterday because this morning I discovered a new bug. The "Tor is
> ready" message was appearing too early, when Iceweasel would still tell
> you that "Tor is not ready". So I reused the same check code from
> Iceweasel for the Tor is ready notification. I tested that by patching a
> running ISO but I didn't built a new one.


Great, thanks! I'm glad this was discovered early enough. I find it
strange that I didn't experience this bug myself, but oh well :)

> I patched it so it needs another review before merging. Sorry...


I'll take care of it. Updated the ticket.

Cheers,
--
intrigeri
| GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
| OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc