emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Failure in building Emacs 23.0.50 on Cygwin (fwd)


From: Angelo Graziosi
Subject: Re: Failure in building Emacs 23.0.50 on Cygwin (fwd)
Date: Tue, 4 Sep 2007 22:41:40 +0200 (MET DST)

I have build emacs-cvs with my patch and with the nicer Darel's patch and
both builds work fine. Note that I use Emacs only under X11...


Cheers,

   Angelo.



On Tue, 4 Sep 2007, Dan Nicolaescu wrote:

> "d.henman" <address@hidden> writes:
>
>   > Angelo,
>   >
>   > content wise it looks good to me, but I think that changing it to the 
> following
>   > format, same content as your patch (see modifed patch below ), but this 
> format makes it easier to understand and would give it more robustness for 
> future mainenance and any possible changes and extensions for CYGWIN specific 
> issues.
>   >
>   > Also I would like to wait and get a response from the cygwin developers, 
> before any commit is made.
>   >
>   > Regards,
>   >   Darel Henman
>   >
>   > ----------------------------------------------------------------
>   > --- term.c        2007-08-30 06:46:50.000000000 +0900
>   > +++ /tmp/term.c.fixed     2007-09-03 09:15:22.915209300 +0900
>   > @@ -3131,6 +3131,10 @@
>   >        setpgrp ();
>   >        no_controlling_tty = 1;
>   >  #else
>   > +#ifdef CYGWIN
>   > +      setsid ();
>   > +      no_controlling_tty = 1;
>   > +#else
>   >  #ifdef TIOCNOTTY                /* Try BSD ioctls. */
>   >        sigblock (sigmask (SIGTTOU));
>   >        fd = emacs_open ("/dev/tty", O_RDWR, 0);
>   > @@ -3145,6 +3149,7 @@
>   >        /* Unknown system. */
>   >        croak ();
>   >  #endif  /* ! TIOCNOTTY */
>   > +#endif  /* ! CYGWIN */
>   >  #endif  /* ! USG */
>   >      }
>   >  #endif
>
>
> Just my curiosity, does this patch make emacs work on cygwin?
> If it doesn't, is this function called before things go bad?
>
> If you configure with X11 support, does the X11 version work?
> Theoretically it should work by default, but if it does not, fixing it
> might help find things that would help the -nw case...
>
>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]