Fix a build-warning when building out-of-tree

We were trying to incorporate all headers in common_sha1.i, not just
the src/common ones.

This is part of bug 6778; fix on 0.2.4.1-alpha

diff --git a/changes/bug6778 b/changes/bug6778
new file mode 100644 (file)
index 0000000..3585140
--- /dev/null
+++ b/changes/bug6778
@@ -0,0 +1,3 @@
+  o Minor bugfixes:
+    - Avoid a warning when building common_sha1.i out of tree. Fixes bug
+      6778; bugfix on 0.2.4.1-alpha.

diff --git a/src/common/include.am b/src/common/include.am
index 0ab4769847bd927be2ff6d536d02428e3a004419..c6fde42781951ae75727f061d97d93e26f45220c 100644 (file)
--- a/src/common/include.am
+++ b/src/common/include.am
@@ -36,7 +36,7 @@ src_common_libor_crypto_a_SOURCES = \
 
 src_common_libor_event_a_SOURCES = src/common/compat_libevent.c
 
-noinst_HEADERS+=                               \
+COMMONHEADERS = \
   src/common/address.h                         \
   src/common/aes.h                             \
   src/common/ciphers.inc                       \
@@ -57,13 +57,15 @@ noinst_HEADERS+=                            \
   src/common/tortls.h                          \
   src/common/util.h
 
+noinst_HEADERS+= $(COMMONHEADERS)
+
 DISTCLEANFILES+= src/common/common_sha1.i
 
-src/common/common_sha1.i: $(libor_SOURCES) $(libor_crypto_a_SOURCES) $(noinst_HEADERS)
+src/common/common_sha1.i: $(libor_SOURCES) $(libor_crypto_a_SOURCES) $(COMMONHEADERS)
        $(AM_V_GEN)if test "@SHA1SUM@" != none; then \
-         (cd "$(srcdir)" && "@SHA1SUM@" $(src_common_libor_SOURCES) $(src_common_libor_crypto_a_SOURCES) $(noinst_HEADERS)) | "@SED@" -n 's/^\(.*\)$$/"\1\\n"/p' > $@; \
+         (cd "$(srcdir)" && "@SHA1SUM@" $(src_common_libor_SOURCES) $(src_common_libor_crypto_a_SOURCES) $(COMMONHEADERS)) | "@SED@" -n 's/^\(.*\)$$/"\1\\n"/p' > $@; \
        elif test "@OPENSSL@" != none; then \
-         (cd "$(srcdir)" && "@OPENSSL@" sha1 $(src_common_libor_SOURCES) $(src_Common_libor_crypto_a_SOURCES) $(noinst_HEADERS)) | "@SED@" -n 's/SHA1(\(.*\))= \(.*\)/"\2  \1\\n"/p' > $@; \
+         (cd "$(srcdir)" && "@OPENSSL@" sha1 $(src_common_libor_SOURCES) $(src_Common_libor_crypto_a_SOURCES) $(COMMONHEADERS)) | "@SED@" -n 's/SHA1(\(.*\))= \(.*\)/"\2  \1\\n"/p' > $@; \
        else \
          rm $@; \
          touch $@; \

diff --git a/src/or/include.am b/src/or/include.am
index 65ff6849259a358fc9eb9ee211e75a6f0e7bacdb..34cd307b49b16ae3908561059a8d514f6c7e2380 100644 (file)
--- a/src/or/include.am
+++ b/src/or/include.am
@@ -81,7 +81,7 @@ src_or_tor_LDADD = src/or/libtor.a src/common/libor.a src/common/libor-crypto.a
        @TOR_ZLIB_LIBS@ @TOR_LIB_MATH@ @TOR_LIBEVENT_LIBS@ @TOR_OPENSSL_LIBS@ \
        @TOR_LIB_WS32@ @TOR_LIB_GDI@
 
-noinst_HEADERS+= \
+ORHEADERS = \
        src/or/buffers.h                                \
        src/or/circuitbuild.h                           \
        src/or/circuitlist.h                            \
@@ -122,8 +122,9 @@ noinst_HEADERS+= \
        src/or/router.h                                 \
        src/or/routerlist.h                             \
        src/or/routerparse.h                            \
-       src/or/status.h                                 \
-       micro-revision.i
+       src/or/status.h
+
+noinst_HEADERS+= $(ORHEADERS) micro-revision.i
 
 src/or/config_codedigest.o: src/or/or_sha1.i
 
@@ -143,12 +144,12 @@ micro-revision.i: FORCE
          mv micro-revision.tmp micro-revision.i;               \
        fi; true
 
-src/or/or_sha1.i: $(src_or_tor_SOURCES) $(src_or_libtor_a_SOURCES)
+src/or/or_sha1.i: $(src_or_tor_SOURCES) $(src_or_libtor_a_SOURCES) $(ORHEADERS)
        $(AM_V_GEN)if test "@SHA1SUM@" != none; then \
-         (cd "$(srcdir)" && "@SHA1SUM@" $(src_or_tor_SOURCES) $(src_or_libtor_a_SOURCES)) | \
+         (cd "$(srcdir)" && "@SHA1SUM@" $(src_or_tor_SOURCES) $(src_or_libtor_a_SOURCES) $(ORHEADERS) ) | \
          "@SED@" -n 's/^\(.*\)$$/"\1\\n"/p' > src/or/or_sha1.i; \
        elif test "@OPENSSL@" != none; then \
-         (cd "$(srcdir)" && "@OPENSSL@" sha1 $(src_or_tor_SOURCES) $(src_or_libtor_a_SOURCES)) | \
+         (cd "$(srcdir)" && "@OPENSSL@" sha1 $(src_or_tor_SOURCES) $(src_or_libtor_a_SOURCES) $(ORHEADERS)) | \
          "@SED@" -n 's/SHA1(\(.*\))= \(.*\)/"\2  \1\\n"/p' > src/or/or_sha1.i; \
        else \
          rm src/or/or_sha1.i; \