On Sun, May 29, 2011 at 08:52:52PM -0400, A. Costa wrote:

>     Package bar::::::::baz not installed or bar::::::::baz.list is empty. 

that output is clearly a bug, not a deliberate design decision (i
somehow missed noticing that part of the original bug report). easily
fixed. will include the following patch or similar in next dlocate
release.

--- dlocate.orig        2009-06-03 11:29:06.000000000 +1000
+++ dlocate     2011-05-30 13:15:56.932384187 +1000
@@ -217,7 +217,7 @@
 FILES_REGEXP="($PKGS_REGEXP)"
 [ "$WORD_RE" = "-w" ] && FILES_REGEXP="\b$FILES_REGEXP\b"

-PKGS=$(echo "$PKGS" | sed -e "s/$SEPARATOR/ /")
+PKGS=$(echo "$PKGS" | sed -e "s/$SEPARATOR/ /g")

 if [ "$VERBOSE" = "1" ] ; then
     echo "PKGS:" $PKGS




BTW, using the 8 colons as a temporary separator is completely
arbitrary. chosen because it's extremely unlikely (i.e. pathologically
stupid and perverse) that any real package or filename will contain a
sequence of 8 colons. 

there are, of course, better ways of doing this, but we already know
that dlocate is in need of a complete rewrite. which i keep putting
off because i can't think of any way of doing it that won't break
backwards-compatibility (which will affect any scripts people might have
written that use dlocate), because arg handling is the number-one
thing that needs to be fixed.


i don't get the point of the rest of your report except that you seem
to be assuming that the bug is intended behaviour.

craig

-- 
craig sanders <[email protected]>

BOFH excuse #76:

Unoptimized hard drive



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to