On Sun, 8 Feb 2004, Luchezar Georgiev wrote:

> Thanks for applying my patches, Bart! I see you improved some of them -
> great! But I have to note that the above patch does NOT fix the EJECT
> bug! ONLY when I appied the patch at my
> http://linux.tu-varna.acad.bg/~lig/freedos/CVSPATCH.TXT to your latest
> CVS kernel was the EJECT bug fixed:
>
> --- cvs/kernel/kernel/dosfns.c        2004-01-27 20:07:26.000000000 +0200
> +++ src/kernel/kernel/dosfns.c        2004-02-05 15:36:08.000000000 +0200
> @@ -538,7 +538,7 @@
>     sftp->sft_shroff = -1;        /* /// Added for SHARE - Ron Cemer */
>     sftp->sft_attrib = attrib = attrib | D_ARCHIVE;
>
> -  if (result & IS_NETWORK)
> +  if ((result & (IS_NETWORK | IS_DEVICE)) == IS_NETWORK)
>     {
>       int status;
>       if ((flags & (O_TRUNC | O_CREAT)) == O_CREAT)

I didn't apply this patch since it is wrong: H:NUL should give H:/NUL in
truename which results in only the IS_DEVICE bit set (unless I missed a
bug in truename).

IS_NETWORK|IS_DEVICE both set is for things such as H:\DOS\NUL which
really should go via the network redirector to check if H:\DOS is valid!

Please see my explanation I sent Friday which showed up a little late for
me (but I sent it privately too!).

> As to the other funny printf "patch", it's only a temporary solution
> until I find the reason why Borland 80386 build hangs if any INSTALL=
> commands were processed, just before kernel() enters...

try to investigate if anything weird happens with the high parts of the
eax, ebx, ... registers. Borland may rely on them in the
multiplication/division routines (at least it does in turbo pascal; it
might have done the same for borland C too)

Bart



-------------------------------------------------------
The SF.Net email is sponsored by EclipseCon 2004
Premiere Conference on Open Tools Development and Integration
See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
http://www.eclipsecon.org/osdn
_______________________________________________
Freedos-kernel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/freedos-kernel

Reply via email to