Re: [Tails-dev] Made New Identity work again, please review

Delete this message

Reply to this message
Author: intrigeri
Date:  
To: The Tails public development discussion list
Subject: Re: [Tails-dev] Made New Identity work again, please review
winterfairy@??? wrote (20 Feb 2014 11:52:17 GMT) :
>>>> 3. It might be overkill, and surely adds some code, but I would pass
>>>>    the port to listen on, control port socket path and authentication
>>>>    cookie path as command-line arguments (preferably named, not
>>>>    positional). Just to make it easier for others to reuse this piece
>>>>    of code, and increase collaboration between similar projects.

>>>
>>> A quick research showed up that this can probably be done
>>> with only a few lines of code if using the modules
>>> optparse or argparse.
>>>
>>> Python 2.6 (which Tails uses) only has optparse.
>>> In python 2.7 optparse is deprecated and replaced by argparse.
>>> This would mean writing one version for squeeze and one for wheezy.
>>>
>>> I don't think it is worth it currently.
>>
>> Fair enough. Care to file a ticket, blocked by #6015, so that this is
>> not forgotten at post-1.1 time?


> I guess I should wait until my branch is actually accepted/merged into devel?


If you also mark the new ticket as blocked by the new identity one,
I see little use in waiting :)

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