RE: [ECOS] RE: GoAhead Web Server

RE: [ECOS] RE: GoAhead Web Server

[amassa]

I am also using version 2.1.


I think the redefinition of NFDBITS is because when it compiles it uses the
definition from sys/bsdtypes.h as well as seeing it in the new definition I
added via the patch.


That's scarry if the same version of the web server has different source. 
This line refers to default.c line 92 which is:

if (websPageOpen(wp, lpath, path, SOCKET_RDONLY | SOCKET_BINARY,

and SOCKET_RDONLY is defined in wsIntrn.h as O_RDONLY, which is the reason
for the conflict.  Do you have O_RDONLY defined somewhere else?




Gary Thomas <gthomas@redhat.com> writes:
>Exactly what version of the GoAhead server do you have?  My patches were
>against the 2.1 version, dated 19 July.
>
>On 26-Oct-2000 amassa@cts.com wrote:
>> powerpc-eabi-gcc -c -o ../default.o -mcpu=860 -msoft-float -g -Wall -O2
>> -DB_STAT
>> S -DB_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM
>> -DOS="eCos" -D
>> ECOS -D__ECOS -D__NO_FCNTL=1 -I.. -Wall
>> -I//d/projects/blackbird/tornado/softwar
>> e/ecoslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
>> -fdata-se
>> ctions -Wp,-MD,../default.d ../default.c
>> In file included from ../wsIntrn.h:113,
>>                  from ../default.c:23:
>> ../uemf.h:216: warning: `NFDBITS' redefined
>> ../uemf.h:151: warning: this is the location of the previous definition
>
>This one shouyldn't hurt anything, but you could check the file "uemf.h"
>and see why it's being defined twice.
>
>> ../default.c: In function `websDefaultHandler':
>> ../default.c:92: `O_RDONLY' undeclared (first use in this function)
>> ../default.c:92: (Each undeclared identifier is reported only once
>> ../default.c:92: for each function it appears in.)
>
>You're on your own for this one - this line does not appear in the code
>I obtained from GoAhead.

RE: [ECOS] RE: GoAhead Web Server

[Gary Thomas]

On 26-Oct-2000 amassa@cts.com wrote:
> I am also using version 2.1.
> 
> 
> I think the redefinition of NFDBITS is because when it compiles it uses the
> definition from sys/bsdtypes.h as well as seeing it in the new definition I
> added via the patch.
> 
> 
> That's scarry if the same version of the web server has different source. 
> This line refers to default.c line 92 which is:
> 
> if (websPageOpen(wp, lpath, path, SOCKET_RDONLY | SOCKET_BINARY,
> 
> and SOCKET_RDONLY is defined in wsIntrn.h as O_RDONLY, which is the reason
> for the conflict.  Do you have O_RDONLY defined somewhere else?
>

No, just make it 0x0.  It really doesn't matter anyway since the only
"files" supported are compiled in, so permissions are irrelevant.
 
> 
> 
> 
> Gary Thomas <gthomas@redhat.com> writes:
>>Exactly what version of the GoAhead server do you have?  My patches were
>>against the 2.1 version, dated 19 July.
>>
>>On 26-Oct-2000 amassa@cts.com wrote:
>>> powerpc-eabi-gcc -c -o ../default.o -mcpu=860 -msoft-float -g -Wall -O2
>>> -DB_STAT
>>> S -DB_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM
>>> -DOS="eCos" -D
>>> ECOS -D__ECOS -D__NO_FCNTL=1 -I.. -Wall
>>> -I//d/projects/blackbird/tornado/softwar
>>> e/ecoslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
>>> -fdata-se
>>> ctions -Wp,-MD,../default.d ../default.c
>>> In file included from ../wsIntrn.h:113,
>>>                  from ../default.c:23:
>>> ../uemf.h:216: warning: `NFDBITS' redefined
>>> ../uemf.h:151: warning: this is the location of the previous definition
>>
>>This one shouyldn't hurt anything, but you could check the file "uemf.h"
>>and see why it's being defined twice.
>>
>>> ../default.c: In function `websDefaultHandler':
>>> ../default.c:92: `O_RDONLY' undeclared (first use in this function)
>>> ../default.c:92: (Each undeclared identifier is reported only once
>>> ../default.c:92: for each function it appears in.)
>>
>>You're on your own for this one - this line does not appear in the code
>>I obtained from GoAhead.

RE: [ECOS] RE: GoAhead Web Server

[Gary Thomas]

Exactly what version of the GoAhead server do you have?  My patches were
against the 2.1 version, dated 19 July.

On 26-Oct-2000 amassa@cts.com wrote:
> powerpc-eabi-gcc -c -o ../default.o -mcpu=860 -msoft-float -g -Wall -O2
> -DB_STAT
> S -DB_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM
> -DOS="eCos" -D
> ECOS -D__ECOS -D__NO_FCNTL=1 -I.. -Wall
> -I//d/projects/blackbird/tornado/softwar
> e/ecoslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
> -fdata-se
> ctions -Wp,-MD,../default.d ../default.c
> In file included from ../wsIntrn.h:113,
>                  from ../default.c:23:
> ../uemf.h:216: warning: `NFDBITS' redefined
> ../uemf.h:151: warning: this is the location of the previous definition

This one shouyldn't hurt anything, but you could check the file "uemf.h"
and see why it's being defined twice.

> ../default.c: In function `websDefaultHandler':
> ../default.c:92: `O_RDONLY' undeclared (first use in this function)
> ../default.c:92: (Each undeclared identifier is reported only once
> ../default.c:92: for each function it appears in.)

You're on your own for this one - this line does not appear in the code
I obtained from GoAhead.

[ECOS] RE: GoAhead Web Server

[amassa]

I added those changes and I get the following errors/warnings when I try to
compile the goahead webserver.  Could you take a look and let me know if I
applied something wrong or what?

Thanks.


D:\Projects\Blackbird\Tornado\software\source\websvr\ecos>make
powerpc-eabi-gcc -c -o main.o -mcpu=860 -msoft-float -g -Wall -O2 -DB_STATS
-DB_
VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM -DOS="eCos"
-DECOS -
D__ECOS -D__NO_FCNTL=1 -I.. -Wall
-I//d/projects/blackbird/tornado/software/ecos
latest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
-fdata-sections
 -Wp,-MD,main.d main.c
In file included from main.c:18:
../uemf.h:216: warning: `NFDBITS' redefined
../uemf.h:151: warning: this is the location of the previous definition
powerpc-eabi-gcc -c -o ../asp.o -mcpu=860 -msoft-float -g -Wall -O2
-DB_STATS -D
B_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM -DOS="eCos"
-DECOS
-D__ECOS -D__NO_FCNTL=1 -I.. -Wall
-I//d/projects/blackbird/tornado/software/ec
oslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
-fdata-sectio
ns -Wp,-MD,../asp.d ../asp.c
In file included from ../wsIntrn.h:113,
                 from ../asp.c:19:
../uemf.h:216: warning: `NFDBITS' redefined
../uemf.h:151: warning: this is the location of the previous definition
powerpc-eabi-gcc -c -o ../balloc.o -mcpu=860 -msoft-float -g -Wall -O2
-DB_STATS
-DB_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM -DOS="eCos"
-DE
COS -D__ECOS -D__NO_FCNTL=1 -I.. -Wall
-I//d/projects/blackbird/tornado/software
/ecoslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
-fdata-sec
tions -Wp,-MD,../balloc.d ../balloc.c
In file included from ../balloc.c:30:
../uemf.h:216: warning: `NFDBITS' redefined
../uemf.h:151: warning: this is the location of the previous definition
../balloc.c: In function `balloc':
../balloc.c:380: warning: unsigned int format, pointer arg (arg 3)
powerpc-eabi-gcc -c -o ../base64.o -mcpu=860 -msoft-float -g -Wall -O2
-DB_STATS
-DB_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM -DOS="eCos"
-DE
COS -D__ECOS -D__NO_FCNTL=1 -I.. -Wall
-I//d/projects/blackbird/tornado/software
/ecoslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
-fdata-sec
tions -Wp,-MD,../base64.d ../base64.c
In file included from ../wsIntrn.h:113,
                 from ../base64.c:17:
../uemf.h:216: warning: `NFDBITS' redefined
../uemf.h:151: warning: this is the location of the previous definition
powerpc-eabi-gcc -c -o ../default.o -mcpu=860 -msoft-float -g -Wall -O2
-DB_STAT
S -DB_VERIFY_CAUSES_SEVERE_OVERHEAD -DWEBS -DUEMF -DWEBS_PAGE_ROM
-DOS="eCos" -D
ECOS -D__ECOS -D__NO_FCNTL=1 -I.. -Wall
-I//d/projects/blackbird/tornado/softwar
e/ecoslatest/ecosnetMBX/ecosnetMBX_install/include -ffunction-sections
-fdata-se
ctions -Wp,-MD,../default.d ../default.c
In file included from ../wsIntrn.h:113,
                 from ../default.c:23:
../uemf.h:216: warning: `NFDBITS' redefined
../uemf.h:151: warning: this is the location of the previous definition
../default.c: In function `websDefaultHandler':
../default.c:92: `O_RDONLY' undeclared (first use in this function)
../default.c:92: (Each undeclared identifier is reported only once
../default.c:92: for each function it appears in.)
make: *** [../default.o] Error 0x1