public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jacob Bachmeyer <jcb62281@gmail.com>
To: Jacek Caban <jacek@codeweavers.com>
Cc: fortran@gcc.gnu.org, NightStrike <nightstrike@gmail.com>,
	 Eric Pouech <eric.pouech@orange.fr>,
	DejaGnu mailing list <dejagnu@gnu.org>
Subject: Re: testsuite under wine
Date: Wed, 21 Dec 2022 22:16:04 -0600	[thread overview]
Message-ID: <63A3DA04.4060804@gmail.com> (raw)
In-Reply-To: <7cb45ab2-cc6e-c502-5592-51ffabcbc6f8@codeweavers.com>

[Bringing the DejaGnu list back into the discussion...]

Jacek Caban wrote:
> Hi all,
>
>
> I'm responsible for Wine changes that cause your problems. I'm also 
> CCing Eric, who is Wine console expert, maybe he has better ideas. 
> Eric, see [1] if you're interested in the context.
>
>
> Recent Wine versions implement Windows pseudo-consoles, see [2] for a 
> bit more details. It's generally Microsoft's semi-recent API that's 
> intended to be more compatible with POSIX than what was present in 
> previous versions of Windows. In theory, with that implemented, we 
> could just plug tty fds that we get from Unix and have Unix consoles 
> working using those Windows APIs.

I would expect that to work very well with MinGW.

> In practice, it's not quite as good as promised and we need some 
> tweaks to make it work nicely. We could improve those tweaks, but 
> there are architectural limitations that will come to play sooner or 
> later.

Of course; "not quite as good as promised" seems to be a typical 
experience with Microsoft and their APIs.

> > I think that the long-term solution is that Wine should properly honor
> > the TERM environment variable and not produce escape codes that the
> > declared terminal does not support.
>
>
> I think that it would not be enough. The way Windows consoles work is 
> that we manage complete internal screen buffer and emit output that 
> synchronizes the buffer with Unix terminal inside conhost.exe process. 
> It means that its output heavily processed and may be very different 
> from what application writes to its console handle. While escape codes 
> discussed in this thread are the most prominent difference (and that 
> part could, in theory, be improved on our side), there are more 
> differences. For example, if application writes "\rA\rB\rC", conhost 
> will process it, update its internal buffer which changes just one 
> character and cursor position, and emit sequence to update it in Unix 
> terminal, which could be just "\rC" (or even "C" if cursor was already 
> at the beginning of the line). Another example would be long lines: 
> conhost will emit additional EOLs instead of depending on embedder to 
> wrap the line.

So conhost is essentially a Wine-specific screen(1) in that sense, 
except that it translates Windows screen buffer manipulations instead of 
VT100 escape codes?  As I understand ncurses also implements most of 
this; perhaps simply delegating output to ncurses would solve the 
problem?  If output were simply delegated to ncurses, (as I understand) 
setting TERM=dumb should be effective to eliminate escape codes from the 
output, since the "dumb" terminal does not support them.

Alternately, could we have a "transparent" mode in conhost where most 
processing is bypassed?  Setting TERM=dumb in the environment could 
reasonably activate this mode, or some other Wine-specific setting could 
be used.  (maybe "WINETERM=raw"?)

> I'm not really familiar with DejaGnu, but if you want to match 
> application output, that's probably not what you're looking for.

DejaGnu is mostly designed to drive text-mode interactive programs 
through simulated user interactions, for which Expect is extremely 
useful, although the tests at issue here are simpler than this general 
case.  Here, the compiler under test is used to build a test program, 
which is then run, its output collected, and that collected output 
compared to an expected pattern.  In Windows terms (as I remember), this 
is equivalent to writing out a batch file that compiles a test program, 
then runs the test program with output redirected to a temporary file, 
running that batch file, and reading/checking the output file.  POSIX 
allows DejaGnu to dispense with the batch file and the output temporary 
file.

(If you think these issues are fun, wait until someone tries to test a 
MinGW port of GDB...)

> The reason the previous workaround of compiling Wine without ncurses 
> worked is that if made Wine treat tty stdin/stdout in a way very 
> similar to regular pipes, so no extra processing was performed. This 
> was more of a side effect than a design choice. It should be possible 
> to provide some way to achieve that with the new Wine architecture. 
> I'm attaching an example of Wine patch that would allow it. With that 
> patch, you may disable conhost.exe (via winecfg or 
> WINEDLLOVERRIDES=conhost.exe=d environment variable) and achieve 
> something similar to previous workaround.

If Wine makes proper use of ncurses, then setting TERM=dumb should have 
the effect of eliminating escape codes from the output.  Why does it not 
do so?

> Long term, I think that it would be best to get rid of console 
> behaviour expectations by using Windows build of DejaGnu. My 
> understanding is that it requires Cygwin, so the stack would be: 
> Windows DejaGnu on Cygwin on Wine on Linux. This would make all 
> similar mismatches in expectations non-existent. Cygwin is tricky to 
> run on Wine, there are a few known problems like [3], but we're 
> getting there.

That could also introduce new problems, since it would be relying on 
Expect's Windows port.  (DejaGnu happens to work on Windows because 
Expect is portable to Windows, or at least Cygwin.)  There are some 
ongoing efforts to reduce gratuitous POSIX dependencies in DejaGnu, i.e. 
to use Tcl's portability aids where appropriate, mostly because using 
[file join $a $b $c] (for example) clearly indicates that $a, $b, and $c 
are file names, while simply [join [list $a $b $c] "/"] or "$a/$b/$c" 
says nothing about the meaning of the operation.

> Jacek
>
>
> [1] https://gcc.gnu.org/pipermail/fortran/2022-December/058645.html
>
> [2] 
> https://devblogs.microsoft.com/commandline/windows-command-line-introducing-the-windows-pseudo-console-conpty/ 
>
>
> [3] https://bugs.winehq.org/show_bug.cgi?id=47808


-- Jacob

  parent reply	other threads:[~2022-12-22  4:16 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-16  2:20 NightStrike
2022-12-16  6:44 ` Thomas Koenig
2022-12-17  0:26   ` NightStrike
2022-12-17 10:52     ` Thomas Koenig
2022-12-17 23:24       ` NightStrike
2022-12-18  3:44         ` Jacob Bachmeyer
2022-12-18 21:13           ` NightStrike
2022-12-19  4:29             ` Jacob Bachmeyer
2022-12-19 10:43               ` Torbjorn SVENSSON
2022-12-19 11:00                 ` NightStrike
2022-12-19 11:13               ` NightStrike
2022-12-20  3:51                 ` Jacob Bachmeyer
2022-12-21 17:37               ` Jacek Caban
2022-12-22  1:01                 ` NightStrike
2022-12-22  4:37                   ` Jacob Bachmeyer
2022-12-23 10:36                     ` NightStrike
2022-12-23 12:43                       ` Eric Pouech
2022-12-24  4:00                       ` Jacob Bachmeyer
2022-12-24 11:05                         ` Mark Wielaard
2023-01-05  2:50                         ` NightStrike
2023-01-06  3:33                           ` Jacob Bachmeyer
2023-01-06  3:44                             ` Jerry D
2023-01-08  7:12                             ` NightStrike
2023-01-11  2:30                               ` Jacob Bachmeyer
2023-01-11  9:33                                 ` NightStrike
2023-01-12  4:11                                   ` Jacob Bachmeyer
2023-01-06  3:41                           ` Jerry D
2022-12-22  4:16                 ` Jacob Bachmeyer [this message]
2022-12-22  8:40                   ` Eric Pouech
2022-12-23  3:51                     ` Jacob Bachmeyer
2022-12-23 23:32                       ` Jacek Caban
2022-12-24  5:33                         ` Jacob Bachmeyer
2023-01-07  1:45                           ` Jacek Caban
2023-01-07  3:58                             ` Jacob Bachmeyer
2023-01-09 16:03                               ` Jacek Caban
2023-01-10  9:19                                 ` NightStrike
2023-01-11  9:10                                   ` NightStrike
2023-01-11 18:41                                   ` NightStrike
2023-01-14 23:36                                     ` NightStrike
2023-01-11  2:44                                 ` Jacob Bachmeyer
2023-01-08  6:47                             ` NightStrike
2023-01-04 15:21                       ` Pedro Alves
2023-01-04 15:45                         ` Eric Pouech
2023-01-04 15:52                           ` 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=63A3DA04.4060804@gmail.com \
    --to=jcb62281@gmail.com \
    --cc=dejagnu@gnu.org \
    --cc=eric.pouech@orange.fr \
    --cc=fortran@gcc.gnu.org \
    --cc=jacek@codeweavers.com \
    --cc=nightstrike@gmail.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).