On Tue, Aug 13, 2002 at 12:28:06PM +0200,  via RT wrote:

> Line 14 in util/pod2mantest should read:
> 
> try_without_dir=true
> 
> otherwise 'first iteration' in the for-loop is never executed.

The code as it currently is doesn't make too much sense
(try_without_dir could be totally abolished), but the change
from
     try_without_dir=true
to
     try_without_dir=flase
was intentional: The reason was we want the complete path
to pod2man, not just the string 'pod2man'.

The reason for this is that the top-level Makefile calls pod2man
indirectly, via perl:

        @pod2man=`cd util; ./pod2mantest ignore`; \
        for i in doc/apps/*.pod; do \
                fn=`basename $$i .pod`; \
                if [ "$$fn" = "config" ]; then sec=5; else sec=1; fi; \
                echo "installing man$$sec/`basename $$i .pod`.$$sec"; \
                (cd `$(PERL) util/dirname.pl $$i`; \
                sh -c "$(PERL) $$pod2man \
                       ^^^^^^^^^^^^^^^^^
                        --section=$$sec --center=OpenSSL \
                        --release=$(VERSION) `basename $$i`") \
                        >  $(INSTALL_PREFIX)$(MANDIR)/man$$sec/`basename $$i 
.pod`.$$sec; \
        done; \

This does not make much sense, though -- if pod2mantest manages to run
.../pod2man directly, why call 'perl .../pod2man' instead?

The one case where $(PERL) is really needed in the invocation of
pod2man is when pod2mantest has not found a working pod2man (in this
case util/pod2man.pl is used instead, and it may not be able
to find the correct version of perl).

I think this should be resolved as follows:

- Switch back to try_without_dir=true
- Don't explicitly invoke $(PERL) in the Makefile when running pod2man
- but give util/pod2mantest an argument so that it can point to perl if needed.


-- 
Bodo M�ller <[EMAIL PROTECTED]>
PGP http://www.informatik.tu-darmstadt.de/TI/Mitarbeiter/moeller/0x36d2c658.html
* TU Darmstadt, Theoretische Informatik, Alexanderstr. 10, D-64283 Darmstadt
* Tel. +49-6151-16-6628, Fax +49-6151-16-6036

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to