GNU bug report logs - #18237
Small fix for MSDOS

Previous Next

Package: emacs;

Reported by: Reuben Thomas <rrt <at> sc3d.org>

Date: Sun, 10 Aug 2014 16:41:03 UTC

Severity: normal

Done: Reuben Thomas <rrt <at> sc3d.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Reuben Thomas <rrt <at> sc3d.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#18237: closed (Small fix for MSDOS)
Date: Sun, 10 Aug 2014 18:43:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Sun, 10 Aug 2014 19:42:14 +0100
with message-id <CAOnWdogqJPL_QoooJRrSQdah-WdGqvKxtmqT9NtbowVuKQW9uQ <at> mail.gmail.com>
and subject line Re: bug#18237: Small fix for MSDOS
has caused the debbugs.gnu.org bug report #18237,
regarding Small fix for MSDOS
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
18237: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=18237
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Reuben Thomas <rrt <at> sc3d.org>
To: bug-emacs <bug-emacs <at> gnu.org>
Subject: Small fix for MSDOS
Date: Sun, 10 Aug 2014 17:40:24 +0100
[Message part 3 (text/plain, inline)]
The following patch helps in the case when a sockets library is installed;
in my case, ls080b.zip from the DJGPP site:

=== modified file 'src/msdos.c'
--- src/msdos.c    2014-08-09 16:12:33 +0000
+++ src/msdos.c    2014-08-09 21:51:56 +0000
@@ -4016,7 +4017,7 @@
 #endif


-#ifndef HAVE_SELECT
+#ifndef HAVE_SYS_SELECT_H
 #include "sysselect.h"

 /* This yields the rest of the current time slice to the task manager.


The reasoning is that having select doesn't imply having the sys/select.h
functionality that the following code emulates.

Does that look OK to install?

-- 
http://rrt.sc3d.org
[Message part 4 (text/html, inline)]
[Message part 5 (message/rfc822, inline)]
From: Reuben Thomas <rrt <at> sc3d.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 18237-done <at> debbugs.gnu.org
Subject: Re: bug#18237: Small fix for MSDOS
Date: Sun, 10 Aug 2014 19:42:14 +0100
[Message part 6 (text/plain, inline)]
On 10 August 2014 19:26, Eli Zaretskii <eliz <at> gnu.org> wrote:

>
> I'd prefer that all the changes for such a build be committed at once
> together, not bit by bit, and only if you eventually succeed to
> produce a working binary using this method.
>

Sorry, I was trying to separate my patch into parts with different
implications (e.g. no effect on the current DOS build system; some effect
on that system but not more generally; more general changes); but instead
I'll concentrate on producing a single patch that, as you say, results in a
working binary.


>  > I think you're also saying that I should test both HAVE_SELECT &&
> > HAVE_SYS_SELECT_H therefore?
>
> Yes, at the very least.


Noted.


>  But let's delay this until you have gone all
> the way to building Emacs using this method.  We won't know all of the
> implications until you do.
>

Sure, OK.

I am closing this bug; since you diagnosed my other patch submission as a
more general bug in configuration, I'll look into a fix for it at that
level.

-- 
http://rrt.sc3d.org
[Message part 7 (text/html, inline)]

This bug report was last modified 10 years and 344 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.