public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/104083] New: Ada testsuite does not respect RUNTESTFLAGS
@ 2022-01-18  9:17 marxin at gcc dot gnu.org
  2022-01-18  9:27 ` [Bug testsuite/104083] " ebotcazou at gcc dot gnu.org
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: marxin at gcc dot gnu.org @ 2022-01-18  9:17 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104083

            Bug ID: 104083
           Summary: Ada testsuite does not respect RUNTESTFLAGS
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
          Assignee: unassigned at gcc dot gnu.org
          Reporter: marxin at gcc dot gnu.org
                CC: ebotcazou at gcc dot gnu.org
  Target Milestone: ---

Running the following:

$ make check -k RUNTESTFLAGS="maintainers.exp"
...
Running chapter c7 ...
Running chapter c7 ...
Running chapter c7 ...
Running chapter c7 ...
Running chapter c7 ...
Running chapter c8 ...
Running chapter c7 ...
Running chapter c8 ...
Running chapter c7 ...
Running chapter c8 ...
Running chapter c8 ...

should not run anything from Ada testsuite.

Moreover, when I terminate the command, there are still new messages of
'Running chapter c7 ...' being printer to terminal.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug testsuite/104083] Ada testsuite does not respect RUNTESTFLAGS
  2022-01-18  9:17 [Bug testsuite/104083] New: Ada testsuite does not respect RUNTESTFLAGS marxin at gcc dot gnu.org
@ 2022-01-18  9:27 ` ebotcazou at gcc dot gnu.org
  2022-01-18  9:31 ` rguenth at gcc dot gnu.org
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2022-01-18  9:27 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104083

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ro at gcc dot gnu.org
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2022-01-18
             Status|UNCONFIRMED                 |SUSPENDED

--- Comment #1 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
The ACATS testsuite is not based on DejaGNU so it cannot.  IIRC Rainer had a
patch to do the conversion at some point but it was heavyweight.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug testsuite/104083] Ada testsuite does not respect RUNTESTFLAGS
  2022-01-18  9:17 [Bug testsuite/104083] New: Ada testsuite does not respect RUNTESTFLAGS marxin at gcc dot gnu.org
  2022-01-18  9:27 ` [Bug testsuite/104083] " ebotcazou at gcc dot gnu.org
@ 2022-01-18  9:31 ` rguenth at gcc dot gnu.org
  2022-01-19 15:17 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2022-01-19 15:20 ` ro at CeBiTec dot Uni-Bielefeld.DE
  3 siblings, 0 replies; 5+ messages in thread
From: rguenth at gcc dot gnu.org @ 2022-01-18  9:31 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104083

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Once could of course argue that eventually the acats/run_all.sh can somehow
pick up options from the RUNTESTFLAGS argument (it might need to be exported
somehow of course).

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug testsuite/104083] Ada testsuite does not respect RUNTESTFLAGS
  2022-01-18  9:17 [Bug testsuite/104083] New: Ada testsuite does not respect RUNTESTFLAGS marxin at gcc dot gnu.org
  2022-01-18  9:27 ` [Bug testsuite/104083] " ebotcazou at gcc dot gnu.org
  2022-01-18  9:31 ` rguenth at gcc dot gnu.org
@ 2022-01-19 15:17 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2022-01-19 15:20 ` ro at CeBiTec dot Uni-Bielefeld.DE
  3 siblings, 0 replies; 5+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2022-01-19 15:17 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104083

--- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> ---
> --- Comment #1 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
> The ACATS testsuite is not based on DejaGNU so it cannot.  IIRC Rainer had a
> patch to do the conversion at some point but it was heavyweight.

Right: the thread started at

        http://gcc.gnu.org/ml/gcc-patches/2011-01/msg02310.html

(almost 11 years ago).  The patch got a considerable part done, but
still needs quite some work on details.  Given that I've almost stopped
testsuite work, I'm very unlikely to finish it, though.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug testsuite/104083] Ada testsuite does not respect RUNTESTFLAGS
  2022-01-18  9:17 [Bug testsuite/104083] New: Ada testsuite does not respect RUNTESTFLAGS marxin at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2022-01-19 15:17 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2022-01-19 15:20 ` ro at CeBiTec dot Uni-Bielefeld.DE
  3 siblings, 0 replies; 5+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2022-01-19 15:20 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104083

--- Comment #4 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> ---
> --- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
> Once could of course argue that eventually the acats/run_all.sh can somehow
> pick up options from the RUNTESTFLAGS argument (it might need to be exported
> somehow of course).

While certainly possible, this would probably be quite fragile given
what DejaGnu accepts in RUNTESTFLAGS.  Besides, for runtest it's not
only RUNTESTFLAGS but also DEJAGNU pointing at an expect script setting
e.g. target_list etc.  If one wants DejaGnu's facilities, one should
just use it instead of reimplementing it.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-01-19 15:20 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-18  9:17 [Bug testsuite/104083] New: Ada testsuite does not respect RUNTESTFLAGS marxin at gcc dot gnu.org
2022-01-18  9:27 ` [Bug testsuite/104083] " ebotcazou at gcc dot gnu.org
2022-01-18  9:31 ` rguenth at gcc dot gnu.org
2022-01-19 15:17 ` ro at CeBiTec dot Uni-Bielefeld.DE
2022-01-19 15:20 ` ro at CeBiTec dot Uni-Bielefeld.DE

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).