Hacker News Reader

Comment by DHowett | Hacker News Reader

"In particular, the presence of \0 in your data will throw xargs for a loop (no pun intended). In this case, it is necessary to use the -0 option. This is particularly relevant if you are using the find command and supplying the -print0 option."

This glosses over the fact that those two things are intended to be used together. Delimiting filenames with a \0 is superior to delimiting them with spaces or newlines, as filenames cannot usually contain ASCII NUL; xargs and find can therefore be used to remove heterogeneous lists of files.

Additionally, the example given may better be worded as "find . -name '* junk* -exec rm '{}' \;" or even "find . -name '* junk* | xargs rm". (as given it strikes me as a useless use of grep and is additionally fragile if ls is aliased to "ls --color=always", spewing ANSI into

EDIT: Please excuse the additional spaces in the find command arguments, they are a formatting avoidance feature.

View on HN

If we're ripping this sample apart, I'd like to point out that

  rm *junk*

in this case would've done the same, without spawning lots of useless stuff.

xargs is cool. The example fails to show a decent use case. For me a line like that would indicate that someone is _not_ familiar with his tools. Similar to

  cat somefile | grep something

instead of

  grep something somefile

Pipes and processes aren't endangered or (in practice) limited, but we still don't need to be wasteful?

Furthermore, parsing the output of `ls` is a terrible practice[1] that makes for an even poorer sample.

A little further in the article, the author even suggests running

    `ls | grep junk`

(instead of the much simpler `ls junk*`. Do I need to point that out?).

I also know some people are going to retort that wasting a process or a pipe is not that bad. This discussion has been had over and over again, probably ever since Unix was invented.

In any case, the poor example chosen, as well as the flaky commands used, seriously hinder the credibility of the article.

[1]: http://mywiki.wooledge.org/ParsingLs

They don't do the quite the same thing.

   ls | grep junk

is more similar to

   ls -d *junk*

You can argue the first is simpler. You can also argue more successfully that it is more general.

edit: I acknowledge the points in your link. Advocates for the unix "bunch of small tools" philosophy tend to overlook its weak spots.

If you still want to use xargs and ls perhaps avoiding \0's (perhaps so you can easily grep on the data) you can use "ls -Q | xargs rm". The advantage of that is you can still easily use grep on the data as it is still newline delimited but you won't have problems with spaces.

Also ls is never aliased to "ls --color=always" unless you've done that more likely is its aliased to "ls --color=auto" or similar which will work normally.

> more likely is its aliased to "ls --color=auto" or similar which will work normally.

My ls (which I haven't much reconfigured) behaves very differently depending on whether it's outputting to a shell or a pipe or redirect. Not just colours, but columning and filetype indicators as well.

Instead of -exec rm {} \; you could use -delete option as well.

This option is not portable at all. For instance, it won't work on Solaris' find. I strongly advise against the use of non portable options.

It may be a good idea when writing scripts that have to be portable.

But there's quite a lot of nice non-standard options in the GNU userspace tools, and it would be a shame not to use them when available.

Little known fact: the -0/-print0 idea is due to DJB.

Reply on news.ycombinator.com