On Tue, May 07, 2002 at 05:10:40PM +0100, Ben Laurie wrote:
> [EMAIL PROTECTED] wrote:
> > 
> > jaenicke    07-May-2002 17:35:18
> > 
> >   Modified:    .        Tag: OpenSSL_0_9_7-stable Makefile.org
> >   Log:
> >   Add missing ";" after fi
> >   Submitted by: [EMAIL PROTECTED]
> >   PR: [openssl.org #18]
> > 
> >   Revision  Changes    Path
> >   No                   revision
> >   No                   revision
> >   1.154.2.4 +3 -3      openssl/Makefile.org
> > 
> >   Index: Makefile.org
> >   ===================================================================
> >   RCS file: /e/openssl/cvs/openssl/Makefile.org,v
> >   retrieving revision 1.154.2.3
> >   retrieving revision 1.154.2.4
> >   diff -u -r1.154.2.3 -r1.154.2.4
> >   --- Makefile.org      2002/04/13 12:28:49     1.154.2.3
> >   +++ Makefile.org      2002/05/07 15:35:09     1.154.2.4
> >   @@ -697,8 +697,8 @@
> >                         cp $$i $(INSTALL_PREFIX)$(INSTALLTOP)/lib; \
> >                         $(RANLIB) $(INSTALL_PREFIX)$(INSTALLTOP)/lib/$$i; \
> >                         chmod 644 $(INSTALL_PREFIX)$(INSTALLTOP)/lib/$$i ); \
> >   -             fi \
> >   -     done
> >   +             fi; \
> >   +     done;
> 
> I can't believe this final ; is required!

The ; after the fi is required, because it is a continuation line due to
the \ (... fi ; done)
The ; after the done should not be required. We do not handle this
consistently ourselves. At several locations there is a trailing ";"
that should not be needed, at several other locations it is not...

Best regards,
        Lutz
-- 
Lutz Jaenicke                             [EMAIL PROTECTED]
http://www.aet.TU-Cottbus.DE/personen/jaenicke/
BTU Cottbus, Allgemeine Elektrotechnik
Universitaetsplatz 3-4, D-03044 Cottbus
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to