Re: HEAD compile error on Alpha Linux and Compaq C Compiler - SOLVED - [PATCH]

HI Henrik,

Il 22.29 28/12/2001 Henrik Nordstrom ha scritto:
>Hmm.. should be fixed in configure I think, if at all. The problem is that
>your development environment have a byteswap.h header that cannot be used.
>
>To work around the problem, tell configure that byteswap.h does not exists:
>
> ac_cv_header_byteswap_h=no ./configure ...
>
>Or if you want to test it proper, add a configure test that verifies if
>byteswap.h can really be used, not only if it exists or not.

Sure, is the right and more elegant solution, but is this a good sense work ?

Alpha is a low spread Linux port and Compaq C compiler for Linux is used
very rarely, plus, Compaq has just announced the death of Alpha processor
......
The future "MUST" be Itanium (Bleaaahhhhh) ....

>The long term fix is to get byteswap.h to work using the Digital/Compaq
>compiler..

OK, but the not working include file is a gcc include file used from ccc,
so, I think that ask to the gcc team to change this file can be ...
blasphemous .....

Guido

>Regards
>Henrik
>
>
>On Friday 28 December 2001 17.57, Guido Serassio wrote:
> > Hi,
> >
> > I just found a minor issue on HEAD:
> >
> > On my Alpha running RHL 7.0, I can't compile HEAD using the ccc Compaq C
> > Compiler for Linux.
> >
> > Compaq C Compiler for Alpha Linux (ccc) use gcc libraries and include
> > files. It seem's that it doesn't like byteswap.h included in ntlmauth.h
> >
> > Applying the following patch, all compiles and seems to work fine.
> >
> > Index: include/ntlmauth.h
> > ===================================================================
> > RCS file: /cvsroot/squid/squid/include/ntlmauth.h,v
> > retrieving revision 1.9
> > diff -u -p -r1.9 ntlmauth.h
> > --- include/ntlmauth.h 2001/11/13 22:19:32 1.9
> > +++ include/ntlmauth.h 2001/12/28 16:53:53
> > @@ -65,7 +65,7 @@
> > #define WSWAP(x) (x)
> > #endif
> >
> > -#ifdef HAVE_BYTESWAP_H
> > +#if defined HAVE_BYTESWAP_H && !(defined(__DECC) && defined(__linux__) &&
> > defined(__alpha__))
> > #include <byteswap.h>
> > #define bswap16(x) bswap_16(x)
> > #define bswap32(x) bswap_32(x)
> >
> > If this is OK, can anybody merge it in HEAD ?
> >
> > Guido
> >
> >
> >
> > -
> > =======================================================
> > Serassio Guido
> > Via Albenga, 11/4 10134 - Torino -
> > ITALY E-mail: serassio@interfree.it
> > serassio@libero.it
> > WWW: http://www.serassio.it

-
=======================================================
Serassio Guido
Via Albenga, 11/4 10134 - Torino - ITALY
E-mail: serassio@interfree.it
         serassio@libero.it
WWW: http://www.serassio.it
Received on Fri Dec 28 2001 - 15:06:04 MST