Frank Küster wrote:
Well, there are still the info files.  Whether it is grave or important
to not include the html docs, this I leave to the maintainer.

Never mind, let's fix it!

However, the bug can be fixed by adding rsync to Build-Depends-Indep.
At least this helped when I tried in a pbuilder chroot, using
"debian/rules binary-indep".  I'm currently trying with
pbuilder/dpkg-buildpackage and a complete build.

Great! Hope it works.

However, I think even if this helps to re-include the html
documentation, there is still a bug left, and that is that the package
builds fine even when this command fails.

At least that's not RC.

 Why this is so I don't know
offhand, but it might be that this (from GNUMakefile.in):

                for a in `find . -name out-www`; do \
                        rsync -a  --link-dest $$a/ $$a/ $$a/.. ; \
                        rm -rf $$a ; \
                done \

will not cause Make to fail when rsync is missing because only the last
command counts.  If that's true, then

rsync -a  --link-dest $$a/ $$a/ $$a/.. && rm -rf $$a ; \

might do (or not, since the for loop might still be successful if only
the first rsync call fails for some reason.

So then how about something like

 rsyncres=0 \
 for a in `find . -name out-www`; do \
   if rsync -a  --link-dest $$a/ $$a/ $$a/.. ; \
     then rm -rf $$a ; \
     else rsyncres=1; break; \
   fi; \
 done; \
 [ $$rsyncres -eq 0 ]

A bit messy, but should do it.

While using binary-indep, I noticed that there are some files left in
debian/tmp/.  Is this intended?

Thanks,
Yitz

Reply via email to