search for eatmydata only searches root , not in target

Bug #1358350 reported by Scott Moser
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
curtin
Fix Released
Medium
Unassigned
curtin (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

util.install_packages tries to determine if 'eatmydata' is available in the target, but inadvertently only checks in the host.
this results in:

chroot: failed to run command 'eatmydata': No such file or directory
Unexpected error while running command.
Command: ['chroot', '/tmp/tmp75AQEC/target', 'eatmydata', 'apt-get', 'install', '--quiet', '--assume-yes', '--option=Dpkg::options::=--force-uns
afe-io', 'linux-generic-lts-trusty']
Exit code: 127

The issue is use of os.path.join():

$ python -c 'import os.path; print(os.path.join("/target", "/usr/sbin", "foo"));'
/usr/sbin/foo

Revision history for this message
Scott Moser (smoser) wrote :

fixed in trunk at revno 148.

Changed in curtin:
status: New → Fix Committed
Scott Moser (smoser)
Changed in curtin (Ubuntu):
status: New → Confirmed
importance: Undecided → Medium
Changed in curtin:
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package curtin - 0.1.0~bzr153-0ubuntu1

---------------
curtin (0.1.0~bzr153-0ubuntu1) utopic; urgency=medium

  * New upstream snapshot.
    * support for creating swap file.
    * fix bug when specifying kernel mapping in config.
    * install_kernel: run apt-get udpate before searching cache (LP: #1358424)
    * correctly check for availability of eatmydata (LP: #1358350)
    * support installation on ppc64 systems (ubuntu ppc64el)
 -- Scott Moser <email address hidden> Thu, 21 Aug 2014 03:35:53 -0400

Changed in curtin (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Scott Moser (smoser) wrote : Fixed in Curtin 17.1

This bug is believed to be fixed in curtin in 17.1. If this is still a problem for you, please make a comment and set the state back to New

Thank you.

Changed in curtin:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.