Autore: anonym Data: To: The Tails public development discussion list Oggetto: Re: [Tails-dev] [review'n'merge:1.1]
bugfix/7443-persistent-files-permission
01/07/14 19:14, intrigeri wrote: > Hi,
>
> anonym wrote (01 Jul 2014 16:38:12 GMT) :
>> 'apt' is not a source dir, but 'apt/cache' and 'apt/lists' are. Also, no
>> exception was added for (the incorrect) 'apt' in the "persistent
>> directories have safe access rights" step (like was done for
>> 'nm-system-connections' and 'cups').
>
> Indeed, I was surprised that the test passed.
>
>> I think I cleaned this up quite a bit in commit a59e28a, and
>> future-proofed it for new persistent non-amnesia-user directories.
>
> Thanks! Quickly reviewed, looks good.
When I've run this branch with my fixes in
bugfix/7345-upgrade-from-iso-from-1.0-to-1.1 all tests are green. Merging!
