Re: [PATCH 2/4] x86, build: Add new paths for isolinux.bin and ldlinux.c32

[Ingo Molnar]

* changbin.du@xxxxxxxxx <changbin.du@xxxxxxxxx> wrote:

> From: Changbin Du <changbin.du@xxxxxxxxx>
> 
> Recently I failed to build isoimage target, because the path of isolinux.bin
> changed to /usr/xxx/ISOLINUX/isolinux.bin, as well as ldlinux.c32 which
> changed to /usr/xxx/syslinux/modules/bios/ldlinux.c32.
> 
> This patch has a improvement of the file search:
>   - Show a error message instead of silent fail.
>   - Add above new paths.

How about:

  This patch improves the file search logic:
    - Show an error message instead of failing silently
    - Add the new paths listed above.


> +		if [ $i = end -a -z "$isolinux" ] ; then
> +			echo 'Need isolinux.bin, please install syslinux/isolinux'

How about:

			echo 'Need an isolinux.bin file, please install syslinux/isolinux'

Thanks,

	Ingo