Merge lp:~cwayne/ubuntu/saucy/lxc-android-config/timezone into lp:ubuntu/saucy/lxc-android-config

Proposed by Chris Wayne
Status: Needs review
Proposed branch: lp:~cwayne/ubuntu/saucy/lxc-android-config/timezone
Merge into: lp:ubuntu/saucy/lxc-android-config
Diff against target: 0 lines
To merge this branch: bzr merge lp:~cwayne/ubuntu/saucy/lxc-android-config/timezone
Reviewer Review Type Date Requested Status
Colin Watson Needs Fixing
Review via email: mp+186953@code.launchpad.net

Description of the change

Adding /etc/timezone and /etc/localtime into /etc/system-image/writable-paths

To post a comment you must log in.
102. By Chris Wayne

Adding in /etc/adjtime

Revision history for this message
Colin Watson (cjwatson) wrote :

/etc/adjtime is misspelt.

Given that it isn't possible to replace a symlink atomically without being able to rename over the top of it, the plan I thought we agreed earlier was to have /etc/localtime be a symlink to another symlink in a directory which is writable, so that we can symlink() and rename() in that directory. I would be concerned about applying your change even as a temporary measure because persistent mounts can apparently be hard to undo later for upgrades.

review: Needs Fixing
Revision history for this message
Colin Watson (cjwatson) wrote :

(And similar considerations might apply to /etc/timezone in some cases, and even perhaps to /etc/adjtime.)

103. By Chris Wayne

Fix typo in /etc/adjtime

Revision history for this message
Martin Pitt (pitti) wrote :

For the record, this is handled in bug 1227520. I reverted that change and replaced it with the two-layer symlink approach there.

Unmerged revisions

103. By Chris Wayne

Fix typo in /etc/adjtime

102. By Chris Wayne

Adding in /etc/adjtime

101. By Chris Wayne

Adding /etc/timezone and /etc/localtime to allow for changing timezone on device

Preview Diff

Empty

Subscribers

People subscribed via source and target branches

to all changes: