Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Luis Machado via Gdb-patches <gdb-patches@sourceware.org>
To: Simon Marchi <simon.marchi@polymtl.ca>, gdb-patches@sourceware.org
Subject: Re: [PATCH] gdb, gdbserver: remove WinCE support code
Date: Mon, 12 Apr 2021 11:07:59 -0300	[thread overview]
Message-ID: <7a0f467b-da91-5ae2-4c18-d38287a4a5f4@linaro.org> (raw)
In-Reply-To: <19a7df6c-614b-94c4-f8c3-1889a1f00199@polymtl.ca>

On 4/12/21 11:01 AM, Simon Marchi wrote:
> On 2021-04-12 9:57 a.m., Luis Machado wrote:> On 4/10/21 8:10 PM, Simon Marchi via Gdb-patches wrote:
>>> The support for WinCE was removed with commit 84b300de3666 ("gdbserver:
>>> remove support for ARM/WinCE").  There is some leftover code for WinCE
>>> support, guarded by the _WIN32_WCE macro, which I didn't know of at the
>>> time.
>>>
>>> I didn't remove the _WIN32_WCE references in the tests, because in
>>> theory we still support the WinCE architecture in GDB (when debugging
>>> remotely).  So someone could run a test with that (although I'd be
>>> really surprised).
>>
>> Is there value in supporting remote-based debugging of WinCE if we're dropping gdbserver support? Should we also drop the GDB and testing parts given this won't likely be exercised anytime soon if there's no maintainer?
> 
> I don't think so.  After doing that cleanup of stale arches in
> GDBserver, I wanted to do the same for GDB, but didn't really finish it.

Understood. I'm not saying you should do the work, but it is good to 
know that it can be done by whoever has some spare cycles to come up 
with the cleanup (myself included).

  reply	other threads:[~2021-04-12 14:08 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-10 23:10 Simon Marchi via Gdb-patches
2021-04-12 11:42 ` Pedro Alves
2021-04-12 15:12   ` Simon Marchi via Gdb-patches
2021-04-12 15:24     ` Simon Marchi via Gdb-patches
2021-04-12 22:20       ` [PATCH] Remove process_stratum_target::hostio_last_error abstraction (Re: [PATCH] gdb, gdbserver: remove WinCE support code) Pedro Alves
2021-04-12 22:52         ` Pedro Alves
2021-04-12 23:56         ` Simon Marchi via Gdb-patches
2021-04-13 13:12           ` Pedro Alves
2021-04-12 13:57 ` [PATCH] gdb, gdbserver: remove WinCE support code Luis Machado via Gdb-patches
2021-04-12 14:01   ` Simon Marchi via Gdb-patches
2021-04-12 14:07     ` Luis Machado via Gdb-patches [this message]
2021-04-13 19:29 ` Tom Tromey
2021-04-13 21:14   ` Pedro Alves
2021-04-13 21:20     ` [pushed] Remove WinCE code from gdbreplay Pedro Alves

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=7a0f467b-da91-5ae2-4c18-d38287a4a5f4@linaro.org \
    --to=gdb-patches@sourceware.org \
    --cc=luis.machado@linaro.org \
    --cc=simon.marchi@polymtl.ca \
    /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