Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 521372 - =net-mail/notmuch-0.18.1 install bash-completion and zsh-completion files unconditionally
Summary: =net-mail/notmuch-0.18.1 install bash-completion and zsh-completion files unc...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Amadeusz Żołnowski (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 516756
  Show dependency tree
 
Reported: 2014-08-28 11:00 UTC by Manuel Rüger (RETIRED)
Modified: 2014-09-18 17:29 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Rüger (RETIRED) gentoo-dev 2014-08-28 11:00:31 UTC
Please drop IUSE="bash-completion zsh-completion" and install those files unconditionally.
Esp. zsh-completion? ( app-shells/zsh ) seems to be incorrect, as the ebuild is  only installing a completion file.
Comment 1 Amadeusz Żołnowski (RETIRED) gentoo-dev 2014-08-28 11:36:33 UTC
+  28 Aug 2014; Amadeusz Żołnowski <aidecoe@gentoo.org> notmuch-0.18.1.ebuild:
+  Unconditional installation of zsh/bash completion files. Fixes bug #521372.
+


Please check if it's OK and then I'll backport it to older, stable versions.
Comment 2 Franz Trischberger 2014-08-30 07:02:10 UTC
I don't like the dependency on the whole bash-completion-stack.

     Fri Aug 29 12:03:35 2014 >>> app-admin/eselect-bashcomp-1.3.6
     Fri Aug 29 12:03:45 2014 >>> app-shells/bash-completion-2.1-r1
     Fri Aug 29 12:03:54 2014 >>> app-shells/gentoo-bashcomp-20121024

/usr/share/bash-completion already contains a lot of completion files, without app-shells/bash-completion.

So could we get rid of the dep on >=app-shells/bash-completion-1.9, please? Thx :)
Comment 3 Amadeusz Żołnowski (RETIRED) gentoo-dev 2014-09-01 07:04:38 UTC
notmuch requires at least version 1.9 and we have to enforce this dependency.
Comment 4 Franz Trischberger 2014-09-01 07:38:44 UTC
(In reply to Amadeusz Żołnowski from comment #3)
> notmuch requires at least version 1.9 and we have to enforce this dependency.

Force a dependency on an OPTIONAL feature?
Other packages use pkg_postinst to make the user aware of optional features that will be available after installing certain additional packages. Just mention that for notmuch you need at least bash-completion-1.9.

I see that bash-completion is a widely used feature and users probably expect it to work. Nevertheless the user will notice when completion does not work and (hopefully) find the message from notmuch he just missed.

But in the end I don't REALLY care about those something-dozen files that I have on my disk additionally ;)
Comment 5 Amadeusz Żołnowski (RETIRED) gentoo-dev 2014-09-01 07:56:06 UTC
Done in stable ebuild as well.

+  01 Sep 2014; Amadeusz Żołnowski <aidecoe@gentoo.org>
+  notmuch-0.16-r1.ebuild, notmuch-0.17.ebuild:
+  Install zsh and bash completion files unconditionally.
+
Comment 6 W. Trevor King 2014-09-02 15:50:10 UTC
(In reply to Franz Fellner from comment #4)
> (In reply to Amadeusz Żołnowski from comment #3)
> > notmuch requires at least version 1.9 and we have to enforce this dependency.
> 
> Force a dependency on an OPTIONAL feature?

I expect the reason for dropping the USE-based dependencies is this policy [1]:

> The usage of a USE flag should not control runtime dependencies when the
> package does not link to it. Doing so will create extra configuration for
> the package and re-compilation for no underlying file change on disk. This
> should be avoided and instead can be conveyed to the user via post install
> messages if needed. 

While the changes made because of the issue comply with the letter of that policy, I join Franz and the devmanual authors in prefering the pkg_postinst route to forcing the dependency.

[1]: http://devmanual.gentoo.org/general-concepts/use-flags/