Horrible hacks to make the image actually create at a size that fits everything.

Feel free to rehack it if you can do it better.
Issue 954


git-svn-id: http://svn.dmdirc.com/trunk@3855 00569f92-eb28-0410-84fd-f71c24880f

installer/osx/makeInstallerOSX.sh

@@ -349,60 +349,83 @@ if [ "" = "${HDIUTIL}" ]; then
 		LINUXIMAGE=${PWD}/DMDirc.dmg
 	fi;
 
-	SIZE=$((`du -sb ${DMG} | awk '{print $1}'`  + 10))
 	# Non-OSX
 	# This doesn't work quite aswell as on OSX, but it works.
-	if [ "" = "${MKHFS}" -a "" != "${MKHFSPLUS}" ]; then
-		MKHFS=${MKHFSPLUS}
-	fi;
-	# mkfs.hfs will only create 512kb+ sized images
-	if [ ${SIZE} -lt 524288 ]; then
-		echo "Size is less than 512kb"
-		SIZE=524288;
-	fi;
-	dd if=/dev/zero of=${LINUXIMAGE} bs=${SIZE} count=1
-	${MKHFS} -v 'DMDirc' ${LINUXIMAGE}
-	# however older versions of mkfs.hfs will only create 4mb+ sized images :/
-	if [ $? -eq 1 ]; then
-		if [ ${SIZE} -lt 4194304 ]; then
-			echo "Size is less than 4MB"
-			SIZE=4194304;
+	IMGBLOCKSIZE=4096 # OSX Default = 4096
+	MVRES=0
+	createImage() {
+		MVRES=0
+		SIZE=${1}
+		if [ "" = "${MKHFS}" -a "" != "${MKHFSPLUS}" ]; then
+			MKHFS=${MKHFSPLUS}
+		fi;
+		# mkfs.hfs will only create 512kb+ sized images
+		if [ ${SIZE} -lt 524288 ]; then
+			echo "Size is less than 512kb"
+			SIZE=524288;
 		fi;
 		dd if=/dev/zero of=${LINUXIMAGE} bs=${SIZE} count=1
-		${MKHFS} -v 'DMDirc' ${LINUXIMAGE}
-	fi;
-	# Now try and mount
-	# This could be a problem, as linux requires either root to mount, or an fstab
-	# entry.
-	# Try to mount, if this fails, let the user know what to add to fstab.
-	if [ -e ${LINUXIMAGEDIR} ]; then
-		rm -Rf ${LINUXIMAGEDIR}
-	fi;
-	mkdir ${LINUXIMAGEDIR}
-	mount ${LINUXIMAGEDIR}
-	MOUNTRES=${?}
-	if [ ${MOUNTRES} -ne 0 ]; then
-		# Try using full parameters - could be running as root.
-		mount -t hfsplus -o loop ${RUNNAME} ${LINUXIMAGEDIR}
+		${MKHFS} -b ${IMGBLOCKSIZE} -v 'DMDirc' ${LINUXIMAGE}
+		if [ ${?} -ne 0 ]; then
+			echo "mkfs.hfs failed to create the image, aborting."
+			exit 1;
+		fi;
+		# Now try and mount
+		# This could be a problem, as linux requires either root to mount, or an fstab
+		# entry.
+		# Try to mount, if this fails, let the user know what to add to fstab.
+		if [ -e ${LINUXIMAGEDIR} ]; then
+			rm -Rf ${LINUXIMAGEDIR}
+		fi;
+		mkdir ${LINUXIMAGEDIR}
+		mount ${LINUXIMAGEDIR}
 		MOUNTRES=${?}
-	fi;
-	if [ ${MOUNTRES} -ne 0 ]; then
-		echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
-		echo "@                               ERROR                               @"
-		echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
-		echo "You do not have permission to mount the image."
-		echo "Please add the following lines to /etc/fstab and rcd oun this script again"
-		echo "# DMDirc OSX dmg image"
-		echo "${LINUXIMAGE} ${LINUXIMAGEDIR} auto users,noauto,loop 0 0"
-		echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
-		echo "@                               ERROR                               @"
-		echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
-		exit 1;
-	fi;
-	mv -fv ${DMG}/* ${LINUXIMAGEDIR}
-	mv -fv ${DMG}/.[A-Za-z]* ${LINUXIMAGEDIR}
-	umount ${LINUXIMAGEDIR}
-	# If anyone finds out how to compress these nicely, add it here.
+		if [ ${MOUNTRES} -ne 0 ]; then
+			# Try using full parameters - could be running as root.
+			mount -t hfsplus -o loop ${RUNNAME} ${LINUXIMAGEDIR}
+			MOUNTRES=${?}
+		fi;
+		if [ ${MOUNTRES} -ne 0 ]; then
+			echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
+			echo "@                               ERROR                               @"
+			echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
+			echo "You do not have permission to mount the image."
+			echo "Please add the following lines to /etc/fstab and re run this script again"
+			echo "# DMDirc OSX dmg image"
+			echo "${LINUXIMAGE} ${LINUXIMAGEDIR} auto users,noauto,loop 0 0"
+			echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
+			echo "@                               ERROR                               @"
+			echo "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@"
+			exit 1;
+		fi;
+		mv -fv ${DMG}/* ${LINUXIMAGEDIR}
+		MVRES=${?}
+		if [ ${MVRES} -ne 0 ]; then
+			echo "Image incorrect size."
+		else
+			mv -fv ${DMG}/.[A-Za-z]* ${LINUXIMAGEDIR}
+			MVRES=${?}
+			if [ ${MVRES} -ne 0 ]; then
+				echo "Image incorrect size."
+			fi;
+		fi;
+		umount ${LINUXIMAGEDIR}
+		# If anyone finds out how to compress these nicely, add it here.
+	}
+	
+	INCREASE=10
+	IMAGESIZE=$(((`du -s --block-size ${IMGBLOCKSIZE} ${DMG} | awk '{print $1}'` + ${INCREASE}) * ${IMGBLOCKSIZE} ))
+	createImage ${IMAGESIZE}
+	
+	# This sucks, but it makes sure the image gets created at a size that contains
+	# everything.
+	# I can't manage to get du to return the size that the files actually end up
+	# using on the image.
+	while [ ${MVRES} -ne 0 ]; do
+		INCREASE=$((${INCREASE} + 5))
+		IMAGESIZE=$(((`du -s --block-size ${IMGBLOCKSIZE} ${DMG} | awk '{print $1}'` + ${INCREASE}) * ${IMGBLOCKSIZE} ))
+		createImage ${IMAGESIZE}
+	done;
 	
 	if [ "${LINUXIMAGE}" != "${RUNNAME}" ]; then
 		# Rename the image