Re: [Tails-dev] Please review and merge feature/remember_ins…

Delete this message

Reply to this message
Author: anonym
Date:  
To: The Tails public development discussion list
Subject: Re: [Tails-dev] Please review and merge feature/remember_installed_packages
21/04/13 03:13, Alan wrote:
> Hi,
>
> I belive the 1st step of additional software installation is ready,
> and hope somebody will have time to review it for 0.18.
>
> Branch: feature/remember_installed_packages
> Ticket: https://tails.boum.org/todo/remember_installed_packages/


I trust intrigeri's review of the code, and sajolida's review of the
docs. I've tested this feature and it works for the following minimal
test case:

1. Boot Tails with read-write persistence (including APT packages + lists).
2. apt-get update; apt-get install --yes cowsay emacs
3. echo cowsay > /lib/live/.../live-additional-software.conf
4. Reboot, activate read-write persistence.
5. Verify that cowsay is installed, but emacs is not.

However, no packages are installed if read-only persistence is used. I
get these errors:

Starting to install additional software...
Reading additional packages list
Found additional packages list in /lib/live/mount/persistence/sda2_unlocked/
Activating persistent software packages
Reading additional packages list
Found additional packages list in /lib/live/mount/persistence/sda2_unlocked/
Will install the following packages: cowsay
Reading package lists...
E: Could not open file /var/cache/apt/pkgcache.bin - open (5:
Input/output error)
E: Failed to truncate file - ftruncate (9: Bad file descriptor)
E: The package lists or status file could not be parsed or opened.
apt-get exited with returncode 100
Warning: installation of cowsay failed
Starting to upgrade additional software...
Hit http://deb.torproject.org squeeze Release.gpg
[...]
Hit http://ftp.us.debian.org squeeze-updates/main i386 Packages
Reading package lists...
E: Could not open file /var/cache/apt/pkgcache.bin - open (5:
Input/output error)
E: Failed to truncate file - ftruncate (9: Bad file descriptor)
E: The package lists or status file could not be parsed or opened.
apt-get exited with returncode 100
Warning: the update failed.


IMHO this is still good enough for a tech review. Hopefully the
read-only bug can be identified and fixed as a bugfix before the 0.18
release.

Merged!

Cheers!