Content-Type: text/plain; charset=utf8
From: Ingmar Vanhassel <ingmar@exherbo.org>
To: notmuch <notmuch@notmuchmail.org>
In-reply-to: <1258640822-4146-1-git-send-email-jeff@ocjtech.us>
References: <1258636656-sup-8390@cannonball>
	<1258640822-4146-1-git-send-email-jeff@ocjtech.us>
Date: Thu, 19 Nov 2009 20:32:22 +0100
Message-Id: <1258648185-sup-3691@cannonball>
User-Agent: Sup/git
Content-Transfer-Encoding: 8bit
Subject: Re: [notmuch] [PATCH] notmuch: improve installation of emacs mode
Precedence: list

Excerpts from Jeffrey C. Ollie's message of Thu Nov 19 15:27:02 +0100 2009:
> 1) Add a separate targets to build and install emacs mode.
> 
> 2) Don't hardcode the installation directory, instead use emacs'
>    pkg-config module.
> 
> 3) Install a byte compiled version of the emacs mode.
> 
> 4) Install the emacs mode in emacs' site-start directory so that it
>    gets loaded automatically.
> 
> 5) Ignore byte-compiled emacs files.
> 
> Signed-off-by: Jeffrey C. Ollie <jeff@ocjtech.us>

Reviewed-by: Ingmar Vanhassel <ingmar@exherbo.org>

Thanks again!

> ---
>  .gitignore     |    2 +-
>  Makefile       |    6 ++++++
>  Makefile.local |   15 ++++++++++++---
>  3 files changed, 19 insertions(+), 4 deletions(-)
> 
> diff --git a/.gitignore b/.gitignore
> index 8417d60..7d3c543 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -5,4 +5,4 @@ notmuch.1.gz
>  *.[ao]
>  *~
>  .*.swp
> -
> +*.elc
> diff --git a/Makefile b/Makefile
> index 023b2ec..17fa4a2 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -7,6 +7,9 @@ CFLAGS=-O2
>  extra_cflags := $(shell pkg-config --cflags glib-2.0 gmime-2.4 talloc)
>  extra_cxxflags := $(shell xapian-config --cxxflags)
>  
> +emacs_lispdir := $(shell pkg-config emacs --variable sitepkglispdir)
> +emacs_startdir := $(shell pkg-config emacs --variable sitestartdir)
> +
>  # Now smash together user's values with our extra values
>  override CFLAGS += $(WARN_FLAGS) $(extra_cflags)
>  override CXXFLAGS += $(WARN_FLAGS) $(extra_cflags) $(extra_cxxflags)
> @@ -28,6 +31,9 @@ include Makefile.config
>  %.o: %.c
>      $(CC) -c $(CFLAGS) $< -o $@
>  
> +%.elc: %.el
> +    emacs -batch -f batch-byte-compile $<
> +
>  .deps/%.d: %.c
>      @set -e; rm -f $@; mkdir -p $$(dirname $@) ; \
>      $(CC) -M $(CPPFLAGS) $(CFLAGS) $< > $@.$$$$; \
> diff --git a/Makefile.local b/Makefile.local
> index 6bc01a5..2bb4521 100644
> --- a/Makefile.local
> +++ b/Makefile.local
> @@ -1,5 +1,7 @@
>  all: notmuch notmuch.1.gz
>  
> +emacs: notmuch.elc
> +
>  notmuch_client_srcs =        \
>      notmuch.c        \
>      notmuch-config.c    \
> @@ -25,15 +27,22 @@ notmuch.1.gz: notmuch.1
>  
>  install: all notmuch.1.gz
>      for d in $(DESTDIR)$(prefix)/bin/ $(DESTDIR)$(prefix)/share/man/man1 \
> -        $(DESTDIR)/$(prefix)/share/emacs/site-lisp/
> $(DESTDIR)/etc/bash_completion.d/ ; \
> +        $(DESTDIR)/etc/bash_completion.d/ ; \
>      do \
>          install -d $$d ; \
>      done ;
>      install notmuch $(DESTDIR)$(prefix)/bin/
>      install -m0644 notmuch.1.gz $(DESTDIR)$(prefix)/share/man/man1/
> -    install -m0644 notmuch.el $(DESTDIR)$(prefix)/share/emacs/site-lisp/
>      install notmuch-completion.bash \
>          $(DESTDIR)/etc/bash_completion.d/notmuch
>  
> +install-emacs: install emacs
> +    for d in $(DESTDIR)/$(emacs_startdir) ; \
> +    do \
> +        install -d $$d ; \
> +    done ;
> +    install -m0644 notmuch.el $(DESTDIR)$(emacs_startdir)
> +    install -m0644 notmuch.elc $(DESTDIR)$(emacs_startdir)
> +
>  SRCS  := $(SRCS) $(notmuch_client_srcs)
> -CLEAN := $(CLEAN) notmuch $(notmuch_client_modules)
> +CLEAN := $(CLEAN) notmuch $(notmuch_client_modules) notmuch.elc
-- 
Exherbo KDE, X.org maintainer