remove obsolete fdisk hack

8321653cb3 (from 2007) introduced a hack to
work around bug #445304 in gnu-fdisk for users who may have replaced fdisk
with the classic gnu version. the hack allowed users to select an alternate
fdisk binary to use to work around the buggy binary.

bug #445304 is marked as found in v1.0-1 and fixed in v1.2-1, though may
have been fixe din v1.1. it was marked fixed in 2009.

checking the package archive, gnu-fdisk does not actually exist anymore
in debian, with one exception - it is available for arm64 on sid via
debports, and that version is 1.3 so thus includes the necessary fix
anyway.

it is thus pointless now that we still carry this hack.

Gbp-Dch: Short
This commit is contained in:
Lyndon Brown 2020-03-11 17:56:03 +00:00
parent dab716e079
commit d9f353c737
6 changed files with 5 additions and 28 deletions

View File

@ -273,22 +273,6 @@ Set_config_defaults ()
;;
esac
# Setting fdisk
if [ -z "${LB_FDISK}" ] || [ ! -x "${LB_FDISK}" ]
then
# Workaround for gnu-fdisk divertion
# (gnu-fdisk is buggy, #445304).
if [ -x /sbin/fdisk.distrib ]
then
LB_FDISK="fdisk.distrib"
elif [ -x /sbin/fdisk ]
then
LB_FDISK="fdisk"
else
Echo_error "Can't process file /sbin/fdisk"
fi
fi
# Setting losetup
if [ -z "${LB_LOSETUP}" ] || [ "${LB_LOSETUP}" != "/sbin/losetup.orig" ]
then

View File

@ -42,7 +42,7 @@ Losetup ()
PARTITION="${3:-1}"
${LB_LOSETUP} --read-only --partscan "${DEVICE}" "${FILE}"
FDISK_OUT="$(${LB_FDISK} -l -u ${DEVICE} 2>&1)"
FDISK_OUT="$(fdisk -l -u ${DEVICE} 2>&1)"
Lodetach "${DEVICE}"
LOOPDEVICE="$(echo ${DEVICE}p${PARTITION})"

View File

@ -103,8 +103,6 @@
[\fB\-\-distribution\-binary\fR \fICODENAME\fR]
.br
[\fB\-\-dump\fR]
.br
[\fB\-\-fdisk\fR fdisk|fdisk.dist]
.br
[\fB\-\-firmware\-binary\fR true|false]
.br
@ -349,8 +347,6 @@ defines the distribution enabled in the resulting live system (defaults to the v
defines the distribution used to build the live system (defaults to the value set in \fB\-\-distribution\fR)
.IP "\fB\-\-dump\fR" 4
prepares a report of the currently present live system configuration and the version of live\-build used. This is useful to provide if you submit bug reports, we do get all information required for us to locate and replicate an error.
.IP "\fB\-\-fdisk\fR fdisk|fdisk.dist" 4
sets the filename of the fdisk binary from the host system that should be used. This is autodetected and does generally not need any customization.
.IP "\fB\-\-firmware\-binary\fR true|false" 4
includes firmware packages in debian-installer. Defaults to true.
.IP "\fB\-\-firmware\-chroot\fR true|false" 4

View File

@ -53,6 +53,7 @@ esac
Check_package chroot /usr/share/doc/mtools mtools
Check_package chroot /sbin/parted parted
Check_package host /sbin/fdisk fdisk
echo "${LB_BOOTLOADERS}" | \
while IFS="," read -r BOOTLOADER

View File

@ -65,7 +65,6 @@ USAGE="${PROGRAM} [--apt apt|aptitude]\n\
\t [--distribution-binary CODENAME]\n\
\t [--distribution-chroot CODENAME]\n\
\t [--dump]\n\
\t [--fdisk fdisk|fdisk.dist]\n\
\t [--firmware-binary true|false]\n\
\t [--firmware-chroot true|false]\n\
\t [--force]\n\
@ -141,7 +140,7 @@ Local_arguments ()
LONG_OPTIONS="apt:,apt-ftp-proxy:,apt-http-proxy:,apt-options:,aptitude-options:,debootstrap-options:,debootstrap-script:,
apt-pipeline:,apt-recommends:,apt-secure:,apt-source-archives:,cache:,cache-indices:,cache-packages:,
cache-stages:,debconf-frontend:,debconf-priority:,dump,
initramfs:,initramfs-compression:,initsystem:,fdisk:,losetup:,mode:,system:,tasksel:,
initramfs:,initramfs-compression:,initsystem:,losetup:,mode:,system:,tasksel:,
architectures:,clean,
distribution:,parent-distribution:,parent-debian-installer-distribution:,
distribution-chroot:,parent-distribution-chroot:,
@ -330,7 +329,7 @@ Local_arguments ()
;;
--fdisk)
LB_FDISK="${2}"
Echo_warning "--fdisk is an obsolete option"
shift 2
;;
@ -1011,10 +1010,6 @@ LB_INITRAMFS_COMPRESSION="${LB_INITRAMFS_COMPRESSION}"
# (Default: ${LB_INITSYSTEM})
LB_INITSYSTEM="${LB_INITSYSTEM}"
# \$LB_FDISK: set fdisk program
# (Default: autodetected)
LB_FDISK="${LB_FDISK}"
# \$LB_LOSETUP: set losetup program
# (Default: autodetected)
LB_LOSETUP="${LB_LOSETUP}"

View File

@ -48,6 +48,7 @@ Create_lockfile .lock
# Checking depends
Check_package chroot /sbin/mkdosfs dosfstools
Check_package chroot /sbin/parted parted
Check_package host /sbin/fdisk fdisk
# Installing depends
Install_package