* changbin...@intel.com <changbin...@intel.com> wrote: > From: Changbin Du <changbin...@intel.com> > > 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