12/01/14 18:02, Alan wrote:
> Hi,
> 
> Please review and merge feature/create-additional-software-config into
> devel
> 
> This fixes https://labs.riseup.net/code/issues/6440: 
> tails-additional-software crashes if persistence is not mounted
> 
> commit 0a27b8908f50f074e3ed45de80e7cd3694f34a41
> Author: Tails developers <[email protected]>
> Date:   Tue Nov 26 15:45:01 2013 +0000
> 
>     additional software: do not crash when tails is used without
>     persistence 
> 
>     Before this, tails-additional-software crashed if there was no
>     persistent volume activated. It now exits gracefully.

Review:

-    else:
+    elif os.path.isdir(PERSISTENCE_DIR):

Here `PERSISTENCE_DIR` is set to
`"/live/persistence/TailsData_unlocked"`, which works. However, in other
places we check whether persistence is enabled by looking if the shell
variable `TAILS_PERSISTENCE_ENABLED` is set to `true` in the file
`/var/lib/live/config/tails.persistence`. See `persistence_is_enabled()`
in
`config/chroot_local-includes/usr/local/lib/tails-shell-library/tails_greeter.sh`.

Still, as the non-fatal exception isn't thrown any more (it's absent
from `/var/log/gdm3/0:-greeter.log`) I won't block on this but I created
the low-priority ticket #6786 to track it.

So, merged!

Cheers!

_______________________________________________
tails-dev mailing list
[email protected]
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
[email protected].

Reply via email to