Subject: bug#15588: closed (Re: bug#15588: [PATCH] cp: correct error message for invalid arguments of '--no-preserve')

From: [email-masked] (GNU bug Tracking System)
Date: Fri, 11 Oct 2013 16:13:03 +0000

Your bug report

#15588: [PATCH] cp: correct error message for invalid arguments of '--no-preserve'

which was filed against the coreutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
15588: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15588
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#15588: [PATCH] cp: correct error message for invalid arguments of '--no-preserve'
From: Pádraig Brady <[email-masked]>
To: [email-masked]
Cc: [email-masked]
Date: Fri, 11 Oct 2013 17:12:09 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

On 10/11/2013 02:07 PM, Ondrej Vasik wrote:
> * src/cp.c (decode_preserve_arg) :
> Correct error message for invalid arguments of '--no-preserve'.
> Reported by M.Vadkerti in rhbz #1018206.
> ---
>  src/cp.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/src/cp.c b/src/cp.c
> index e235b32..7bc8630 100644
> --- a/src/cp.c
> +++ b/src/cp.c
> @@ -854,7 +854,8 @@ decode_preserve_arg (char const *arg, struct
> cp_options *x, bool on_off)
>          *comma++ = 0;
>  
>        /* process S.  */
> -      val = XARGMATCH ("--preserve", s, preserve_args, preserve_vals);
> +      val = XARGMATCH (on_off ? "--preserve" : "--no-preserve",
> +                       s, preserve_args, preserve_vals);
>        switch (val)
>          {
>          case PRESERVE_MODE:
> 

Pushed.

thanks!
Pádraig.

Subject: [PATCH] cp: correct error message for invalid arguments of '--no-preserve'
From: Ondrej Vasik <[email-masked]>
To: CoreutilsBugs <[email-masked]>
Date: Fri, 11 Oct 2013 15:07:04 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

* src/cp.c (decode_preserve_arg) :
Correct error message for invalid arguments of '--no-preserve'.
Reported by M.Vadkerti in rhbz #1018206.
---
 src/cp.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/cp.c b/src/cp.c
index e235b32..7bc8630 100644
--- a/src/cp.c
+++ b/src/cp.c
@@ -854,7 +854,8 @@ decode_preserve_arg (char const *arg, struct
cp_options *x, bool on_off)
         *comma++ = 0;
 
       /* process S.  */
-      val = XARGMATCH ("--preserve", s, preserve_args, preserve_vals);
+      val = XARGMATCH (on_off ? "--preserve" : "--no-preserve",
+                       s, preserve_args, preserve_vals);
       switch (val)
         {
         case PRESERVE_MODE:
-- 
1.7.1