[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

gregames, are you sure about Makefile.in rev 1.122 ?

From: Karl Fogel <kfogel_at_newton.ch.collab.net>
Date: 2002-07-19 21:18:21 CEST

I'm building HEAD of httpd right now, like this:

   $ ./buildconf
   $ ./configure --enable-dav --enable-so --enable-maintainer-mode
   $ make depend
   $ make
   $ make install

The last step fails because /usr/local/apache2/conf is not a
directory, it's a file! The file appears to be a mod_mime_magic
configuration file or something.

Sander Striker mentioned to me that Greg Ames (I assume that's the
expansion of "gregames", never met him personally :-) ) had made some
changes very recently that might have affected this.

So Greg, are you sure about this change to Makefile.in:

    2002-07-19 10:11 gregames

        * Makefile.in (1.122): Install mime.types and magic in conf/ if
        they don't already exist.
        
        This also re-enables existing logic to always install *-std.conf
        with substitutions made. These are then copied into *.conf
        (without the -std) if that file doesn't already exist.
        
        style police: please chill for a bit. I intentionally didn't
        re-format the indenting in this commit to make it easier to review.

The patch is below. In the added code, it looks like perhaps the
sense of the first `if' test is reversed, though that may be
relatively harmless. Then I'm not sure about the next `if', but I'd
guess the line

   "$(INSTALL_DATA) $$i $(DESTDIR)$(sysconfdir);"

is being run when it shouldn't. If "$(DESTDIR)$(sysconfdir)" does not
exist, then the conf file will itself be copied to that name!

--- Makefile.in 1 Jul 2002 11:44:21 -0000 1.121
+++ Makefile.in 19 Jul 2002 15:11:58 -0000 1.122
@@ -27,14 +27,15 @@
 include $(top_srcdir)/build/program.mk
 
 install-conf:
- -@if [ -d $(DESTDIR)$(sysconfdir) ]; then \
- echo "[PRESERVING EXISTING CONF SUBDIR: $(DESTDIR)$(sysconfdir)]"; \
- else \
- echo Installing configuration files ; \
+ @echo Installing configuration files ; \
+ if [ -d $(DESTDIR)$(sysconfdir) ]; then \
             $(MKINSTALLDIRS) $(DESTDIR)$(sysconfdir) ; \
+ fi ; \
             cd $(top_srcdir)/docs/conf; \
             for i in mime.types magic; do \
+ if test ! -f $(DESTDIR)$(sysconfdir)/$$i; then \
                     $(INSTALL_DATA) $$i $(DESTDIR)$(sysconfdir); \
+ fi; \
             done; \
             for j in $(top_srcdir)/docs/conf $(top_builddir)/docs/conf ; do \
                     cd $$j ; \
@@ -86,7 +87,6 @@
                 if test ! -f $(DESTDIR)$(sbindir)/envvars; then \
                     cp -p envvars-std $(DESTDIR)$(sbindir)/envvars ; \
                 fi ; \
- fi ; \
         fi
 
 install-build:

Can anyone else run "make install" with head and get a correct conf/
directory?

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jul 19 21:30:39 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.