Re: misc/39459: etcupdate can't parse directory names with special chars

[Alan Barrett <apb%netbsd.org@localhost>]

The following reply was made to PR misc/39459; it has been noted by GNATS.

From: Alan Barrett <apb%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
Cc: 
Subject: Re: misc/39459: etcupdate can't parse directory names with special
        chars
Date: Fri, 5 Sep 2008 09:59:12 +0200

 On Thu, 04 Sep 2008, paul%whooppee.com@localhost wrote:
 >      {107} etcupdate -al -s 
 > /build/release-2008-09-04_20:04:46/amd64/binary/sets/etc.tgz 
 >      *** Cannot understand -s 
 > /build/release-2008-09-04_20:04:46/amd64/binary/sets/etc.tgz
 
 Colons in particular will be a problem.
 
 The reason is that etcupdate's "-s" option takes several different
 types of arguments, and one of the possible types is a colon-separated
 list of files (e.g. {etcupdate -s etc.tgz:xetc.tgz}).  The main reason
 that etcupdate does this is because postinstall does the same.  The
 main reason that postinstall accepts a colon-separated list of files
 is to make it easy for etcupdate to invoke postinstall.  For example,
 if you run {etcupdate -s etc.tgz -s xetc.tgz} then etcupdate will run
 {postinstall -s etc.tgz:xetc.tgz [other options]}.
 
 The obvious way of fixing this would be to remove the ability to use
 colon-separated lists of file names, and to let multiple "-s" options to
 etcupdate be passed through as multiple "s" options to etcupdate.  The
 quoting issues are a bit of a problem, but I can deal with them if there
 is no objection to removing this syntax.
 
 --apb (Alan Barrett)