public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Mike Stump <mikestump@comcast.net>
To: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Cc: Mike Stump via Gcc-patches <gcc-patches@gcc.gnu.org>,
	Hans-Peter Nilsson <hp@axis.com>,
	jwakely@redhat.com, Thomas Schwinge <thomas@codesourcery.com>
Subject: Re: Splitting up 27_io/basic_istream/ignore/wchar_t/94749.cc (takes too long)
Date: Mon, 12 Jun 2023 15:26:35 -0700	[thread overview]
Message-ID: <30721EE6-4503-44FE-B0A4-28F9F92585F0@comcast.net> (raw)
In-Reply-To: <20230612103506.0d9842fe@nbbrfq.loc>

On Jun 12, 2023, at 1:35 AM, Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> wrote:
> 
> On Sat, 10 Jun 2023 11:29:36 -0700
> Mike Stump <mikestump@comcast.net> wrote:
> 
>> On Jun 9, 2023, at 2:47 PM, Bernhard Reutner-Fischer
>> <rep.dot.nop@gmail.com> wrote:
> 
>>>                                    But well. Either way, what
>>> should we do about remote env, if there is one? If the target
>>> supports it, send it and skip otherwise?  
> 
>> So, to focus a
>> conversation, which target, which host, canadian? Which part of the
>> environment? What part is missing you want to fix? Want to unify
>> between targets... and so on.
>> 
> 
> The most recent target where this came up again was GCN i think.
> See the last block in
> https://inbox.sourceware.org/gcc-patches/20230508195217.4897009f@nbbrfq/
> and Thomas' reference therein to Tobias'
> https://inbox.sourceware.org/gcc-patches/018bcdeb-b3bb-1859-cd0b-a8a92e26d3a0@codesourcery.com/
> 
> thoughts?

I kinds like the remote_setenv approach, but the low level details when one gets in there and wires up all they need to make it work are important.

If someone wants to tilt at the problem, I'm inclined to let them find the incantation they like and approve it.

I'd rather approve a patch that takes us a step closer to perfection then go another 10 years.  For quoting. I like quite using ' if there is no ' in the string.  For each ' in the string, replace it with '\'', inelegant, but easier to reason about and don't have to worry about as many meta characters if you try and go the "" route.

This assumes something like sh is taking in commands.  This isn't always the case.  One can canadian into a windows box or some other weird thing so the mechanism used has to be next to the thing that knows what it is talking to.

Just because someone contributes code for talking to sh, doesn't mean they have to bother with windows or other weird stuff.  The next person to come along can tilt at that.


  reply	other threads:[~2023-06-12 22:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-09 16:20 Hans-Peter Nilsson
2023-06-09 17:18 ` Mike Stump
2023-06-09 17:49   ` Hans-Peter Nilsson
2023-06-09 21:47   ` Bernhard Reutner-Fischer
2023-06-10 18:29     ` Mike Stump
2023-06-12  8:35       ` Bernhard Reutner-Fischer
2023-06-12 22:26         ` Mike Stump [this message]
2023-06-09 20:40 ` Jonathan Wakely
2023-06-10  4:05   ` [PATCH] (Re: Splitting up 27_io/basic_istream/ignore/wchar_t/94749.cc (takes too long)) Hans-Peter Nilsson
2023-06-10  7:12     ` Jonathan Wakely

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=30721EE6-4503-44FE-B0A4-28F9F92585F0@comcast.net \
    --to=mikestump@comcast.net \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=hp@axis.com \
    --cc=jwakely@redhat.com \
    --cc=rep.dot.nop@gmail.com \
    --cc=thomas@codesourcery.com \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).