Re: standards for info install

[Aubrey Jaffer]

 | 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)
        @$(NORMAL_INSTALL)
        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
        @$(POST_INSTALL)
        @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)' 
'$(DESTDIR)$(infodir)/$$relfile'";\
            install-info --info-dir="$(DESTDIR)$(infodir)" 
"$(DESTDIR)$(infodir)/$$relfile" || :;\
          done; \
        else : ; fi
install-man: