Gentoo ebuild for notmuch

classic Classic list List threaded Threaded
5 messages Options
James Rowe James Rowe
Reply | Threaded
Open this post in threaded view
|

Gentoo ebuild for notmuch

Hi,

  Just in case other Gentoo users are trying notmuch out I thought I'd post my
ebuild(perhaps you'll make it better for me too :).  It is working well up to
at least e5da2b70.

  I won't bother the list if it requires changes, as it is available from my
main overlay[1] if you wish to check for updated versions.

Thanks,

James
  1. http://github.com/JNRowe/misc-overlay/tree/master/mail-client/notmuch/


notmuch-9999.ebuild (1K) Download Attachment
50notmuch-gentoo.el (115 bytes) Download Attachment
Carl Worth-2 Carl Worth-2
Reply | Threaded
Open this post in threaded view
|

Re: Gentoo ebuild for notmuch

On Thu, 19 Nov 2009 00:11:46 -0800 (PST), James Rowe <[hidden email]> wrote:
>   Just in case other Gentoo users are trying notmuch out I thought I'd post my
> ebuild(perhaps you'll make it better for me too :).  It is working well up to
> at least e5da2b70.

Hi James,

Thanks for your contribution.

>   I won't bother the list if it requires changes, as it is available from my
> main overlay[1] if you wish to check for updated versions.

If you think it makes sense, I can add this to the notmuch repository in
a packaging/gentoo directory. Just let me know if you'd like that.

-Carl

James Rowe James Rowe
Reply | Threaded
Open this post in threaded view
|

Re: Gentoo ebuild for notmuch

On Fri, 20 Nov 2009 10:39:35 +0100, Carl Worth <[hidden email]> wrote:
> If you think it makes sense, I can add this to the notmuch repository in
> a packaging/gentoo directory. Just let me know if you'd like that.

  Hmm... The problem is the ebuild can't currently use the install
target because the file locations are incorrect for Gentoo[1]. Which
means it could require quite a bit of churn to keep it synced as the
build process changes.

  The patch that follows makes it easier to use the install target
within the ebuild. If this sort of change is acceptable then the ebuild
will be simpler, and it is more likely to stay working without me
kicking heaps of patches at you. Given that I'd say "yes, please include
it."

Thanks,

James
 1. fex. Bash completion files are installed in to
    /usr/share/bash-completion, then enabled via a link in
    /etc/bash_completion.d if desired

James Rowe James Rowe
Reply | Threaded
Open this post in threaded view
|

[PATCH] Make bash completion directory configurable.

Some systems install completion scripts in /usr/share/bash-completion, make the
location configurable from Makefile.config.
---
 Makefile.config |    1 +
 Makefile.local  |    4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/Makefile.config b/Makefile.config
index 63c90a8..d72a39e 100644
--- a/Makefile.config
+++ b/Makefile.config
@@ -1 +1,2 @@
 prefix = /usr/local
+bash_completion_dir = /etc/bash_completion.d
diff --git a/Makefile.local b/Makefile.local
index ecd4ceb..1017a8c 100644
--- a/Makefile.local
+++ b/Makefile.local
@@ -27,14 +27,14 @@ notmuch.1.gz: notmuch.1
 
 install: all notmuch.1.gz
  for d in $(DESTDIR)$(prefix)/bin/ $(DESTDIR)$(prefix)/share/man/man1 \
- $(DESTDIR)/etc/bash_completion.d/ ; \
+ $(DESTDIR)$(bash_completion_dir) ; \
  do \
  install -d $$d ; \
  done ;
  install notmuch $(DESTDIR)$(prefix)/bin/
  install -m0644 notmuch.1.gz $(DESTDIR)$(prefix)/share/man/man1/
  install notmuch-completion.bash \
- $(DESTDIR)/etc/bash_completion.d/notmuch
+ $(DESTDIR)$(bash_completion_dir)/notmuch
 
 install-emacs: install emacs
  for d in $(DESTDIR)/$(emacs_lispdir) ; \
--
1.6.4.4


Carl Worth-2 Carl Worth-2
Reply | Threaded
Open this post in threaded view
|

Re: Gentoo ebuild for notmuch

In reply to this post by James Rowe
On Fri, 20 Nov 2009 16:09:54 +0000, James Rowe <[hidden email]> wrote:
>   The patch that follows makes it easier to use the install target
> within the ebuild. If this sort of change is acceptable then the ebuild
> will be simpler, and it is more likely to stay working without me
> kicking heaps of patches at you. Given that I'd say "yes, please include
> it."

That change looks just fine to me. Pushed now.

Feel free to send me more if you'd like it integrated to our repository.

-Carl