Re: add min and max to coreutils?

2009-07-03 Thread Jim Meyering
Pádraig Brady wrote: Joshua Bronson wrote: I was looking for a command line utility to get the min and max from a file (rather than the much less efficient sort | {head,tail} -1) and I couldn't find one. Someone on linuxquestions.org directed me toward the Generic Mapping Tools

Re: [PATCH] tail: add comments noting potential inotify-related problems

2009-07-03 Thread Jim Meyering
Giuseppe Scrivano wrote: I took a look at the problems you reported. The first one is fixed with the first attached patch. I have tested it under Linux 2.6.18-6-xen-686. Thank you. Applied with only minor formatting changes (i.e., I moved to start of continued line). tail -F works until

Re: add min and max to coreutils?

2009-07-03 Thread Pádraig Brady
Joshua Bronson wrote: What are your concerns? Adding the kitchen sink to `sort`. Though if we were to add this functionality to coreutils, sort seems like the right place. I've often used sort | tail, but only for small data sets where the overhead is negligible. I'm leaning towards this

Re: [PATCH] tail: add comments noting potential inotify-related problems

2009-07-03 Thread Giuseppe Scrivano
Hi Jim, Jim Meyering j...@meyering.net writes: I don't (yet?) see why a tree would be the preferred data structure. ... Because inotify doesn't add recursive watchers. For example, you want to follow by name `/var/foo/bar', and `/var/foo' doesn't exist yet. To catch the event for the `bar'

Re: [PATCH] tail: add comments noting potential inotify-related problems

2009-07-03 Thread Jim Meyering
Giuseppe Scrivano wrote: Hi Jim, Jim Meyering j...@meyering.net writes: I don't (yet?) see why a tree would be the preferred data structure. ... Because inotify doesn't add recursive watchers. For example, you want to follow by name `/var/foo/bar', and `/var/foo' doesn't exist yet. To

FYI, updated to gnulib's latest

2009-07-03 Thread Jim Meyering
I've just updated coreutils gnulib submodule to the latest. ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils

Re: [PATCH] tail: add comments noting potential inotify-related problems

2009-07-03 Thread Giuseppe Scrivano
Hi Jim, Jim Meyering j...@meyering.net writes: I'm not convinced that adding a lot of new code just to make tail -f handle a far-fetched case like that is worthwhile. But that's just my opinion, and if someone can present a use-case that makes it seem the additional code would be put to

Re: add min and max to coreutils?

2009-07-03 Thread Joshua Bronson
2009/7/3 Pádraig Brady p...@draigbrady.com I'm leaning towards this being useful given how much faster it would be for large datasets. Glad to hear others might find this useful! If coreutils has an issue tracker (didn't see one linked off http://www.gnu.org/software/coreutils/), I'd love to