RE: [ECOS] RE: GoAhead Web Server

[Gary Thomas <gthomas at redhat dot com>]

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.