Merge ~jawn-smith/ubuntu/+source/flash-kernel:ubuntu/devel into ubuntu/+source/flash-kernel:ubuntu/devel

Proposed by William Wilson
Status: Needs review
Proposed branch: ~jawn-smith/ubuntu/+source/flash-kernel:ubuntu/devel
Merge into: ubuntu/+source/flash-kernel:ubuntu/devel
Diff against target: 30 lines (+9/-2)
2 files modified
debian/changelog (+7/-0)
functions (+2/-2)
Reviewer Review Type Date Requested Status
Dave Jones (community) Approve
Review via email: mp+429001@code.qastaging.launchpad.net

Description of the change

When running in a chroot, flash-kernel should exit cleanly as it does when running in a container.

To post a comment you must log in.
Revision history for this message
Dave Jones (waveform) wrote :

Discussed this with Alex, who first introduced FK_FORCE_CONTAINER; he'll need to update livecd-rootfs which is the only thing using FK_FORCE_CONTAINER externally at the moment but otherwise there don't appear to be any problems with extending the exception to cover chroots as well.

review: Approve
Revision history for this message
Dave Jones (waveform) wrote :

Follow up comment from Alex: he'd actually favour introducing a separate var to avoid having to change the external interface.

Any chance you could post a follow up commit to implement that (presumably as FK_FORCE_CHROOT)? Given that wouldn't require any coordination with livecd-rootfs I could sponsor that change immediately.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches