Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: mark@codesourcery.com
Cc: eliz@gnu.org, drow@false.org, me@cgf.cx, gdb-patches@sources.redhat.com
Subject: Re: PATCH: Support Windows in event-loop.c
Date: Mon, 25 Apr 2005 22:29:00 -0000	[thread overview]
Message-ID: <200504252228.j3PMS8iR028118@elgar.sibelius.xs4all.nl> (raw)
In-Reply-To: <426D6A6E.1020505@codesourcery.com> (message from Mark Mitchell on Mon, 25 Apr 2005 15:08:46 -0700)

   Date: Mon, 25 Apr 2005 15:08:46 -0700
   From: Mark Mitchell <mark@codesourcery.com>

   Mark Kettenis wrote:

   >    Thanks, committed.
   > 
   > Thanks!  That was the last patch, isn't it?  If so, can you add some
   > blurp to NEWS about it?

   Not *quite* the last patch.  There are still readline changes required. 
     By and large, I should think these will be rather less controversial; 
   most of them are just using autoconf macros to avoid using stuff that 
   Windows doesn't have.  There is also one patch that basically says 
   "don't use any of that termcap stuff" -- but that's really just a 
   Makefile patch not to link in one file under Windows.

   I understand that the readline used by GDB contains some GDB-local 
   patches.  So, I'm planning to submit my patches to this list, too.  I 
   know that normally we'd want to go to the upstream source, but Daniel 
   says that's probably not practical in this case.

   So, is it OK to submit readline patches here?

I'd say, please make an effort to get them into the official readline
first.  While we have some local readline patches, we should really
try to keep the number of them to the minimum.  Every local patch
makes merging the next version of readline more difficult :-(.  Once
the stuff is accepted upstream, no one should stop you from importing
those changes into GDB's readline.

If that really is not practical, try to keep the impact on existing
files as minimal as possible.  Try to keep things localized,
preferably in a seperate source files.  That will make life much
easier for us in the future.

Mark


  reply	other threads:[~2005-04-25 22:29 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-21  5:53 Mark Mitchell
2005-04-21 18:46 ` Eli Zaretskii
2005-04-21 18:49   ` Daniel Jacobowitz
2005-04-21 18:56   ` Mark Mitchell
2005-04-21 20:30     ` Eli Zaretskii
2005-04-21 20:56       ` Daniel Jacobowitz
2005-04-21 21:15         ` Mark Mitchell
2005-04-22  8:26           ` Eli Zaretskii
2005-04-22 12:08             ` Christopher Faylor
2005-04-22 13:23               ` Eli Zaretskii
2005-04-22 15:04                 ` Christopher Faylor
2005-04-22 15:14                   ` Ian Lance Taylor
2005-04-22 15:28                     ` Christopher Faylor
2005-04-22 15:52               ` Mark Mitchell
2005-04-22  8:16         ` Eli Zaretskii
2005-04-24 22:18           ` Daniel Jacobowitz
2005-04-24 22:30             ` Mark Kettenis
2005-04-25  0:04               ` Mark Mitchell
2005-04-24 23:57             ` Mark Mitchell
2005-04-25  4:25               ` Christopher Faylor
2005-04-25 13:16                 ` Daniel Jacobowitz
2005-04-25 14:50                   ` Christopher Faylor
2005-04-25 14:59                     ` Mark Mitchell
2005-04-25 15:04                       ` Daniel Jacobowitz
2005-04-25 15:18                         ` Mark Mitchell
2005-04-25 15:23                           ` Daniel Jacobowitz
2005-04-25 15:26                             ` Mark Mitchell
2005-04-25 15:36                               ` Daniel Jacobowitz
2005-04-25 16:44                                 ` Eli Zaretskii
2005-04-25 20:23                                 ` Mark Mitchell
2005-04-25 21:07                                   ` Eli Zaretskii
2005-04-25 21:49                                     ` Mark Mitchell
2005-04-25 22:00                                       ` Mark Kettenis
2005-04-25 22:09                                         ` Mark Mitchell
2005-04-25 22:29                                           ` Mark Kettenis [this message]
2005-04-25 22:47                                             ` Mark Mitchell
2005-04-26  3:55                                             ` Eli Zaretskii
2005-04-25 23:16                                       ` Christopher Faylor
2005-04-25 23:20                                         ` Mark Mitchell
2005-04-25 23:33                                           ` Christopher Faylor
2005-04-26  0:21                                             ` Mark Mitchell
2005-04-26  3:58                                           ` Eli Zaretskii
2005-04-26  3:59                                             ` Mark Mitchell
2005-04-25 15:50                               ` Ian Lance Taylor
2005-04-26  3:49                         ` Eli Zaretskii
2005-04-26 13:17                           ` Daniel Jacobowitz
2005-04-25 15:52                     ` M.M. Kettenis
2005-04-25 16:00                       ` Daniel Jacobowitz
2005-04-25 16:08                       ` Ian Lance Taylor
2005-04-25 16:24                         ` Christopher Faylor
2005-04-25 17:08                       ` Mark Mitchell
2005-04-25 21:04                         ` Eli Zaretskii
2005-04-25 16:42                     ` Eli Zaretskii
2005-04-21 21:01     ` Christopher Faylor
2005-04-21 21:03       ` Christopher Faylor

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200504252228.j3PMS8iR028118@elgar.sibelius.xs4all.nl \
    --to=mark.kettenis@xs4all.nl \
    --cc=drow@false.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mark@codesourcery.com \
    --cc=me@cgf.cx \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox