[OpenAFS-devel] pod2man vandalism

[Russ Allbery]

Jim Rees <rees@umich.edu> writes:

> I'm not convinced that pod2man should be done from regen.sh.  I'd be
> happier if the docs were a target in some Makefile.  Maybe regen.sh
> should go away and be replaced by some kind of pre-configure Makefile.

I'd like to do that in the long run.

> The vandalism appears to be due to a difference between my OpenBSD
> pod2doc and the one that regen.sh is expecting to use.

Thanks for sending a trace.  The problem has nothing to do with pod2man
and is actually what I consider to be weird shell behavior, but I can work
around it.

> + basename doc/man-pages/pod1/*.pod
> + sed s/\.pod$//

I'm guessing that you don't have a doc tree in your checkout at all, and
therefore have no POD pages.  Your shell therefore left the wildcard alone
in the for statement rather than expanding it, basename reduced it to
*.pod, the sed removed the .pod part, and then pod2man ran on *.  It would
be more useful if the shell would expand the wildcard to an empty list in
this case, but there's probably some obscure bit of POSIX that requires a
different behavior.

I'll fix this to detect and avoid this pathological case.

-- 
Russ Allbery (rra@stanford.edu)             <http://www.eyrie.org/~eagle/>