Size: 2417
Comment:
|
Size: 2993
Comment: reply to anthony about testing a glob-expanded array with [[ -n.
|
Deletions are marked like this. | Additions are marked like this. |
Line 28: | Line 28: |
Anthony, The above test only works if `shopt -s nullglob` is set. If it is not set (it's not set by default), then files=(*.txt) will put the string `*.txt` in `files`'s first element. So `[[ $files ]]` will test `[[ "*.txt" ]]`. It's also not very good style to use the side effect of treating an array as a string to expand the first element, it's a good idea to be explicit that you expect `files` to be an array and didn't just write a typo. It's easy to think that the author that wrote `cp "$files" "$dir/"` intended to copy all the files, not just the first. |
This page is for any feedback on the BashGuide. Feel free to ask any questions you want.
Q: I would like to cover bash in a course I will soon be teaching. May I use these materials for my course, provided the source is cited? Thanks for this excellent work, btw.
Absolutely. Perhaps we should consider putting a license on these documents. Creative commons something perhaps. - Lhunath
Q. The following suggestion can be harmful: echo 'PATH="$HOME/bin:$PATH"' >> "$HOME/.bashrc" If someone places a script named 'ls' in the bin directory of $HOME, you would be running it every time you typed 'ls', instead of running the builtin 'ls'. This can be harmful. I suggest echo 'PATH="$PATH:$HOME/bin"' >> "$HOME/.bashrc"
It is not harmful. Anyone with write access to ~/bin will generally have write access to ~/.bashrc, so there is no additional loss of security here. The original proposal also allows the user to wrap standard utilities, which yours cannot (in fact, that exactly why you proposed it). I believe being able to wrap utilities can be very useful. - Lhunath
Feedback: I could not understand the HereDocument page (found another source that helped) and I think the page could do with working examples and a more thorough explantion. Rest of course is wonderful though, I learn so much today, thanks
Suggestion
First off, many thanks for providing the best online resource for learning Bash scripting. Secondly, I hope this page is the appropriate place to make this comment.
In a script, I want to check if there are files in a directory that matched a certain glob pattern. Question 4, check whether a directory is empty or not provided the information that I was looking for. I haven't previously used arrays in Bash but it occurred to me that instead of (( ${#files[*]} )), the following test might be easier (simpler syntax to remember) for beginners to use:
`[[ ${files} ]]`
From man bash, Referencing an array variable without a subscript is equivalent to referencing the array with a subscript of 0. If the first array element is empty (or unset?), that should imply that the array itself is empty and that the glob pattern failed to match any files. I've done a few tests locally and it seems to work fine for me - though there may be some edge cases that I haven't considered. - AnthonyGeoghegan
Anthony,
The above test only works if shopt -s nullglob is set. If it is not set (it's not set by default), then files=(*.txt) will put the string *.txt in files's first element. So [[ $files ]] will test [[ "*.txt" ]]. It's also not very good style to use the side effect of treating an array as a string to expand the first element, it's a good idea to be explicit that you expect files to be an array and didn't just write a typo. It's easy to think that the author that wrote cp "$files" "$dir/" intended to copy all the files, not just the first.