All,

On 2007-02-21, al2.osd@gmail.com wrote:
> Hi
>
> I've tied to build Minix from CVS and achieved such error
>
> $ pwd
> /rep/src/servers/fs
>
> $ make
> exec cc -c -I/usr/include select.c
> "select.c", line 172: call of non-function (int)
> "select.c", line 230: call of non-function (int)
> make in /export/repsrc/src/servers/fs: Exit code 1
>
> 172: fd_setsize = _FDSETWORDS(e->nfds)*_FDSETBITSPERWORD/8;
> after PP:
> fd_setsize = (( 30 + (sizeof(fd_mask)*8) -1)/
> (sizeof(fd_mask)*8) ) (e->nfds)* (sizeof(fd_mask)*8) /8;
>
> 230: fd_setsize = _FDSETWORDS(nfds)*_FDSETBITSPERWORD/8;
> after PP:
> fd_setsize = (( 30 + (sizeof(fd_mask)*8) -1)/
> (sizeof(fd_mask)*8) ) (nfds)* (sizeof(fd_mask)*8) /8;
>
> Obviously it is wrong but what is problem? And what should I use
> instead.

_FDSETWORDS is being expanded as a constant instead of a macro that
depends on the size of the fd set. This was changed at some point.
Chances are you didn't isntall a new copy of the include files after
updating; try 'make install' in /usr/src/include. (Or /rep/src/include,
as the case may be.)

=Ben

this is definition of _FDSETWORDS from CVS head
(command: "cvs -d server:anonymous@derelict.cs.vu.nl:/cvsup/minix co
src")

#define _FDSETWORDS ((FD_SETSIZE+_FDSETBITSPERWORD-1)/
_FDSETBITSPERWORD)

On Feb 21, 4:59 pm, Ben Gras wrote:
> All,
>
> On 2007-02-21, al2....@gmail.com wrote:
>
>
>
> > Hi
>
> > I've tied to build Minix from CVS and achieved such error
>
> > $ pwd
> > /rep/src/servers/fs
>
> > $ make
> > exec cc -c -I/usr/include select.c
> > "select.c", line 172: call of non-function (int)
> > "select.c", line 230: call of non-function (int)
> > make in /export/repsrc/src/servers/fs: Exit code 1
>
> > 172: fd_setsize = _FDSETWORDS(e->nfds)*_FDSETBITSPERWORD/8;
> > after PP:
> > fd_setsize = (( 30 + (sizeof(fd_mask)*8) -1)/
> > (sizeof(fd_mask)*8) ) (e->nfds)* (sizeof(fd_mask)*8) /8;
>
> > 230: fd_setsize = _FDSETWORDS(nfds)*_FDSETBITSPERWORD/8;
> > after PP:
> > fd_setsize = (( 30 + (sizeof(fd_mask)*8) -1)/
> > (sizeof(fd_mask)*8) ) (nfds)* (sizeof(fd_mask)*8) /8;
>
> > Obviously it is wrong but what is problem? And what should I use
> > instead.
>
> _FDSETWORDS is being expanded as a constant instead of a macro that
> depends on the size of the fd set. This was changed at some point.
> Chances are you didn't isntall a new copy of the include files after
> updating; try 'make install' in /usr/src/include. (Or /rep/src/include,
> as the case may be.)
>
> =Ben

All,

On 2007-02-21, al2.osd@gmail.com wrote:
> this is definition of _FDSETWORDS from CVS head
> (command: "cvs -d server:anonymous@derelict.cs.vu.nl:/cvsup/minix co
> src")
>
> #define _FDSETWORDS ((FD_SETSIZE+_FDSETBITSPERWORD-1)/
> _FDSETBITSPERWORD)

It seems you're right. Maybe the CVS repository is inconsistent.

But the CVS repository is obsolete anyway, as we've switched to SVN.
More information about where it is and so on is on the 'tracking
current' page on the website.

=Ben