chenall / grub4dos

外部命令和工具源码:https://github.com/chenall/grubutils 下载:
http://grub4dos.chenall.net
GNU General Public License v2.0
642 stars 136 forks source link

ls ()/ does not list files in root 2017-03-30 #136

Closed steve6375 closed 7 years ago

steve6375 commented 7 years ago

In previous versions

root /_ISO ls / << lists files in /_ISO ls ()/ << lists files in root

With new versions root /_ISO ls / << lists files in /_ISO ls ()/ << lists files in /_ISO

cat command is also affected

This breaks batch files, etc, serious bug!

yaya2007 commented 7 years ago

"root /_ISO" usage, grub4dos-0.4.5c-2016-01-18 version also can not do The correct approach is "root ()/_ISO"

steve6375 commented 7 years ago

Sorry - I made typo, BUG is shown below

\menu.lst

root ()/_ISO/e2b/
ls /
echo
echo
ls ()/
pause

image image

steve6375 commented 7 years ago

Same result if

root ()/_ISO/e2b
ls /
echo
echo
ls ()/
pause
steve6375 commented 7 years ago

P.S. re.

VOL_Dir is "xxxx".

Should be Volume Name and also add \n

Volume Name is "xxxx".\n

and The current working directory(i.e., relative path) is should be just The current working directory (relative path) is

space before ( and just (relative path)

steve6375 commented 7 years ago

That seems to work. But current working directory text could be made better?

yaya2007 commented 7 years ago

"But current working directory text could be made better?" Please explain further

steve6375 commented 7 years ago

See comment above.

The current working directory(i.e., relative path) is incorrect English should be just The current working directory (relative path) is

space before ( and just (relative path)

image

i.e., is incorrect. Sentences are 'xxxxxx, i.e. yyyyyyy' or use (i.e. relative path). no comma after i.e. Always use space before (

steve6375 commented 7 years ago

FYI: Many other examples where a space should be used but is missing... image

steve6375 commented 7 years ago

Thanks!