Hi Ingo, On Sun, Nov 05, 2017 at 10:33:53AM +0100, Ingo Molnar wrote: > > * 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' > The new description is much better, will update. Thanks.
> Thanks, > > Ingo -- Thanks, Changbin Du
signature.asc
Description: PGP signature
