GNU bug report logs - #17087
cp -i/yes gets ignored

Previous Next

Full log

View this message in rfc822 format

From: Pádraig Brady <P <at> draigBrady.com>
To: Bernhard Voelker <mail <at> bernhard-voelker.de>
Cc: Paul Eggert <eggert <at> cs.ucla.edu>, 17087 <at> debbugs.gnu.org, Karl Berry <karl <at> freefriends.org>
Subject: bug#17087: cp -i/yes gets ignored
Date: Wed, 26 Mar 2014 14:20:13 +0000

[Message part 1 (text/plain, inline)]

On 03/26/2014 01:26 PM, Bernhard Voelker wrote:
> On 03/25/2014 07:25 AM, Paul Eggert wrote:
>> (the second is just a code cleanup).
> 
> Oops, that 2nd patch
>   http://git.sv.gnu.org/cgit/coreutils.git/commit/?id=7f669b40
> now produces the following check failure:
> 
> FAIL: tests/mv/i-3
> ==================

That's an interesting one. I guess our CI didn't flag this
as the test is skipped without a controlling tty:
http://hydra.nixos.org/jobset/gnu/coreutils-master

Also the test is skipped with a parallel make check!
I.E. this first command detects the fail, while the second skips the test:
  $ make -j2 check TESTS="tests/mv/i-3.sh"
  $ make -j2 check TESTS="tests/mv/i-2.sh tests/mv/i-3.sh"

So to handle both cases I'm updating the "release tasks" list to:
  setsid make check $(($(nproc) * 2)) # tests without controlling tty
  make check -j1 # tests with controlling tty

> BTW: in check_overwrite_prompt, the file 'out' contains
>   "mv: unwritable 'g' (mode 0000, ---------); try anyway?"

That should have been apparent in the test log immediately,
without you having to add debug.

In general we should allow these patches a little
while for review before pushing.

Hopefully the attached fixes up these issues.

thanks,
Pádraig.

[mv-prompt-fix.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.