[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Patch for mips miniiso



Nicholas Breen wrote:
> The attached patch corrects the parameters sent to genisovh[1] in mips's
> build_miniiso target.  After doing so, the image is bootable and appears
> to be installing normally.
> 
> genisovh is still unpackaged - the path I'm calling it from certainly
> won't work for anyone else.  The only obvious difficulty for packaging
> is its lack of a copyright statement, which I'm sure Florian Lohoff (its
> author) could correct rapidly.

>  	install -m 644 -D $(BASE_TMP)r4k-ip22/netboot/boot.img $(TEMP_CD_TREE)/install/r4k-ip22-boot.img
>  	install -m 644 -D $(BASE_TMP)r5k-ip22/netboot/boot.img $(TEMP_CD_TREE)/install/r5k-ip22-boot.img
>  	mkisofs -r -J -o $(TEMP_MINIISO) $(TEMP_CD_TREE)
> -# Commented out for now, genisovh needs to be packaged first.
> -#	set -e; \
> -#	ISOVH_BOOTFILES= ; \
> -#	$(foreach SUB,r4k-ip22 r5k-ip22, \
> -#		start=$$(isoinfo -i $(TEMP_MINIISO) -lsR |grep -A 10 'Directory listing of /install' |grep $(SUB)-boot.img |awk '{print $$10}'); \
> -#		size=$$(isoinfo -i $(TEMP_MINIISO) -lsR |grep -A 10 'Directory listing of /install' |grep $(SUB)-boot.img |awk '{print $$5}'); \
> -#		ISOVH_BOOTFILES="$$ISOVH_BOOTFILES $(SUB):$$start,$$size";) \
> -#	../../genisovh-0.1/genisovh $(TEMP_MINIISO) $$ISOVH_BOOTFILES
> +
> +	set -e; \
> +	ISOVH_BOOTFILES= ; \
> +	$(foreach SUB,r4k-ip22 r5k-ip22, \
> +		start=$$(isoinfo -i $(TEMP_MINIISO) -lR |grep -A 10 'Directory listing of /install' |grep $(SUB)-boot.img |awk '{print $$10}'); \
> +		start=`echo $$[ 4 * $$start ]`; \
> +		size=$$(isoinfo -i $(TEMP_MINIISO) -lR |grep -A 10 'Directory listing of /install' |grep $(SUB)-boot.img |awk '{print $$5}'); \
> +		ISOVH_BOOTFILES="$$ISOVH_BOOTFILES $(SUB):$$start,$$size";) \
> +	# not a permanent path for genisovh!
> +	~/src/genisovh-0.1/genisovh $(TEMP_MINIISO) $$ISOVH_BOOTFILES

Please leave it commented out until genisovh is in the archive.
Uncommenting this would break autobuilds and daily builds.

-- 
see shy jo

Attachment: signature.asc
Description: Digital signature


Reply to: