Re: uniq with sort-like "--key" support

2013-02-13 Thread Pádraig Brady
On 02/13/2013 04:45 PM, Assaf Gordon wrote: On 02/12/2013 01:31 AM, Assaf Gordon wrote: I'd like to offer a proof-of-concept patch for adding sort-like "--key" support for the 'uniq' program, as discussed here: http://lists.gnu.org/archive/html/bug-coreutils/2006-06/msg00211.html and in se

Re: uniq with sort-like "--key" support

2013-02-13 Thread Pádraig Brady
On 02/13/2013 05:34 PM, Assaf Gordon wrote: Hello Jim, Jim Meyering wrote, On 02/13/2013 12:05 PM: Assaf Gordon wrote: Assaf Gordon wrote, On 02/13/2013 11:45 AM: ... One more update with two changes: ... src/uniq_sort_common.h | 1096 H

Re: new snapshot available: coreutils-8.20.119-54cdb0

2013-02-13 Thread Pádraig Brady
On 02/11/2013 11:55 AM, Pádraig Brady wrote: This is another snapshot mainly to roll up translations. We plan to release coreutils 8.21 this Thursday. coreutils snapshot: http://pixelbeat.org/cu/coreutils-ss.tar.xz 5.1 MB http://pixelbeat.org/cu/coreutils-ss.tar.xz.sig http

coreutils-8.21 released [stable]

2013-02-14 Thread Pádraig Brady
): Mike Frysinger (2): Ondrej Oprala (2): Ondřej Vašík (1): Paul Eggert (12): Pádraig Brady (46): Stefano Lattarini (1): Stephan Krempel (1): Zartaj Majeed (1): Ángel González (1): Pádraig [on behalf of the coreutils maintainers

Re: [PATCH] join: Add -z option

2013-02-14 Thread Pádraig Brady
On 02/14/2013 08:51 PM, Assaf Gordon wrote: Hello, This patch add "-z" to join, supporting joining zero-terminated lines. The patch is heavily based on James Youngman's patch of adding -z to uniq (commit e062524). -gordon P.S. This patch is independent of the key-comparison patches discussed

Re: [PATCH 1/2] maint: don't reset PATH in out commit-msg git hook

2013-02-15 Thread Pádraig Brady
On 02/15/2013 01:39 PM, Stefano Lattarini wrote: I have a custom 'editor' script in ~/bin, and a system-provided 'editor' program in /usr/bin (on Debian, this is a link set up the "debian alternatives" subsystem). My '$EDITOR' and '$GIT_EDITOR' variables are set simply to 'editor' (no absolute p

Re: [PATCH 2/2] maint: choose editor for our commit-msg git hook the same way git does

2013-02-15 Thread Pádraig Brady
looks good. thanks, Pádraig.

Re: [PATCH 2/2] maint: choose editor for our commit-msg git hook the same way git does

2013-02-16 Thread Pádraig Brady
On 02/16/2013 10:31 AM, Bernhard Voelker wrote: Hi Stefano, On 02/16/2013 07:16 AM, Stefano Lattarini wrote: Hi Bernhard. On 02/15/2013 04:54 PM, Bernhard Voelker wrote: Strange, when I provoke a 2nd re_edit() by giving a bad commit message, then "Vim" is used as $editor instead my $GIT_EDITO

Re: 8.21.2-c53c - failure on rm/deep-2 and du/long-from-unreadable

2013-02-18 Thread Pádraig Brady
On 02/18/2013 06:14 PM, C de-Avillez wrote: Both tests failed at the same point: foreach my $i (1..52)' -e ' { mkdir ($d, 0700) && chdir $d or die "$!" }' File name too long at -e line 2. I do not remember seeing this error anytime recently (but I have been lax on running the tests, sorry). I

Re: 8.21.2-c53c - failure on rm/deep-2 and du/long-from-unreadable

2013-02-18 Thread Pádraig Brady
On 02/18/2013 09:25 PM, C de-Avillez wrote: On Mon, 18 Feb 2013 14:55:24 -0600 C de-Avillez wrote: On Mon, 18 Feb 2013 19:38:20 + Pádraig Brady wrote: On 02/18/2013 06:14 PM, C de-Avillez wrote: Both tests failed at the same point: foreach my $i (1..52)' -e ' { mkdir

Re: [Patch] expand,unexpand multibyte support

2013-02-18 Thread Pádraig Brady
On 02/18/2013 03:30 PM, Ondrej Oprala wrote: Hi, I've been working on multibyte support for the {un,}expand utilities lately, my approach being similar to Padraig's from 2010 ( http://lists.gnu.org/archive/html/coreutils/2010-09/msg00029.html ) . Both tools now read by lines, not bytes, and th

Re: 8.21.2-c53c - failure on rm/deep-2 and du/long-from-unreadable

2013-02-19 Thread Pádraig Brady
On 02/18/2013 11:27 PM, C de-Avillez wrote: For the ecryptfs FS: $ stat -f -c %l . 143 python -c "import os; print os.pathconf('.','PC_PATH_MAX')" 4096 For a standard ext4: $ stat -f -c %l . 255 $ python -c "import os; print os.pathconf('.','PC_PATH_MAX')" 4096 So ecrypts limits NAME_MAX. I

Re: touch --create?

2013-02-19 Thread Pádraig Brady
On 02/19/2013 01:39 PM, Bernhard Voelker wrote: Sometimes, I want to create a file and want it to succeed only if the file wasn't previously there. What about adding a complementary option to touch --no-create which could be used like this? $ touch --create file \ || { echo "creating fi

Re: 8.21.2-c53c - failure on rm/deep-2 and du/long-from-unreadable

2013-02-19 Thread Pádraig Brady
On 02/19/2013 01:06 PM, Bernhard Voelker wrote: On 02/19/2013 01:10 PM, Pádraig Brady wrote: On 02/18/2013 11:27 PM, C de-Avillez wrote: So ecrypts limits NAME_MAX. Indeed, running the perl snippet with $dir set to 142 works. OK I'll take the simple approach and skip those tests in

Re: coreutils-8.21.chmod_speedup.patch

2013-02-19 Thread Pádraig Brady
On 02/19/2013 08:57 PM, Alexey Potehin wrote: Hello people! This is path for chmod speedup (chmod only if newmode != oldmode), see file in attach. This version adding command key '--noatime' for this. Apply this please) --noctime you mean. The previous thread about this was at: http://en.

Re: [PATCH]: uniq: add "--group" option

2013-02-20 Thread Pádraig Brady
On 02/20/2013 06:44 PM, Assaf Gordon wrote: Hello, Attached is a suggestion for "--group" option in uniq, as discussed here: http://lists.gnu.org/archive/html/coreutils/2011-03/msg0.html http://lists.gnu.org/archive/html/coreutils/2012-03/msg00052.html The patch adds two parameters:

Re: [PATCH]: uniq: add "--group" option

2013-02-21 Thread Pádraig Brady
On 02/21/2013 03:42 PM, Assaf Gordon wrote: Hello Pádraig, Pádraig Brady wrote, On 02/20/2013 08:47 PM: On 02/20/2013 06:44 PM, Assaf Gordon wrote: Hello, Attached is a suggestion for "--group" option in uniq, as discussed here: http://lists.gnu.org/archive/html/coreuti

Re: [patch] install with a missing strip tool

2013-02-22 Thread Pádraig Brady
On 02/22/2013 01:18 PM, Bernhard Voelker wrote: On 02/21/2013 08:40 PM, Ondrej Oprala wrote: so let's do your approach ;) Hi Ondrej, thank you. In your patch, you missed the case when fork failed: the target would yet be left behind. Therefore, and to have slightly smaller code, I inverted

Re: Merging (-m/--merge) slower than sorting irrespective of --parallel option

2013-02-22 Thread Pádraig Brady
On 02/22/2013 04:55 PM, Vlad ROMASCANU wrote: Anyone have any idea why these non-intuitive results? Why does "sort -m" appear so slow even with --parallel=1? Should this go to bug-coreut...@gnu.org? You got the right list for a general query like this. I don't know either, why you're getting

Re: ls and root directory indicator

2013-02-24 Thread Pádraig Brady
On 02/24/2013 05:39 PM, Sami Kerola wrote: > Hello, > > When using file type indicator to a root, e.g., > > $ ls -Fd / > // > > two slashes are printed. To me that feels wrong. Patch below is an > attempt to make root print out to be a single slash. Perhaps there is > more elegant way to do the

Re: coreutils FAQ link in manpages and/or --help output

2013-02-25 Thread Pádraig Brady
On 02/25/2013 02:53 PM, Ondrej Oprala wrote: > Hi, > to reduce the amount of questions about date, sort and anything > multibyte-related, I think > it'd be a good idea to add a link to the coreutils FAQ to the man pages > (and/or --help output), maybe something like > "Report TOOL bugs to bug-cor

Re: coreutils FAQ link in manpages and/or --help output

2013-02-25 Thread Pádraig Brady
On 02/25/2013 11:23 PM, Assaf Gordon wrote: > Philipp Thomas wrote, On 02/25/2013 01:18 PM: >> Am 25.02.13 19:07, schrieb Pádraig Brady: >> >>> $ truncate --help | tail -n4 >>> GNU coreutils online help: <http://www.gnu.org/software/coreutils/#help> &g

Re: coreutils FAQ link in manpages and/or --help output

2013-02-25 Thread Pádraig Brady
On 02/25/2013 02:53 PM, Ondrej Oprala wrote: > Hi, > to reduce the amount of questions about date, sort and anything > multibyte-related, I think > it'd be a good idea to add a link to the coreutils FAQ to the man pages > (and/or --help output), maybe something like > "Report TOOL bugs to bug-cor

Re: [PATCH]: uniq: add "--group" option

2013-02-27 Thread Pádraig Brady
On 02/21/2013 07:40 PM, Assaf Gordon wrote: > Assaf Gordon wrote, On 02/21/2013 11:37 AM: >> >> You were planning on "--group" to mean explicitly "output all input lines, >> and add group-markers for unique groups" (meaning -u/-d/-D and --group are >> mutually exclusive). >> > > Attached is a ve

Re: [PATCH] dircolors.hin: add .cab, .alz, .lzo, .lrz, .t7z, .tzo and .lha to colorized archives

2013-02-27 Thread Pádraig Brady
On 02/20/2013 03:05 PM, Ondrej Vasik wrote: > Hello, > as reported in https://bugzilla.redhat.com/868510 by Ville > Skyttä, .cab, .lzo, .lrz, and .tzo extensions are missing in the > colorized archives. Based on the atool manpage, I added few more missing > to the list. Attached patch adds the exte

Re: [PATCH] join: Add -z option

2013-02-27 Thread Pádraig Brady
On 02/14/2013 11:15 PM, Pádraig Brady wrote: > On 02/14/2013 08:51 PM, Assaf Gordon wrote: >> Hello, >> >> This patch add "-z" to join, supporting joining zero-terminated lines. >> The patch is heavily based on James Youngman's patch of adding -z t

Re: coreutils FAQ link in manpages and/or --help output

2013-02-27 Thread Pádraig Brady
On 02/25/2013 11:23 PM, Assaf Gordon wrote: > Philipp Thomas wrote, On 02/25/2013 01:18 PM: >> Am 25.02.13 19:07, schrieb Pádraig Brady: >> >>> $ truncate --help | tail -n4 >>> GNU coreutils online help: <http://www.gnu.org/software/coreutils/#help> &g

Re: [PATCH]: uniq: add tests for "--ignore-case"

2013-02-27 Thread Pádraig Brady
On 02/12/2013 03:44 PM, Assaf Gordon wrote: > Hello, > > Attached are three small tests for uniq with "--ignore-case" (they pass, the > option was simply not tested before). pushed. > > Also, > I noticed that by running the default test suite ("make check SUBDIRS=."), > the majority of uniq t

Re: csplit - split by content of field

2013-02-27 Thread Pádraig Brady
On 02/07/2013 04:39 PM, Pádraig Brady wrote: > On 02/07/2013 03:02 PM, Assaf Gordon wrote: >> Pádraig Brady wrote, On 02/06/2013 05:38 PM: >>> On 02/06/2013 10:09 PM, Assaf Gordon wrote: >>>> >>>> Attach is a patch that gives 'csplit' th

Re: coreutils FAQ link in manpages and/or --help output

2013-02-28 Thread Pádraig Brady
On 02/28/2013 08:40 AM, Ondrej Vasik wrote: > On Thu, 2013-02-28 at 09:26 +0100, Bernhard Voelker wrote: >>> On February 28, 2013 at 4:23 AM Pádraig Brady wrote: >>> I've adjusted the above to only reference online resources, >>> and ensure the links are at the

Re: [PATCH]: uniq: add "--group" option

2013-02-28 Thread Pádraig Brady
On 02/28/2013 05:16 PM, Assaf Gordon wrote: > Pádraig Brady wrote, On 02/27/2013 08:16 PM: >> On 02/21/2013 07:40 PM, Assaf Gordon wrote: >>> Assaf Gordon wrote, On 02/21/2013 11:37 AM: >>>> >>>> You were planning on "--group" to mean explicitly

Re: [PATCH]: uniq: add tests for "--ignore-case"

2013-02-28 Thread Pádraig Brady
On 02/28/2013 07:16 PM, Assaf Gordon wrote: > Attached a patch to not-skip all uniq tests if french locale is missing. Pushed. thanks, Pádraig.

Re: coreutils FAQ link in manpages and/or --help output

2013-03-01 Thread Pádraig Brady
On 03/01/2013 03:34 PM, Assaf Gordon wrote: > Pádraig Brady wrote, On 02/28/2013 08:12 AM: >> On 02/28/2013 08:40 AM, Ondrej Vasik wrote: >>> On Thu, 2013-02-28 at 09:26 +0100, Bernhard Voelker wrote: >>>>> On February 28, 2013 at 4:23 AM Pádraig Brady wrote: >

Re: That '[' file in /usr/bin

2013-03-03 Thread Pádraig Brady
On 03/03/2013 04:18 AM, Mike Frysinger wrote: > On Saturday 23 February 2013 15:56:20 Eric Blake wrote: >> On 02/23/2013 10:25 AM, Salvador Girbau wrote: >>> Hi all, >>> would it be possible to replace /usr/bin/[ with a symlink to >>> /usr/bin/test ? >> >> Not really. Although [ and test are compi

Re: That '[' file in /usr/bin

2013-03-03 Thread Pádraig Brady
On 03/04/2013 12:01 AM, H. Peter Anvin wrote: > Mike Frysinger wrote: >> On Sunday 03 March 2013 14:03:24 H. Peter Anvin wrote: >>> On 03/03/2013 02:53 AM, Pádraig Brady wrote: >>>>> personally, i find it a bit ironic that a lot of GNU apps >> (including >

Re: potential feature addition to coreutils' sort.c: print at most N lines

2013-03-03 Thread Pádraig Brady
On 03/03/2013 05:32 PM, James Dowdell wrote: > I'm considering writing a patch for sort.c to add a new feature, related to a > stackoverflow inquiry I wrote > (http://stackoverflow.com/questions/14882897/what-standard-commands-can-i-use-to-print-just-the-first-few-lines-of-sorted-out). > > This

[PATCH 1/3] maint: ensure proper backslash quoting in texinfo macros

2013-03-03 Thread Pádraig Brady
* doc/coreutils.texi (ambiguousGroupNote): Ensure '\' is escaped appropriately within the macro. This was verified to generate a single '\' in both "info" and "pdf" outputs. --- doc/coreutils.texi |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/doc/coreutils.texi b/doc/c

[PATCH 2/3] doc: remove a redundant numfmt heading from texinfo

2013-03-03 Thread Pádraig Brady
* doc/coreutils.texi (detailmenu): Remove the redundant numfmt heading. --- doc/coreutils.texi |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/doc/coreutils.texi b/doc/coreutils.texi index bc6ff64..76a299f 100644 --- a/doc/coreutils.texi +++ b/doc/coreutils.texi @@ -243,8

[PATCH 3/3] doc: mention `numfmt` as an alternative to `sort -h`

2013-03-03 Thread Pádraig Brady
* doc/coreutils.texi (sort invocation): Mention that numfmt can achieve the same results with a possibly more accurate sort. --- doc/coreutils.texi |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/doc/coreutils.texi b/doc/coreutils.texi index 76a299f..ef65711 100644 --- a

Re: [PATCH 3/3] doc: mention `numfmt` as an alternative to `sort -h`

2013-03-03 Thread Pádraig Brady
On 03/04/2013 01:18 AM, Jim Meyering wrote: > Good point. > > s/posibly/possibly/, > > but you might prefer to word it like this: > > numbers to human format @emph{after} the sort, thus often allowing > sort to operate on more accurate numbers. > Amended with that. thanks! Pádraig.

[PATCH] build: fix factor build failure on aarch64

2013-03-04 Thread Pádraig Brady
From: Torbjorn Granlund * src/longlong.h (__aarch64__): Make add_ss and sub_ddmmss work. * NEWS: Mention the build fix. Reported at https://bugzilla.redhat.com/917735 --- NEWS |4 src/longlong.h | 21 +++-- 2 files changed, 11 insertions(+), 14 deletions(

Re: potential feature addition to coreutils' sort.c: print at most N lines

2013-03-06 Thread Pádraig Brady
On 03/06/2013 01:26 AM, James Dowdell wrote: > Pádraig, > > Thanks for the pointers. I am slowly making my way through sort.c, > and preparing a patch; I estimate I'll have something ready in a few > days or weeks. > > One final question before I go make this thing, regarding the thread > conver

Re: [PATCH] shuf: use reservoir-sampling when possible

2013-03-06 Thread Pádraig Brady
On 03/06/2013 11:50 PM, Assaf Gordon wrote: > Hello, > > Attached is a suggestion to implement reservoir-sampling in shuf: > When the expected output of lines is known, it will not load the entire file > into memory - allowing shuffling very large inputs. > > I've seen this mentioned once: > ht

Re: [Patch] expand,unexpand multibyte support

2013-03-06 Thread Pádraig Brady
On 02/20/2013 05:59 PM, Ondrej Oprala wrote: > On 02/20/2013 12:21 AM, Bernhard Voelker wrote: >> On 02/18/2013 04:30 PM, Ondrej Oprala wrote: >>> Hi, I've been working on multibyte support for the {un,}expand utilities >>> lately, my approach being similar to Padraig's from 2010 ( >>> http://lists

Re: [Patch] expand,unexpand multibyte support

2013-03-07 Thread Pádraig Brady
On 03/07/2013 02:23 PM, Ondrej Oprala wrote: > should I integrate the changes into the patch and re-send it or rather not > for now? Please hold off for now. thanks, Pádraig.

Re: [PATCH] shuf: use reservoir-sampling when possible

2013-03-07 Thread Pádraig Brady
On 03/07/2013 07:32 PM, Assaf Gordon wrote: > Hello, > > Attached is an updated version. > > Pádraig Brady wrote, On 03/06/2013 08:24 PM: >> On 03/06/2013 11:50 PM, Assaf Gordon wrote: >>> Attached is a suggestion to implement reservoir-sampling in shuf: >>&g

Re: date opposite case flag

2013-03-07 Thread Pádraig Brady
On 03/07/2013 03:59 PM, Ondrej Oprala wrote: > date invoked with # as one of the flags following % often gives very > surprising results > (apart from +%#Z, which seems to be working fine). > This has also been mentioned here: > https://bugzilla.redhat.com/show_bug.cgi?id=749704 . > Though JW rai

Re: Multithreaded sort hangs on Solaris

2013-03-12 Thread Pádraig Brady
On 03/11/2013 03:47 PM, McFarland, Jeffrey wrote: > I have come across some odd results regarding the sort utility in coreutils > version 8.20. I’ve looked through the archives and don’t see any similar > issues so it may be something specific to our systems. > > > > System: SunOS 5.10 Gene

Re: Multithreaded sort hangs on Solaris

2013-03-13 Thread Pádraig Brady
On 03/13/2013 02:18 PM, McFarland, Jeffrey wrote: > Here are the values from another sort that has been running for over 12 hours > now. This time that second argument (number of threads) looks fine in all > three cases. And this time there are no zombie threads. > >> : pstack 20632 > 20632:

Re: Octal file permissions in ls

2013-03-18 Thread Pádraig Brady
On 03/18/2013 04:43 PM, Sakse Dalum wrote: > Hi there, > > I recently found myself wanting ls to show file permissions in octal format, > so > I decided to implement the feature. Being somewhat unfamiliar with C coding > and > C coding standards in particular, I've hunched my way through the >

[PATCH] maint: improve make src/fs-magic-compare

2013-03-22 Thread Pádraig Brady
* README-release: fix the `make` command, and mention how to get the latest results without requring running a system with the latest kernel. * src/local.mk (src/fs-latest-magic.h): A new target to document how/where to place the latest magic header. (src/fs-kernel-magic): Adjust to include separat

Re: [PATCH] cp: do not create empty dst file if failed to make reflink

2013-03-22 Thread Pádraig Brady
On 03/11/2013 07:05 PM, Guangyu Sun wrote: > If making reflink across devices or if the filesystem does not support > reflink, we want it to return an error and do nothing else. However, > now it will create a new empty file to the dst. Fix it. > > test case in an ext4 filesystem: > $ ls > foo > $

Re: [PATCH] shuf: use reservoir-sampling when possible

2013-03-24 Thread Pádraig Brady
On 03/11/2013 09:10 PM, Assaf Gordon wrote: > Hello, > > Pádraig Brady wrote, On 03/07/2013 06:26 PM: >> On 03/07/2013 07:32 PM, Assaf Gordon wrote: >>> Pádraig Brady wrote, On 03/06/2013 08:24 PM: >>>> On 03/06/2013 11:50 PM, Assaf Gordon wrote: >>

Re: [PATCH] shuf: use reservoir-sampling when possible

2013-03-25 Thread Pádraig Brady
On 03/25/2013 04:30 PM, Assaf Gordon wrote: > Hello Pádraig, > > Pádraig Brady wrote, On 03/24/2013 11:45 PM: >>>>>> On 03/06/2013 11:50 PM, Assaf Gordon wrote: >>>>>>> Attached is a suggestion to implement reservoir-sampling in shuf: >>>

[PATCH] tail: exit without reading input if would never output

2013-03-25 Thread Pádraig Brady
* src/tail.c (main): If -n0 or -c0 were specified without -f, then no data would ever be output, so exit without reading input. * tests/tail-2/tail-n0f.sh: Augment the related test with this case. --- src/tail.c |4 tests/tail-2/tail-n0f.sh |5 + 2 files changed, 9 i

shuf: exit without reading if would never output

2013-03-27 Thread Pádraig Brady
Similar to the tail -n0 patch recently sent to the list, this adjusts shuf so that it never reads input with -n0. Now unlike tail -n0 which is useful with -f, shuf -n0 isn't needed, as it always truncates --output if specified, and doesn't produce anything. I don't think we should disallow -n0 tho

[PATCH] shuf: exit without reading if would never output

2013-03-27 Thread Pádraig Brady
* src/shuf.c (main): If -n0 specified then no data would ever be output, so exit without reading input. * tests/misc/shuf.sh: Augment the related test with this case. --- src/shuf.c |6 -- tests/misc/shuf.sh |6 ++ 2 files changed, 10 insertions(+), 2 deletions(-) diff --g

Re: [PATCH] csplit: new option --suppress-matched

2013-03-29 Thread Pádraig Brady
On 03/28/2013 10:10 PM, Assaf Gordon wrote: > Hello, > > > Assaf Gordon wrote, On 03/07/2013 05:39 PM: >> >> Attached is a new option for csplit, suppress-matched, as been mentioned few >> times before (e.g. >> http://lists.gnu.org/archive/html/coreutils/2013-02/msg00170.html ). >> > > Attache

Re: tail --retry not re-attempting to open file

2013-04-03 Thread Pádraig Brady
On 04/03/2013 09:42 AM, Bernhard Voelker wrote: > re-adding the list. > > On 04/03/2013 05:12 AM, Noel Morrison wrote: >> Sorry to inform you but in the standard tail command even if a file is >> not present, the tail -f --retry would retry at some interval to attach >> to the filename . If the fi

Re: tail --retry not re-attempting to open file

2013-04-03 Thread Pádraig Brady
On 04/03/2013 11:46 PM, Bernhard Voelker wrote: > Hi Padraig, > > On 04/03/2013 01:02 PM, Pádraig Brady wrote: >> Yes I agree that this is a regression associated with the inotify support. >> I take `tail --follow=descriptor --retry` to mean, >> Wait for the file to

Re: Absolute symlink created relatively to the current directory

2013-04-04 Thread Pádraig Brady
On 04/04/2013 10:54 AM, Rémy Lefevre wrote: > Hello, > > the common way to create a symlink is to run the following command: > > ln -s target_path link_path > > where target_path is the relative or absolute path to which the symbolic link > should point. > > In the case of relative symlink, th

Re: Move Command Feature

2013-04-05 Thread Pádraig Brady
On 04/05/2013 06:23 PM, Michael Boldischar wrote: > Hello, > > This is a suggestion for a new feature in the "mv" command. This feature > applies to moving directories. If a user moves a directory with a lot of > files and encounters an error, it can often leave the source directory in a > pa

Re: [PATCH] csplit: new option --suppress-matched

2013-04-10 Thread Pádraig Brady
On 03/31/2013 02:19 AM, Assaf Gordon wrote: > On 03/30/13 01:08, Pádraig Brady wrote: >> On 03/28/2013 10:10 PM, Assaf Gordon wrote: >>>> Attached is a new option for csplit, suppress-matched, as been mentioned >>>> few times before (e.g. >>>> http:/

Re: [PATCH] delaying dd allocation

2013-04-10 Thread Pádraig Brady
On 01/23/2013 10:48 AM, Ondrej Oprala wrote: > + /* Delay buffer allocation if possible. */ > + if ((skip_records > OFF_T_MAX / input_blocksize) > + || 0 > skip_via_lseek (input_file, STDIN_FILENO, 0, SEEK_CUR)) > +alloc_ibuf (); > > - /* Write a sentinel to the slop after the buffer

Re: How to compile the git mirror of coreutils?

2013-04-12 Thread Pádraig Brady
On 04/12/2013 02:22 PM, Hongxu Chen wrote: > Eric Blake writes: > >> On 04/12/2013 05:19 AM, Hongxu Chen wrote: >>> >>> Hi guys, >>> >>> This question might be a bit silly, but I just cannot build the >>> latest development version of coreutils, which I cloned from >>> >>> git://git.sv.

Re: Make mv work better with SELinux.

2013-04-12 Thread Pádraig Brady
On 04/12/2013 08:25 PM, Daniel J Walsh wrote: > On 01/09/2013 11:02 AM, P£draig Brady wrote: >> On 01/09/2013 03:50 PM, Daniel J Walsh wrote: >>> -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 >>> >>> Any word on this patch making its way upstream? Or is there something >>> else I need to do? > >>

Re: du -s excluding size from directory entries?

2013-04-13 Thread Pádraig Brady
On 04/13/2013 08:22 AM, Aaron Davies wrote: > is there an easy way to `du -s' a tree and ignore the size of the directories > themselves? > > if, e.g., i have one deep directory tree with a long history of > modifications, and another directory tree created from a `cp -a' of the > first, their

Re: tail --retry not re-attempting to open file

2013-04-17 Thread Pádraig Brady
On 04/17/2013 06:39 AM, Bernhard Voelker wrote: > On 04/04/2013 03:42 AM, Pádraig Brady wrote: >> I've nothing against simplicity though since the >> inotify code already deals with missing files when >> following by name, by watching the parent dir, >> it might

Re: tail --retry not re-attempting to open file

2013-04-18 Thread Pádraig Brady
On 04/17/2013 06:39 AM, Bernhard Voelker wrote: > On 04/04/2013 03:42 AM, Pádraig Brady wrote: >> I've nothing against simplicity though since the >> inotify code already deals with missing files when >> following by name, by watching the parent dir, >> it might

Re: tail --retry not re-attempting to open file

2013-04-20 Thread Pádraig Brady
On 04/20/2013 03:47 PM, Bernhard Voelker wrote: > On 04/18/2013 11:52 PM, Bernhard Voelker wrote: >> On 04/18/2013 06:10 PM, Pádraig Brady wrote: >>> You may want to consider using the existing retry_delay_() >>> rather than wait4lines() in the tests, but otherwis

Re: Feature request: fold -p STR ... Prefix each folded line with STR

2013-04-20 Thread Pádraig Brady
unarchive 6195 stop On 04/20/2013 11:50 AM, Jazzoslav Jammovitch wrote: > Greetings, Supaninjas! > > I wish to make a request for a new feature to the fold program. Often, I find > myself folding lines that comprise some kind of list such as: > > == A == > 1. Hola Mundo! > 2. Cows say moo when t

Re: [PATCH] id: show SMACK security context

2013-04-22 Thread Pádraig Brady
On 04/17/2013 09:30 PM, Jarkko Sakkinen wrote: > Enable showing SMACK security context with -Z command-line switch. > Adds dependency to libsmack. > --- > configure.ac | 5 + > src/id.c | 21 + > src/local.mk | 2 +- > 3 files changed, 23 insertions(+), 5 deletions(-)

Re: [PATCH] id: show SMACK security context

2013-04-24 Thread Pádraig Brady
On 04/22/2013 02:09 PM, Jarkko Sakkinen wrote: > Hi > > And thanks for replying! > (and sorry for double reply) > > On Mon, Apr 22, 2013, at 13:15, Pádraig Brady wrote: >> On 04/17/2013 09:30 PM, Jarkko Sakkinen wrote: >>> Enable showing SMACK security co

[PATCH] tests: fix usage of require_ulimit_

2013-04-25 Thread Pádraig Brady
* init.cfg (require_ulimit_v_): Renamed from require_ulimit_ as this only checks for ulimit -v support. Other uses of ulimit -t and ulimit -n in tests shouldn't cause false failures if not supported. * cfg.mk (sc_prohibit_test_ulimit_without_require_): A new syntax check to ensure that require_uli

Re: When was stdbuf introduced?

2013-04-25 Thread Pádraig Brady
On 04/25/2013 07:08 PM, Peng Yu wrote: > Hi, > > I see stdbuf available in coreutils. > > http://www.gnu.org/software/coreutils/manual/html_node/stdbuf-invocation.html > > But I don't find it in my system. Was stdbuf just introduced recently? > Or there is something wrong with my installation? >

[PATCH] doc: correct a URL to older textutils source

2013-04-30 Thread Pádraig Brady
* doc/coreutils.texi (Putting the tools together): Adjust the textutils-1.22 URL, and add a URL for newer sources. --- doc/coreutils.texi |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/coreutils.texi b/doc/coreutils.texi index f6f2eb4..d607eaf 100644 --- a/doc/

Re: Possible vulnerability in csplit

2013-05-06 Thread Pádraig Brady
On 05/06/2013 05:41 AM, Kapil Anand wrote: > Hi, > > I work in information flow analysis of programs and my analysis gave a > possible warning with respect to format string vulnerability in csplit > (Version 8.17). > > In csplit, there is a global variable called "suffix" which is initialized

Re: [PATCH] id: show SMACK security context

2013-05-07 Thread Pádraig Brady
On 05/04/2013 06:44 PM, Jarkko Sakkinen wrote: > Enable showing SMACK security context with -Z command-line switch. > Adds dependency to libsmack. This looks good thanks. SMACK is a little esoteric to warn about by default in ./configure --quiet, so I've removed that warning. This deserves a ment

Re: Add --parents option to mv command in order to create parents directories

2013-05-08 Thread Pádraig Brady
On 04/28/2013 05:55 PM, Rémy Lefevre wrote: > Hello, > > mkdir and cp commands have the --parents option to create parents directories > as needed. But this option is missing in mv. One has to create first the > directory structure using mkdir --parents and then mv the file. This option > is fi

Re: Add --parents option to mv command in order to create parents directories

2013-05-08 Thread Pádraig Brady
On 05/08/2013 01:53 PM, Rémy Lefevre wrote: > 2013/5/8 Pádraig Brady mailto:p...@draigbrady.com>> > > On 04/28/2013 05:55 PM, Rémy Lefevre wrote: > > Hello, > > > > mkdir and cp commands have the --parents option to create parents > direc

Re: Adding "progress bar" feature to "cp" and "mv" commands

2013-05-12 Thread Pádraig Brady
l the "Rejected ideas" sections, from the mailing list summaries to a permanent URL somewhere. That would be useful for both requesters and responders. I've put this in place at: http://www.gnu.org/software/coreutils/rejected_requests.html and linked to it from the appropriate place in REA

Re: [PATCH] ls: do not print directory indicator for root

2013-05-12 Thread Pádraig Brady
On 05/12/2013 09:40 PM, Sami Kerola wrote: > diff --git a/tests/ls/root-slash.sh b/tests/ls/root-slash.sh > new file mode 100755 > index 000..6646322 > --- /dev/null > +++ b/tests/ls/root-slash.sh > +LC_MESSAGES=C ls -pd / > out || fail=1 > +LC_MESSAGES=C ls -Fd / >> out || fail=1 Note tests

Re: Adding "progress bar" feature to "cp" and "mv" commands

2013-05-12 Thread Pádraig Brady
On 05/13/2013 12:38 AM, Bob Proulx wrote: > Pádraig Brady wrote: >> I've put this in place at: >> http://www.gnu.org/software/coreutils/rejected_requests.html > > I like it! In retrospect it should be grouped by command, which I'll do when a get a few mins. thanks, Pádraig.

Re: [PATCH] quotearg: do not read beyond end of buffer

2013-05-13 Thread Pádraig Brady
On 05/13/2013 06:14 AM, Jim Meyering wrote: > I ran gcc's -fsanitize=address against coreutils, and two > sort tests failed due to buffer overruns. Both arose via > a bug in quotearg.c. Patch below. Two things remain to do: > 1) find when the bug was introduced (before push) > 2) address the

Re: coretutils package produces the ownership issue during "mv" command execution.

2013-05-14 Thread Pádraig Brady
On 05/14/2013 10:15 AM, Koteswararao Nelakurthi wrote: > lchown("/media/sda2/aaa", 0, 0) = 0 > write(2, "mv: ", 4mv: ) = 4 > write(2, "failed to preserve ownership for"..., 48failed to preserve > ownership for /media/sda2/aaa) = 48 > write(2, ": Function not implemented

Re: coretutils package produces the ownership issue during "mv" command execution.

2013-05-14 Thread Pádraig Brady
On 05/14/2013 11:14 AM, Koteswararao Nelakurthi wrote: > On Tue, May 14, 2013 at 3:39 PM, Pádraig Brady <mailto:p...@draigbrady.com>> wrote: > > On 05/14/2013 10:15 AM, Koteswararao Nelakurthi wrote: > > lchown("/media/sda2/aaa", 0, 0)

Re: [PATCH] ls: show SMACK label of a file when available

2013-05-14 Thread Pádraig Brady
On 05/13/2013 11:27 AM, Jarkko Sakkinen wrote: > Read security.SMACK64 and display it when '-Z' option is given > to the ls command. > --- > src/local.mk |1 + > src/ls.c | 73 > ++ > 2 files changed, 59 insertions(+), 15 deletions

Re: Add --dereference option to df

2013-05-15 Thread Pádraig Brady
On 05/15/2013 02:55 PM, Ondrej Oprala wrote: > Hi, this patch adds the --dereference option to df, so if a symlink is > specified > as an argument to df (possibly /dev/disk/by-uuid/*), df outputs info about the > filesystem the symlinked file is on instead. Now, I realize the option's name > is >

Re: Add --dereference option to df

2013-05-15 Thread Pádraig Brady
On 05/15/2013 03:32 PM, Ondrej Oprala wrote: > On 05/15/2013 04:19 PM, Pádraig Brady wrote: >> On 05/15/2013 02:55 PM, Ondrej Oprala wrote: >>> Hi, this patch adds the --dereference option to df, so if a symlink is >>> specified >>> as an argument to df (possi

Re: [PATCH] tests: don't assume expr was built with GMP

2013-05-19 Thread Pádraig Brady
On 05/19/2013 12:31 AM, Paul Eggert wrote: > * tests/misc/cut-huge-range.sh (subtract_one): New string. > (CUT_MAX): Don't pass a too-large integer to 'expr'. Ah right. $SSIZE_MAX would have been OK for expr, but not appropriate for this test. Thanks for all the clean ups / improvements. Pádraig

Re: bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-05-26 Thread Pádraig Brady
On 05/26/2013 06:07 PM, Jim Meyering wrote: > Pádraig Brady wrote: > ... >> I expect to push soon, the attached more complete fix to realloc the array. > > That new test would still fail on 32-bit systems, and on any system > with SIZE_MAX < 1E. I expect to push the ad

Re: bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-05-26 Thread Pádraig Brady
unarchive 13530 stop On 05/26/2013 11:07 PM, Jim Meyering wrote: > Jim Meyering wrote: > >> Pádraig Brady wrote: >>> On 05/26/2013 06:07 PM, Jim Meyering wrote: >>>> Pádraig Brady wrote: >>>> ... >>>>> I expect to push soon

Re: [PATCH] tests: avoid spurious failure when SIZE_MAX is 2^32-1

2013-05-26 Thread Pádraig Brady
On 05/27/2013 03:04 AM, Jim Meyering wrote: > FYI, this avoids another 32-bit (small SIZE_MAX, actually) test failure: +1 thanks

Re: [PATCH] tests: fix an ls test not to fail when user or group name contains SP

2013-05-26 Thread Pádraig Brady
On 05/27/2013 03:10 AM, Jim Meyering wrote: > This avoids a test failure when the current user or group name > contains a space. To demonstrate, you can probably just edit > /etc/group and make your primary group name two words with a > space between them. +1 thanks, Pádraig.

Re: bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-05-26 Thread Pádraig Brady
On 05/27/2013 01:29 AM, Jim Meyering wrote: > Pádraig Brady wrote: > >> unarchive 13530 >> stop > > Thanks. > > ... >>> @@ -358,6 +356,14 @@ elide_tail_bytes_pipe (const char *filename, int fd, >>> uintmax_t n

Re: bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-05-28 Thread Pádraig Brady
On 05/28/2013 02:38 AM, Jim Meyering wrote: > Paul Eggert wrote: >> On 05/27/2013 06:04 PM, Jim Meyering wrote: >>> +lim=$(echo $SIZE_MAX | subtract_one_) >>> +lim=$(expr $lim - $max_BUFSIZ) >> >> Sorry, I don't see how this will work either. >> It's common for a GMP-less expr to handle values >> o

Re: bug#13530: head: memory exhausted when printing all from stdin but last P/E bytes

2013-06-02 Thread Pádraig Brady
On 06/02/2013 04:46 AM, Jim Meyering wrote: > diff --git a/tests/misc/head-c.sh b/tests/misc/head-c.sh > index 37a86ce..a5b6c6a 100755 > --- a/tests/misc/head-c.sh > +++ b/tests/misc/head-c.sh > @@ -19,6 +19,7 @@ > . "${srcdir=.}/tests/init.sh"; path_prepend_ ./src > print_ver_ head > require_ul

Re: Failed with output: Hydra job gnu:coreutils-master:build on x86_64-linux

2013-06-03 Thread Pádraig Brady
On 06/02/2013 08:24 PM, Hydra Build Daemon wrote: > Hi, > > The status of Hydra job gnu:coreutils-master:build (on x86_64-linux) has > changed from "Success" to "Failed with output". For details, see > > http://hydra.nixos.org/build/5221053 > > Go forth and fix it. The failing test was: FA

Re: [PATCH] ls: with -Z, show SMACK context for each file

2013-06-03 Thread Pádraig Brady
On 06/03/2013 10:19 AM, Jarkko Sakkinen wrote: > Enable showing of file SMACK labels with -Z command-line switch. > > * src/ls.c (gobble_file): Output the smack context if available. > * src/ls.c: New function getsmackcon_cache() for grabbing SMACK label. Better thanks. Though you only addressed

Re: Failed with output: Hydra job gnu:coreutils-master:build on x86_64-linux

2013-06-03 Thread Pádraig Brady
On 06/03/2013 12:14 PM, Bernhard Voelker wrote: > Hi Padraig, > > thanks again for working on this. > > On 06/03/2013 12:13 PM, Pádraig Brady wrote: >> From 12b6c3bf70dde23c2c1bf5db7650a959f327b04f Mon Sep 17 00:00:00 2001 >> From: =?UTF-8?q?P=C3=A1draig=20Brady?= >

<    4   5   6   7   8   9   10   11   12   13   >