[Bash-completion-devel] Bug#522545: Bug#522545: bash-completion: configuration file conflict with mtx

David Paleino d.paleino at gmail.com
Mon Apr 6 08:03:38 UTC 2009


# explanation for this later...
clone 522545 -1
reassign -1 mtx
retitle -1 please update bash-completion
owner -1 bash-completion-devel at lists.alioth.debian.org
user -1 bash-completion-devel at lists.alioth.debian.org
usertags -1 move-completions-upstream
thanks

Hello Bdale,

On Sun, 05 Apr 2009 18:41:15 -0600, Bdale Garbee wrote:

> On Sat, 2009-04-04 at 21:03 +0200, David Paleino wrote:
> 
> > > processing /var/cache/debtorrent/ftp.pl.debian.org_debian_dists_unstable_main_binary-all/pool/main/b/bash-completion/bash-completion_1.0-1_all.deb
> > > (--unpack): trying to overwrite `/etc/bash_completion.d/mtx', which is
> > > also in package mtx
> > 
> > Is the version in mtx updated? If not, I'd suggest mtx's maintainer (CCed)
> > to grab the one from bash-completion's git repo.
> > 
> > http://git.debian.org/?p=bash-completion/bash-completion.git;a=blob_plain;f=contrib/mtx;hb=a4c9bdc1774d234f322d214cf7636c10bd2db1c0
> 
> So, what's the right strategy for these?  I added the file in Feb 2004
> on request from Jon Middleton, per Debian bug #227456.  I haven't given
> it a single thought since then...
>
> Does it make sense for me to continue to include the bash completion
> file in the mtx package,

Please do so, as already replied by Luk.

> or should I instead stop including it and add a suggests on bash-completion
> to the package dependencies?

Please don't.
We're planning to "purg" bash-completion from third-party files -- we're going
to give only a framework people can use to write completions. Yes, we'll also
help writing completions (or do that on our own), but those will be
maintained/packaged/submitted upstream (not sure about maintained, but this is
another question).

As suggested by Luk, I've given the completion a look. I'm attaching a "general
patch", since I don't have one of those devices mtx controls :) (so I'm not
really able to parse "mtx status" for tapes and drives)

The most notable thing is:

-    cur=${COMP_WORDS[COMP_CWORD]}
+    cur=`_get_cword`

(the rest of the patch is "preventive")

Also, I kindly ask you to use dh_bash-completion: we're soon going to change
directory layout structure (yes, we'll have some compatibility level), and I'll
update dh_bash-completion accordingly when this happens. dh_bash-completion(1)
is your friend :)

(basically, your package needs no modifications to use it. Just the patch I'm
attaching ;) -- the above quoted fragment *must* be applied)

Thank you,
David

-- 
 . ''`.  Debian maintainer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 ----|---- http://snipr.com/qa_page
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mtx.diff.gz
Type: application/x-gzip
Size: 623 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/bash-completion-devel/attachments/20090406/8ff86449/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/bash-completion-devel/attachments/20090406/8ff86449/attachment-0002.pgp>


More information about the Bash-completion-devel mailing list