[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: standards for info install

From: Aubrey Jaffer
Subject: Re: standards for info install
Date: Sat, 20 Feb 2010 17:16:01 -0500 (EST)

 | Date: Thu, 18 Feb 2010 23:05:21 GMT
 | From: address@hidden (Karl Berry)
 |     Shouldn't mkinstalldirs be a dependency for do-install-info to
 |     insure that $(DESTDIR)$(infodir)/ exists?
 | No, because mkinstalldirs might not exist, in general.  Ensuring
 | that the directory exists, which is certainly desirable, is another
 | can of worms that would go far afield here.  It's an example.

Oops -- I meant installdirs; I have changed it in my Makefiles.

 |       d=.
 |       install -m 644 $d/slib.info /usr/local/share/info/slib.info
 |       install: cannot stat `/slib.info': No such file or directory
 | That's the mistake Ralf pointed out that I made, leaving off the ;\
 | in the trial example in my example (that was never committed), so
 | $d wasn't defined in the install command.  The example that was
 | actually in standards.texi did work, aside from the problems you
 | pointed out originally.

My mistake again; sorry.

I took a look at guile-1.9.7/doc/ref/Makefile for an example
(appended).  This is beyond my scripting knowledge; "sed 's|.|.|g'"
looks like a no-op to me.

It is doing "test -f" like your example.  If I have an old build of
guile.info and update guile.texi, then the "test -f" line looks to me
like it would install the old guile.info; that seems wrong.


INFO_DEPS = $(srcdir)/guile.info
install-info-am: $(INFO_DEPS)
        test -z "$(infodir)" || $(MKDIR_P) "$(DESTDIR)$(infodir)"
        @srcdirstrip=`echo "$(srcdir)" | sed 's|.|.|g'`; \
        list='$(INFO_DEPS)'; test -n "$(infodir)" || list=; \
        for file in $$list; do \
          case $$file in \
            $(srcdir)/*) file=`echo "$$file" | sed "s|^$$srcdirstrip/||"`;; \
          esac; \
          if test -f $$file; then d=.; else d=$(srcdir); fi; \
          file_i=`echo "$$file" | sed 's|\.info$$||;s|$$|.i|'`; \
          for ifile in $$d/$$file $$d/$$file-[0-9] $$d/$$file-[0-9][0-9] \
                       $$d/$$file_i[0-9] $$d/$$file_i[0-9][0-9] ; do \
            if test -f $$ifile; then \
              echo "$$ifile"; \
            else : ; fi; \
          done; \
        done | $(am__base_list) | \
        while read files; do \
          echo " $(INSTALL_DATA) $$files '$(DESTDIR)$(infodir)'"; \
          $(INSTALL_DATA) $$files "$(DESTDIR)$(infodir)" || exit $$?; done
        @if (install-info --version && \
             install-info --version 2>&1 | sed 1q | grep -i -v debian) 
>/dev/null 2>&1; then \
          list='$(INFO_DEPS)'; test -n "$(infodir)" || list=; \
          for file in $$list; do \
            relfile=`echo "$$file" | sed 's|^.*/||'`; \
            echo " install-info --info-dir='$(DESTDIR)$(infodir)' 
            install-info --info-dir="$(DESTDIR)$(infodir)" 
"$(DESTDIR)$(infodir)/$$relfile" || :;\
          done; \
        else : ; fi

reply via email to

[Prev in Thread] Current Thread [Next in Thread]