Re: [Tails-dev] [review'n'merge:1.0] feature/tor-0.2.4.21-1+…

Delete this message

Reply to this message
Autore: intrigeri
Data:  
To: The Tails public development discussion list
Oggetto: Re: [Tails-dev] [review'n'merge:1.0] feature/tor-0.2.4.21-1+tails1_{d60.squeeze, d70.wheezy}+1
Hi,

anonym wrote (24 Apr 2014 17:53:16 GMT) :
> For the code review it may be useful to have a look at the (rather
> trivial) backporting work I did, which can be found in our 'tor' Git
> repo. The backporting to 0.2.4.21 can be found in isolation in the
> 'bug11464_backport' branch, but the packages was built from the
> 'debian-tor-0.2.4.21-1+tails1_d60.squeeze+1' tag.


This looks good to me. sajolida is going to test and merge this.

> Separate packages was built for wheezy, so please also review and merge
> into 'devel':


> Tails Git branch: feature/tor-0.2.4.21-1+tails1_d70.wheezy+1
> APT suite:        feature-tor-0.2.4.21-1-tails1-d70.wheezy-1


> Again, only an APT suite merge is needed. The packages was built from
> the 'debian-tor-0.2.4.21-1+tails1_d70.wheezy+1' tag in our 'tor' Git repo.


There's something wrong there: the
debian-tor-0.2.4.21-1+tails1_d70.wheezy+1 tag points to the same
commit as the debian-tor-0.2.4.21-1+tails1_d60.squeeze+1 one.

Please fix that part and resubmit ASAP (it feels wrong to have a newer
Tor on our stable branch than on the devel one). Thanks!

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