system-partitions-formatted validation missing from new partitioner

Bug #89461 reported by Kristian Klette on 2007-03-03
38
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
High
Colin Watson

Bug Description

Binary package hint: ubiquity

ubiquity crashes if you try to install on an allready installed system. ubiquity-installer gives OSError: File exists on line 428 (ish, around there somewhere)

Should be an easy fix, just wrap a try tag around it and maybe give the user some options, overwrite the file, skip the file or reformat the partition.
Shouldn't die hard at least ;)

This is from the herd5 i386 desktop cd btw :)

Related branches

Kristian Klette (klette) on 2007-03-03
description: updated
Stephen Gornick (sgornick) wrote :

With Feisty Herd-5 i386 I too get this:
   OSError: [Errno 17] File exists.

From syslog:
   File "/usr/share/ubiquity/install.py", line 1468, in <module>
     install.run()
   File "/usr/share/ubiquity/install.py", line 307, in run
     self.copy_all()
   File "/usr/share/ubiquity/install.py", line 481, in copy_all
     os.mknod(targetpath, stat.S_IFCHR | mode, st.st_rdev)
 OSError: [Errno 17] File exists

It occurs when I set as the root mount point (/) an existing ext3 partition that already has an Ubuntu Feisty Herd installation.

For me, there is no indication that the installer crashed, and instead had to determine problem from the syslog. On the desktop is a /target mount:
  $ cat /etc/mtab |grep -i target
/dev/hda7 /target ext3 rw 0 0

Stephen Gornick (sgornick) wrote :

For anyone else hitting this error, a workaround (if you truly don't need anything on that partition) is to click on the "format" checkbox in the "Prepare partitions" page.

Colin Watson (cjwatson) on 2007-03-08
Changed in ubiquity:
assignee: nobody → kamion
importance: Undecided → High
status: Unconfirmed → Confirmed
Colin Watson (cjwatson) on 2007-03-09
Changed in ubiquity:
status: Confirmed → Fix Committed
Colin Watson (cjwatson) wrote :

ubiquity (1.3.27) feisty; urgency=low

  * Add a 'debian/rules update-local' target to allow updating d-i/source/
    without updating debian/changelog, which is sometimes convenient for
    local hacking.
  * New partitioner: Add validation for system partitions being formatted
    (LP: #89461).
  * Really use MyKeyboardNames.pl (based on xkb-data) rather than
    KeyboardNames.pl.
  * Automatic update of included source packages: console-setup 1.13ubuntu8.

 -- Colin Watson <email address hidden> Mon, 12 Mar 2007 12:15:39 +0000

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

Other bug subscribers

Bug attachments