Re: echo -e produces no outputs

2009-07-17 Thread Bo Borgerson
Bauke Jan Douma wrote: Eric Blake wrote on 07/17/2009 09:09 PM: Instead of using echo (which POSIX itself admits is fraught with portability problems), use printf: printf -- '-e\n' or: echo -n - ; echo e or: echo -e \0055e ;) Bo ___

Re: [PATCH] tests: refactor to use the new getlimits utility

2008-12-12 Thread Bo Borgerson
Pádraig Brady wrote: I'd especially like a review of the perl bits Hi Pádraig, I'm not sure that this function will behave quite as you intended it to: +sub getlimits() +{ + my $NV; + open NV, getlimits | or die Error running getlimits\n; + my %limits = map {split /=|\n/} NV; + return

Re: [bug #24974] Document that comm's option -1, -2 and -3 can be combined

2008-12-01 Thread Bo Borgerson
Pádraig Brady wrote: p.s. Those new --check-order --nocheck-order options confuse me. When they were added I only took a quick look at the implementation rather than the interface (which Bo Borgerson kindly sped up for us). Perhaps something like this would be clearer: --check-order

Re: [RFC] wc: add new option, --progress

2008-11-03 Thread Bo Borgerson
Pádraig Brady wrote: I'm not sure this is generally that useful. It reminds me of the more general pv tool that I have found useful in the past: http://www.ivarch.com/programs/quickref/pv.shtml Thanks Pádraig, this is exactly what I was looking for :) Bo

[RFC] wc: add new option, --progress

2008-10-31 Thread Bo Borgerson
80089c8b1d616ec3c3a88b4f58131506a5aa43f3 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 29 Oct 2008 11:00:06 -0400 Subject: [PATCH] wc: add new option, --progress * src/wc.c (set_do_monitor) ALRM handler sets DO_MONITOR flag that triggers output. (write_counts) Unset DO_MONITOR flag. New argument, is_final

Re: RFC: wc --max-line-length vs. TABs [Re: Bug in wc

2008-08-22 Thread Bo Borgerson
Jim Meyering wrote: I'm tempted to make the change, but it seems too drastic, after 11 years. Do any of you rely on the current TAB-counting behavior of GNU wc? Hi, It looks like TAB characters aren't alone in being counted by printed width rather than count: $ echo '好' | wc -L 2 Does it

Re: sort -m does too much work

2008-08-05 Thread Bo Borgerson
David Muir Sharnoff wrote: I've got 200 1GB pre-sorted files. If I try to merge them with sort -m, it is obvioulsy trying to do too much work: after running for a couple minutes, it has not produced any output but it has made a 5 GB temporary file. When the input is pre-sorted, no

Re: Multi-threading in sort(or core-utils)

2008-06-28 Thread Bo Borgerson
James Youngman wrote: On Thu, Jun 26, 2008 at 1:22 AM, Bo Borgerson [EMAIL PROTECTED] wrote: If all inputs are regular files then SORTERS read directly rather than being fed by an extra process. Does that work with multi-byte character sets? Hi James, Each sorter's portion of input

Re: Multi-threading in sort(or core-utils)

2008-06-25 Thread Bo Borgerson
Bo Borgerson wrote: Cons: - It's limited by the feeder. If the sorters were able to read at their own pace I think this would scale better. - It uses N+2 processes. When sorters are run in parallel there are two helper processes, one feeding input and one merging output. Hello again

Re: who(1) exit status

2008-06-23 Thread Bo Borgerson
Andreas Schwab wrote: Eric Blake [EMAIL PROTECTED] writes: According to Shal-Linux-Ind on 6/23/2008 4:05 AM: | Hi, | | who(1) exit status is always 0. | | $ who --v | who (coreutils) 5.2.1 Thanks for the report. Consider upgrading - that is several years old, and the latest stable

Re: Multi-threading in sort(or core-utils)

2008-06-22 Thread Bo Borgerson
Paul Eggert wrote: Bo Borgerson [EMAIL PROTECTED] writes: Does this sound like a step in the right direction for sort? If I were to clean this up and submit it would you be willing to assess its viability as a portable improvement? Yes, and yes. And thanks! Hi Paul, When I isolated my

sort --batch-size non-merge bug

2008-06-19 Thread Bo Borgerson
in severe performance degradation. I've attached a fix for this bug, including a test that exercises it. I've also pushed to repo.or.cz. Sorry for introducing this. Thanks, Bo From 91aa3fb5a2636dc918bafa67f3a097d646cac075 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Thu, 19

Re: Feature request - base64 Filename Safe Alphabet

2008-06-18 Thread Bo Borgerson
2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 18 Jun 2008 19:16:01 -0400 Subject: [PATCH] base64: Provide an interface for alphabet configurationi and a filesafe alphabet. * lib/base64.c (base64_encode_a): Was base64_encode. Takes an alphabet. (base64_encode_alloc_a

Re: rebased patches?

2008-06-16 Thread Bo Borgerson
Jim Meyering wrote: This brings up another (as yet unwritten) guideline: Don't change translatable strings if you can avoid it. If you must rearrange lines, extract and create new strings, rather than extracting and moving into existing blocks. This avoids making unnecessary work for

Re: Multi-threading in sort(or core-utils)

2008-06-16 Thread Bo Borgerson
Paul Eggert wrote: [EMAIL PROTECTED] wrote: I think it is good idea to make option(or by default) for sorting in threads to increase performance on systems that might execute more than one thread in parallel. Klimentov Konstantin. I agree. That's been on my to-do list for years. (It

Re: Multi-threading in sort(or core-utils)

2008-06-13 Thread Bo Borgerson
[EMAIL PROTECTED] wrote: Hello Few minutes ago i used sort -u for sorting big file(236 Mb). I have 2 core cpu(core 2 duo), but i found that sort use only one cpu(work in one thread). I think it is good idea to make option(or by default) for sorting in threads to increase performance on

Re: rebased patches?

2008-06-12 Thread Bo Borgerson
Jim Meyering wrote: Also, I made some syntactic changes to fit with my policy preferences (no operators, and adjusted const placement): Thanks Jim. BTW - Do you have these policy preferences collected somewhere? I don't remember seeing some of them in the general GNU standards document. If

Re: [OT] Is od broken?

2008-06-12 Thread Bo Borgerson
Eric Blake wrote: OK, I'll keep them as separate commits. Bo inspired me, and I finally figured out how to use repo.or.cz. Now you can do: git fetch git://repo.or.cz/coreutils/ericb.git refs/heads/od to see my patch series. Awesome! That was actually Jim's suggestion, but I'm glad to

Re: du v5.93: traverses subdirectories although --separate-dirs and --summarize are set?

2008-05-29 Thread Bo Borgerson
Volker Badziong wrote: Hello, I am running du (GNU coreutils) 5.93. When executing e.g. du --separate-dirs --summarize /etc/ you are only interested in the total space consumed by stuff in /etc/, not within any subfolders. But nevertheless du traverses all subdirectories,

Re: horrible utf-8 performace in wc

2008-05-08 Thread Bo Borgerson
Bruno Haible wrote: If you want wc to count characters after canonicalization, then you can invent a new wc command-line option for it. But I would find it more useful to have a filter program that reads from standard input and writes the canonicalized output to standard output; that would be

Re: horrible utf-8 performace in wc

2008-05-07 Thread Bo Borgerson
Pádraig Brady wrote: canonically équivalent canonically équivalent Pádraig. p.s. I Notice that gnome-terminal still doesn't handle combining characters correctly, and my mail client thunderbird is putting the accent on the q rather than the e, sigh. They both render correctly here

Re: horrible utf-8 performace in wc

2008-05-07 Thread Bo Borgerson
Jim Meyering wrote: Bo Borgerson [EMAIL PROTECTED] wrote: I may be misinterpreting your patch, but it seems to me that decrementing count for zero-width characters could potentially lead to confusion. Not all zero-width characters are combining characters, right? It looks ok to me, since

Re: Feature request - base64 Filename Safe Alphabet

2008-05-05 Thread Bo Borgerson
and is rebased against the current HEAD. I've also made this available via: $ git fetch git://repo.or.cz/coreutils/bo.git base64-merge:base64-merge Thanks, Bo From 9131d82c32e00b606eb79d083ef8309178460ac5 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 30 Apr 2008 17:40

Re: Feature request - base64 Filename Safe Alphabet

2008-05-05 Thread Bo Borgerson
. (decode_4): Helper function to be used by base64_decode_ctx. Currently unused. (base64_decode_ctx_init): Initialize a decode context structure. * lib/base64.h (struct base64_decode_context) To be used by base64_decode_ctx Signed-off-by: Bo Borgerson [EMAIL PROTECTED] --- lib/base64.c | 135

[PATCH] base64: remove some unused/redundant getopt code

2008-05-05 Thread Bo Borgerson
Hi, I noticed these when I was poking around in base64 recently. Looks like they're vestigial. Bo From 29288df82cd764b384bcd6535925c82ffca8ffc6 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Mon, 5 May 2008 21:58:28 -0400 Subject: [PATCH] base64: remove some unused

Re: [PATCH] Add new program: psub

2008-05-03 Thread Bo Borgerson
Bo Borgerson wrote: Hi, This program uses the temporary fifo management system that I built for zargs to provide generic process substitution for arguments to a sub-command. This program has some advantages over the process substitution built into some shells (bash, zsh, ksh, ???): 1

Re: coreutils test coverage

2008-04-30 Thread Bo Borgerson
upstream to lcov. I'm sure they'll appreciate that, and when it works its way down onto coreutils hackers' boxes they'll appreciate it, too. :) Thanks again! Bo From decc65cb8f2608743ae906cf4479dd084219ae5d Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 30 Apr 2008 08:49:59

Re: Feature request - base64 Filename Safe Alphabet

2008-04-30 Thread Bo Borgerson
for a merge of the two? Thanks, Bo From e63ed95710560a7da7f4fd681add4f0e8172bc7a Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 30 Apr 2008 17:40:38 -0400 Subject: [PATCH] A step toward an upstream compatible base64 * gl/lib/base64.c (base64_decode_ctx): If no context

Re: Feature request - base64 Filename Safe Alphabet

2008-04-30 Thread Bo Borgerson
Jim Meyering wrote: http://thread.gmane.org/gmane.comp.lib.gnulib.bugs/8670/focus=12523 Sorry I didn't dig that up initially. Since his packages are the main consumer other than coreutils, if you make him happy, I'll probably be happy, too ;-) Ah, thanks, that helps put things in

Re: Feature request - base64 Filename Safe Alphabet

2008-04-30 Thread Bo Borgerson
should have good performance in both cases, as well. Please let me know if this meets your requirements. Thanks, Bo From a302f7beca7d0e2bfcb7770ff31947e3d2965db2 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 30 Apr 2008 17:40:38 -0400 Subject: [PATCH] An upstream compatible

Re: Feature request - base64 Filename Safe Alphabet

2008-04-29 Thread Bo Borgerson
Christopher Kerr wrote: After being burned by using `head -c6 /dev/urandom | base64` as part of a directory name, I realised that it would be useful if base64 had an option to generate URL and Filename safe encodings, as specified in RFC 3548 section 4. This would make cat FILE | base64

Re: Feature request - base64 Filename Safe Alphabet

2008-04-29 Thread Bo Borgerson
Pádraig Brady wrote: Perhaps `tr '+/' '._'` would be better so that you don't need to worry about - at the start of a filename? I'm think `.' at the beginning of a filename also has the potential to give users unexpected behavior. Bo ___

Re: Feature request - base64 Filename Safe Alphabet

2008-04-29 Thread Bo Borgerson
Pádraig Brady wrote: tr '+/' '._' = hidden files tr '+/' '-_' = awkward option clashes tr '/' '_' = not POSIX portable ho hum, the awkward option clashes is probably best. Yeah, there's no really ideal option, is there... It almost might be nice to have a totally user-configurable

Re: Feature request - base64 Filename Safe Alphabet

2008-04-29 Thread Bo Borgerson
Gabriel Barazer wrote: AFAIK, POSIX filenames allow any character except the slash character and the null byte. Especially when this is the RFC recommanded translation. This would avoid confusing people with multiple translation sets and stick to the RFC (considered by many as the

Re: coreutils test coverage

2008-04-29 Thread Bo Borgerson
Jim Meyering wrote: If you're reading this list, you probably noticed that some kind souls at Stanford uncovered a surprising number of bugs in coreutils recently. Part of their analysis was coverage-related, and they produced these coverage reports:

Re: FYI, 11 mostly-test-related patches

2008-04-28 Thread Bo Borgerson
___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils From dd8e78633f60a4a266b870326ac87d9844dab02b Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Mon, 28 Apr 2008 10:30:22 -0400 Subject: [PATCH] Only cleanup

Re: FYI, 11 mostly-test-related patches

2008-04-28 Thread Bo Borgerson
Jim Meyering wrote: Bo Borgerson [EMAIL PROTECTED] wrote: I think File::Temp does this internally as well, but it looks like chmod_tree will just warn about the failed chdir and procede to recursively chmod whatever directory it was in at the time if $dir is yanked out from under it. Yes

Re: FYI, 11 mostly-test-related patches

2008-04-28 Thread Bo Borgerson
07cf6a226f75aa9e91060d657fdfebff6e74a618 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Mon, 28 Apr 2008 14:58:51 -0400 Subject: [PATCH] Remove references to tests/wc from bootstrap * bootstrap: Don't try to initialize anything in tests/wc. Signed-off-by: Bo Borgerson [EMAIL PROTECTED] --- bootstrap

[PATCH] Add new program: renice

2008-04-27 Thread Bo Borgerson
'. Unlike both of the above this version also provides long forms for each option. Please see attached or fetch at: $ git fetch git://repo.or.cz/coreutils/bo.git renice:renice Thanks, Bo From c4717c8b0add43a67a8ff052b458306230259e11 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED

Re: [PATCH] Add new program: renice

2008-04-27 Thread Bo Borgerson
On Sun, Apr 27, 2008 at 5:16 PM, Jim Meyering [EMAIL PROTECTED] wrote: Thanks, but Bob Proulx has written one already, as hinted at in TODO. I think it's nearly ready... Ah, so I see. Oh well, it was a fun weekend project. :) Thanks, Bo

[PATCH] Add new program: psub

2008-04-27 Thread Bo Borgerson
: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 27 Apr 2008 20:11:39 -0400 Subject: [PATCH] Add new program: psub * src/psub.c: New program to manage temporary fifos for pipelines. * man/psub.x: Manfile template for new program. * src/Makefile.am: List new program. Signed-off-by: Bo Borgerson [EMAIL

Re: [PATCH] Use a hash rather than a linked-list for cycle check in cp

2008-04-22 Thread Bo Borgerson
On Tue, Apr 22, 2008 at 3:03 PM, Jim Meyering [EMAIL PROTECTED] wrote: Hi Bo, Thanks for that patch. However, let's see if Cai Xianchao and Li Zefan are still working on rewriting cp to use openat-style functions. http://thread.gmane.org/gmane.comp.gnu.coreutils.bugs/12041

[PATCH] Improve memory management in join

2008-04-22 Thread Bo Borgerson
== malloc/free: 1,405 allocs, 1,405 frees, 65,858 bytes allocated. $ time src/join.master ja jb user0m27.126s $ time src/join ja jb user0m17.297s Thanks, Bo From 623a2f43593093b3fb8cde9472bf5ecec652b6d3 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Tue, 22 Apr 2008 16

Re: [PATCH] Make comm check order of input files

2008-04-21 Thread Bo Borgerson
Hi, The previous version did not warn if the final record in a file was out of order and `--check-order' was not in effect. Thanks, Bo From dc34eed9e6ee34f473a8d74b98bccaf082fe79c2 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 20 Apr 2008 21:24:16 -0400 Subject

Re: [PATCH] Make comm check order of input files

2008-04-21 Thread Bo Borgerson
Hi, Pádraig pointed out that there's no reason to copy data around here. This version avoids the copies. Thanks Pádraig, Bo From 49ec3883efc8a89e8a4260f25bb50178aced1be4 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 20 Apr 2008 21:24:16 -0400 Subject: [PATCH] Make

Re: [Coreutils-announce] coreutils-6.11 released

2008-04-20 Thread Bo Borgerson
On Sun, Apr 20, 2008 at 5:24 PM, Jim Meyering [EMAIL PROTECTED] wrote: [EMAIL PROTECTED] (Karl Berry) wrote: join now verifies that the inputs are in sorted order. This check can How about doing the same for comm? Makes sense. Did you just volunteer? ;-) If not, I'll be

[PATCH] Make comm check order of input files

2008-04-20 Thread Bo Borgerson
On Sun, Apr 20, 2008 at 8:35 PM, Karl Berry [EMAIL PROTECTED] wrote: If not, I'll be happy to do it. Please! Here's a patch. Bo From 1a651ab6aedea0d0cc383f2e60c82fe7f0d395f0 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 20 Apr 2008 21:24:16 -0400 Subject

[PATCH] Use a hash rather than a linked-list for cycle check in cp

2008-04-16 Thread Bo Borgerson
user0m0.624s-- with patch sys 0m48.639s Thanks, Bo From 224328e4bda44aa25cd5c98b1c13751ecea865c7 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Wed, 16 Apr 2008 13:44:36 -0400 Subject: [PATCH] Use a hash rather than a linked-list for cycle check in cp * NEWS

Re: [PATCH] Add new program: zargs

2008-04-15 Thread Bo Borgerson
Hi, I noticed that the name of the --files0-from option description macro in doc/coreutils.texi changed, so I updated the call in this patch. I also replaced the polling reaper with a SIGCHLD sigaction handler. Thanks, Bo On Fri, Apr 11, 2008 at 11:24 AM, Bo Borgerson [EMAIL PROTECTED] wrote

Re: [PATCH] Add new program: zargs

2008-04-15 Thread Bo Borgerson
On Tue, Apr 15, 2008 at 9:01 AM, Pádraig Brady [EMAIL PROTECTED] wrote: How does this compare with `zrun` from moreutils? Would it be more appropriate to merge zargs into moreutils? Hi Pádraig, Yes, it looks like zrun attempts to perform a similar task. From a quick peek I notice a few

[PATCH] Add new program: zargs

2008-04-11 Thread Bo Borgerson
/zargsE0oy5L/TODO.gz 177 9766908 input/TODO.txt 5312928 20724 total Thanks, Bo From 42e176d01982f038f12859880c530d3a49a5f4ac Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 6 Apr 2008 17:54:08 -0400 Subject: [PATCH] Add new program: zargs * AUTHORS

Re: [PATCH] Add new program: magic

2008-04-10 Thread Bo Borgerson
On Wed, Apr 9, 2008 at 9:48 PM, Bob Proulx [EMAIL PROTECTED] wrote: I like this sort of general purpose utility that can work with a broad set of things much better than hacks to every utility. It is definitely a better direction. But I don't like the name. The name is too generic and

[PATCH] Add new program: magic

2008-04-09 Thread Bo Borgerson
0a16d0698590f137e04c8351f8f14383147e827f Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 6 Apr 2008 17:54:08 -0400 Subject: [PATCH] Add new program: magic * AUTHORS: Register as the author. * NEWS: Advertise new program. * README: List new program. * doc/coreutils.texi: Describe new program. * man

[PATCH] Add support for --output-delimiter=STR to comm

2008-04-07 Thread Bo Borgerson
. Thanks, Bo From 4edd3361a500d31b5bc2b645e93c4aef02c00cab Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Fri, 4 Apr 2008 20:40:58 -0400 Subject: [PATCH] Add support for --output-delimiter=STR to comm * src/comm.c: (static char *delimiter) Points to the delimiter string

Re: Bug#474436: coreutils: ls --time-style=locale no longer works

2008-04-06 Thread Bo Borgerson
d239d2fffcee499da5f3c5b4ddacc9caa258f3b0 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Sun, 6 Apr 2008 11:47:28 -0400 Subject: [PATCH] Use default English time-formats for `en_*' locales src/ls.c: (decode_switches) only goto case_long_iso_time_style from `locale' time-format setting

Re: Bug#474436: coreutils: ls --time-style=locale no longer works

2008-04-06 Thread Bo Borgerson
On Sun, Apr 6, 2008 at 12:35 PM, Jim Meyering [EMAIL PROTECTED] wrote: Thanks. That feels pretty kludgy. I hope we end up with something cleaner. Yeah, I suppose so. Short of including `translations' for English, though, what's a better option? BTW, such a patch would almost certainly

Re: Bug#474436: coreutils: ls --time-style=locale no longer works

2008-04-06 Thread Bo Borgerson
On Sun, Apr 6, 2008 at 1:25 PM, Michael Stone [EMAIL PROTECTED] wrote: Yeah, I suppose so. Short of including `translations' for English, though, what's a better option? What's the downside to that? Mike Stone Good question. My thought was because there aren't any now, but I guess

Re: [PATCH] Standardize some error messages.

2008-04-06 Thread Bo Borgerson
/ ;-) Yikes. :) Bo From fda400023db314046b6792b1b51c242b2bb62996 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Fri, 4 Apr 2008 11:13:13 -0400 Subject: [PATCH] Standardize some error messages. * maint.mk: (sc_error_message_warn_fatal, sc_error_message_uppercase

Re: [PATCH] add new sort option --xargs (-x)

2008-04-06 Thread Bo Borgerson
I had a capitalized error message in this patch. I also didn't use a correct commit message format. Thanks Bo From 9a37b547bcc892d1d5e2542c43d77b13497318db Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Thu, 3 Apr 2008 18:42:57 -0400 Subject: [PATCH] Add new sort option

Re: [PATCH] add new sort option --xargs (-x)

2008-04-06 Thread Bo Borgerson
. Thanks, Bo From 404e23daf6874e4d36e2048de569bcac057b7400 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Thu, 3 Apr 2008 18:42:57 -0400 Subject: [PATCH] Add new sort option --files0-from=F * src/sort.c: Support new option. * tests/misc/sort-files0-from: Test new option. * tests

Re: [PATCH] Standardize some error messages.

2008-04-04 Thread Bo Borgerson
*** rule rather than to the code it complained about. Bo From 41f5f14c10a47335fd1b19130201a3a57202e8c9 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Fri, 4 Apr 2008 11:13:13 -0400 Subject: [PATCH] Standardize some error messages. maint.mk: Add automatic checks for non-standard error

Re: [PATCH] Standardize some error messages.

2008-04-04 Thread Bo Borgerson
On Fri, Apr 4, 2008 at 12:38 PM, Jim Meyering [EMAIL PROTECTED] wrote: Please remove the #FIXME comment, now that you've fixed it ;-) Ha, tunnel vision :) Thanks, Bo ___ Bug-coreutils mailing list Bug-coreutils@gnu.org

[PATCH] add new sort option --xargs (-x)

2008-04-03 Thread Bo Borgerson
8568528acd4b5eea20d06136aaaf7b18a36f03c0 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Thu, 3 Apr 2008 12:05:55 -0400 Subject: [PATCH] add new sort option --xargs (-x) * src/sort.c: if --xargs option, add input to FILES * tests/misc/sort-xargs: test new option * tests/misc

Re: [PATCH] add new sort option --xargs (-x)

2008-04-03 Thread Bo Borgerson
On Thu, Apr 3, 2008 at 12:18 PM, Jim Meyering [EMAIL PROTECTED] wrote: I suppose you have a real application where this is useful? If so, please describe it -- motivation/justification helps ;-) Just a merge with a lot of source files. It's the same motivation as the nmerge patch. I've

Re: [PATCH] add new sort option --xargs (-x)

2008-04-03 Thread Bo Borgerson
On Thu, Apr 3, 2008 at 2:05 PM, Jim Meyering [EMAIL PROTECTED] wrote: Sounds interesting. I suppose it can work with an arbitrary decompressor? Note this relatively new option: --compress-program=PROG compress temporaries with PROG; decompress

Re: [PATCH] add new sort option --xargs (-x)

2008-04-03 Thread Bo Borgerson
Okay, here's a version that supports argument input in --files0-from=F style. Bo From 3108b79cbbb5d6c2fe3c2f8d5037f166cb0f1ca6 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Thu, 3 Apr 2008 18:42:57 -0400 Subject: [PATCH] Add new sort option --files0-from=F src/sort.c

Re: [PATCH] Add timeout utility

2008-04-02 Thread Bo Borgerson
Pádraig Brady [EMAIL PROTECTED] wrote: Subject: [PATCH] Add new program: timeout Great idea for a tool! Have you considered an alternate run-mode where it could operate as a filter and timeout on 'inactivity' of the pipeline? If, for instance, I have a pipeline that processes a lot of data and

Re: [PATCH] Add timeout utility

2008-04-02 Thread Bo Borgerson
On Wed, Apr 2, 2008 at 10:20 AM, Pádraig Brady [EMAIL PROTECTED] wrote: It will always go through though as the kernel will buffer it. Yes, that introduces some fuzz, but I think the principle remains viable -- the kernel will only buffer so much. Consider the following using a timeout.c

Re: Modifiable NMERGE in sort

2008-04-01 Thread Bo Borgerson
there previously hadn't been any, so I replaced those with spaces. Bo From d1c257dc8c0bd6892ef252e153b72f273879c267 Mon Sep 17 00:00:00 2001 From: Bo Borgerson [EMAIL PROTECTED] Date: Mon, 31 Mar 2008 16:58:21 -0400 Subject: [PATCH] sort: added --batch-size=NMERGE option. * src/sort.c: Replace

Re: Modifiable NMERGE in sort

2008-03-31 Thread Bo Borgerson
On Mon, Mar 31, 2008 at 2:29 AM, Paul Eggert [EMAIL PROTECTED] wrote: Alas, that patch assumes C99, and we can't assume that quite yet. Also, it mishandles nmerge values that are too large (you'll get core dumps or worse on many hosts). That being said, it might be worth adding an

Re: Modifiable NMERGE in sort

2008-03-31 Thread Bo Borgerson
On Mon, Mar 31, 2008 at 11:05 AM, Pádraig Brady [EMAIL PROTECTED] wrote: Jim Meyering wrote: One more suggestion ;-) Add tests you beat me to it :) Also I would mention that to ammend a patch do: edit your files git commit --amend -e -a git format-patch --stdout

Modifiable NMERGE in sort

2008-03-29 Thread Bo Borgerson
Hi, I've found at times that it's useful to merge more than NMERGE (currently 16) inputs without using temp files. I've modified my local version of sort to allow this number to be overridden on the command-line (please see enclosed patch). Is this an option that might be worth including in a