unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] Fix dependency generation for CLI sources
@ 2012-01-15 20:20 Austin Clements
  2012-01-18 15:38 ` Tomi Ollila
  2012-01-19 17:15 ` David Bremner
  0 siblings, 2 replies; 3+ messages in thread
From: Austin Clements @ 2012-01-15 20:20 UTC (permalink / raw)
  To: notmuch

Previously, the dependency file list was generated before the CLI
sources were added to SRCS, so dependency files weren't generated for
CLI sources.  This moves that code to after the CLI sources are added.
---
 Makefile.local |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Makefile.local b/Makefile.local
index d3bf947..1131dea 100644
--- a/Makefile.local
+++ b/Makefile.local
@@ -273,10 +273,6 @@ quiet ?= $($(shell echo $1 | sed -e s'/ .*//'))
 	sed 's,'$$(basename $*)'\.o[ :]*,$*.o $@ : ,g' < $@.$$$$ > $@; \
 	rm -f $@.$$$$
 
-DEPS := $(SRCS:%.c=.deps/%.d)
-DEPS := $(DEPS:%.cc=.deps/%.d)
--include $(DEPS)
-
 .PHONY : clean
 clean:
 	rm -f $(CLEAN); rm -rf .deps
@@ -350,3 +346,7 @@ install-desktop:
 
 SRCS  := $(SRCS) $(notmuch_client_srcs)
 CLEAN := $(CLEAN) notmuch notmuch-shared $(notmuch_client_modules) notmuch.elc
+
+DEPS := $(SRCS:%.c=.deps/%.d)
+DEPS := $(DEPS:%.cc=.deps/%.d)
+-include $(DEPS)
-- 
1.7.7.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Fix dependency generation for CLI sources
  2012-01-15 20:20 [PATCH] Fix dependency generation for CLI sources Austin Clements
@ 2012-01-18 15:38 ` Tomi Ollila
  2012-01-19 17:15 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: Tomi Ollila @ 2012-01-18 15:38 UTC (permalink / raw)
  To: Austin Clements, notmuch

On Sun, 15 Jan 2012 15:20:23 -0500, Austin Clements <amdragon@MIT.EDU> wrote:
> Previously, the dependency file list was generated before the CLI
> sources were added to SRCS, so dependency files weren't generated for
> CLI sources.  This moves that code to after the CLI sources are added.
> ---

+1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Fix dependency generation for CLI sources
  2012-01-15 20:20 [PATCH] Fix dependency generation for CLI sources Austin Clements
  2012-01-18 15:38 ` Tomi Ollila
@ 2012-01-19 17:15 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: David Bremner @ 2012-01-19 17:15 UTC (permalink / raw)
  To: Austin Clements, notmuch

On Sun, 15 Jan 2012 15:20:23 -0500, Austin Clements <amdragon@MIT.EDU> wrote:
> Previously, the dependency file list was generated before the CLI
> sources were added to SRCS, so dependency files weren't generated for
> CLI sources.  This moves that code to after the CLI sources are added.

pushed, 

d

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-01-19 17:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-15 20:20 [PATCH] Fix dependency generation for CLI sources Austin Clements
2012-01-18 15:38 ` Tomi Ollila
2012-01-19 17:15 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).