Re: [Tails-dev] [review'n'merge:1.3] bugfix/8257-more-robust…

Delete this message

Reply to this message
Author: anonym
Date:  
To: The Tails public development discussion list
Old-Topics: [Tails-dev] [review'n'merge:1.3] bugfix/8257-more-robust-tor_bootstrap_progress
Subject: Re: [Tails-dev] [review'n'merge:1.3] bugfix/8257-more-robust-tor_bootstrap_progress
On 27/11/14 22:35, intrigeri wrote:
> hi,
>
> I've noticed that bug thanks to the Journal capturing such error
> output on Tails/Jessie, but it affects Wheezy too. We're using the
> affected function in tor-has-bootstrapped, that some users can run
> with passwordless sudo, so better make it more robust to be on the
> slightly safer side.
>
> Please review'n'merge into devel for 1.3, unless you feel comfortable
> to take it into 1.2.1 (if I were the RM, I wouldn't, since AFAICT it's
> mostly cosmetic, and without any practical implication unless we use
> the affected function in a "set -e" script -- not checked if we did).
>
> It passes the time sync', torified browsing and Windows camouflage
> features of our automated test suite. Not tried to run the rest.


These tests passes for me too with this branch. Merged!

Cheers!