I am trying to output the name of every file that in it's last line has "#FIXME" into a new file.
find -type f -print0 |while IFS= read -rd '' file; do
tail -n1 "$file" | grep -q FIXME && echo "$file" > newlog.log
done
This only outputs one file containing "#FIXME" to newlog.log when there are actually mutliple files.
Could anyone tell me what is going wrong?
You have setup a loop which executes this line in the middle for many files:
tail -n1 "$file" | grep -q FIXME && echo "$file" > newlog.log
This line however creates and writes the file "newlog.log" each time,
which results in the already existing content being overwritten each time, with only one file name. This gave you the impression that only one file name was written to the log file. It happens to be the last one.
In order to get a list of all the written file names, i.e. to keep all of the content, you need to append to the existing loggile, instead of overwriting it.
To do that, use >> instead of >.
tail -n1 "$file" | grep -q FIXME && echo "$file" >> newlog.log
This creates the need to consider the content of the log file before executing the script.
Either you are fine with always also keeping the content of the previous script execution, which I guess you are probably not, or you can make sure the file has no content.
You can either delete the file and create it freshly. Or you do once use the > intentionally. This would allow you to make something of a headline for the list, e.g. containing the date of the creation and the path you executed it in.
>> newlog.log inside the loop, it's probably better to redirect the loop as a whole to newlog.log, by putting the > newlog.log after the done.You need to append to the file, not overwrite. ANd this
tail -n1 "$file" | grep -q FIXME && echo "$file" > newlog.log
should be
tail -n1 "$file" | grep -q FIXME && echo "$file" >> newlog.log
> write to the file from begin (erase previous content) >> append to the existing content of the file
FIXMElines aren't reliably the last line, you could simplify your life withfind . -type f -exec grep -l -e FIXME {} + > newlog.log.>> newlog.logto also keep all the previous output?