Your message dated Tue, 12 May 2020 04:49:46 +0000
with message-id <[email protected]>
and subject line Bug#892301: fixed in ucf 3.0039
has caused the Debian Bug report #892301,
regarding ucf with -n as first argument results in error
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
892301: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=892301
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: ucf
Version: 3.0038

repro steps:

run with valid file arguments:

    # /usr/bin/ucf -n --debconf-ok /file1 /file2
    *** ERROR: Need exactly two arguments, got 3                                
                                                  

    Debian GNU/Linux ucf Revision: 3.00 .                                       
                                                  
               Copyright (C) 2002-2005 Manoj Srivastava.                        
                                                  
    This is free software; see the GNU General Public Licence for copying       
                                                  
    conditions.  There is NO warranty.                                          
                                                  

    Usage: ucf  [options] new_file  destination                                 
                                                  
    Options:                                                                    
                                                  
         -h,     --help          print this message                             
                                                  
         -s foo, --src-dir  foo  Set the src dir (historical md5sums live here) 
                                                  
                 --sum-file bar  Force the historical md5sums to be read from   
                                                  
                                 this file.  Overrides any setting of 
--src-dir.                                                  
         -d[n], --debug=[n]      Set the Debug level to N. Please note there 
must                                                 
                                 be no spaces before the debug level            
                                                  
         -n,     --no-action     Dry run. No action is actually taken.          
                                                  
         -v,     --verbose       Make the script verbose                        
                                                  
                 --three-way     Register this file in the cache, and turn on 
the                                                 
                                 diff3 option allowing the merging of 
maintainer                                                  
                                 changes into a (potentially modified) local    
                                                  
                                 configuration file. )                          
                                                  
                 --state-dir bar Set the state directory to bar instead of the  
                                                  
                                 default '/var/lib/ucf'. Used mostly for 
testing.                                                 
                 --debconf-ok    Indicate that it is ok for ucf to use an 
already                                                 
                                 running debconf instance for prompting.        
                                                  
                 --debconf-template bar                                         
                                                  
                                 Specify an alternate, caller-provided debconf  
                                                  
                                 template to use for prompting.                 
                                                  
    Usage: ucf  -p  destination                                                 
                                                  
         -p,     --purge         Remove any reference to destination from 
records                                                 

    By default, the directory the new_file lives in is assumed to be the 
src-dir,                                                 
    which is where we look for any historical md5sums.       


expected: does dry run as it should without error.

This is because it tries to save the -n argument then use it to call
itself again. But when saving the argument, it runs echo -n, expecting
the output to be -n, but it's actually an empty string, so it saves that
emptry string and passes that instead of -n.

Patch is attached.

--- ucf-3.0038/ucf	2018-02-25 19:58:23.000000000 -0500
+++ new/ucf	2018-03-07 16:42:55.727057127 -0500
@@ -308,7 +308,7 @@
 
 # Escape single quotes in the arguments passed in
 quote_single() {
-    echo "$1" | sed -e "s,','\\\\'',g"
+    printf "%s\n" "$1" | sed -e "s,','\\\\'',g"
 }
 
 
-- 
Ian Kelling | Senior Systems Administrator, Free Software Foundation
GPG Key: B125 F60B 7B28 7FF6 A2B7  DF8F 170A F0E2 9542 95DF
https://fsf.org | https://gnu.org

--- End Message ---
--- Begin Message ---
Source: ucf
Source-Version: 3.0039
Done: Manoj Srivastava <[email protected]>

We believe that the bug you reported is fixed in the latest version of
ucf, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Manoj Srivastava <[email protected]> (supplier of updated ucf package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Mon, 11 May 2020 20:53:59 -0700
Source: ucf
Architecture: source
Version: 3.0039
Distribution: unstable
Urgency: medium
Maintainer: Manoj Srivastava <[email protected]>
Changed-By: Manoj Srivastava <[email protected]>
Closes: 892301 918255 935439 951006
Changes:
 ucf (3.0039) unstable; urgency=medium
 .
   * NMU ack. Thanks to Helge Kreutzmann
   * Updated thee VCS-* links in control file to salsa
   * Updated the standards version to 4.3.0, no changes needed
   * Bug fix: "ucf shouldn't change the file permissions", thanks to
     Laurent Bigonville (Closes: #935439).
   * Updated the debhelper dependency usding the new debhelper-compat
     dependency convention.
   * Bug fix: "ucf with -n as first argument results in error", thanks to
     Ian Kelling (Closes: #892301).
   * Bug fix: "Please load file names into environment when starting
     &quot;a new shell to examine the situation&quot; (DPKG_CONFFILE_OLD,
     DPKG_CONFFILE_NEW)", thanks to Christoph Biedl (Closes: #951006). mI
     changed the variable names to not stomp over tyhe dpkg namespace; the
     variables are actuall UCF_*
   * Bug fix: "[INTL:nl] Dutch translation of debconf messages", thanks to
     Frans Spiesschaert (Closes: #918255).
Checksums-Sha1:
 18775fdf3f62bd9a05ce953c40eb53b1932427e7 1419 ucf_3.0039.dsc
 677df859ed4ff465b16e0ba4faf630d2238837d2 66240 ucf_3.0039.tar.xz
Checksums-Sha256:
 8ae5e9ba14d264edbaf9e62b4a945d194215950ac590226ec3dea7c48c1300b2 1419 
ucf_3.0039.dsc
 d7adb61cf16d6f273673b39ef825629a3c4a129753592fb1036fefc3d582c0c4 66240 
ucf_3.0039.tar.xz
Files:
 ed74a38e446a5c2a7e80047d6754bfbc 1419 utils standard ucf_3.0039.dsc
 a14af416236bf0a83a8a8bb3319cc6f7 66240 utils standard ucf_3.0039.tar.xz

-----BEGIN PGP SIGNATURE-----

iQGYBAEBCgCCFiEEq6cQJaG1qIpOX2jCNr1yD29XZHIFAl66JWVPFIAAAAAAGwAr
c3JpdmFzdGFAZ29sZGVuLWdyeXBob24uY29tQUJBNzEwMjVBMUI1QTg4QTRFNUY2
OEMyMzZCRDcyMEY2RjU3NjQ3Ml84OBQcc3JpdmFzdGFAZGViaWFuLm9yZwAKCRA2
vXIPb1dkclkeB/4++UCdF0Fzf2/WNENeXzn+bhmiiBcZlD4Yxh2s4Ql+eBGOPUGz
4NuJURDr5+/IFZr8Kw/Mezs0IBgvlUurtWmKPNcSkpHVdC8xcCgvhOX33Qu3nhyh
tq5B+XCtfVosCNBcLgHvdENpG/liMZs1wskpZrTADdPOXNGMBNgd7yB06aVDy+8v
nBqjatyzHW14b8kkNGmMJXFcjUsCBkFS4XMQRyUL5QtoTzK2ouNU/mViEnXJkGLP
D3wFzdaz5kbPr5zht0eLN1rkc1mf6IXYvHIlChn6kDJVEPpJNH3mO64HD3LTzYcM
j0kw9ZKhgYD47Ocwk47YNpshZdUgK+tc1h57
=bHhP
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to