GNU bug report logs - #74786
[PATCH] Add custom args to test compile step

Previous Next

Package: emacs;

Reported by: "O'Brien, Will" <will.08rien <at> gmail.com>

Date: Wed, 11 Dec 2024 11:29:02 UTC

Severity: wishlist

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #29 received at 74786 <at> debbugs.gnu.org (full text, mbox):

From: "O'Brien, Will" <will.08rien <at> gmail.com>
To: "J.P." <jp <at> neverwas.me>
Cc: Randy Taylor <dev <at> rjt.dev>, Robert Pluim <rpluim <at> gmail.com>,
 Eli Zaretskii <eliz <at> gnu.org>, 74786 <at> debbugs.gnu.org
Subject: Re: bug#74786: [PATCH] Add custom args to test compile step
Date: Tue, 17 Dec 2024 08:32:50 +0000
On Mon, Dec 16 2024 at 18:41, "J.P." <jp <at> neverwas.me> wrote:

>>    "Compile the tests matching REGEXP.
>>  This function respects the `go-ts-mode-build-tags' variable for
>>  specifying build tags."
>> -  (compile (format "go test -v %s -run '%s'"
>> +  (compile (format "go test %s -v %s -run '%s'"
>> +                   (string-join go-ts-mode-test-args)
>                                                        ^
>
> Should this specify a separator argument?

It should indeed!


> Or, since
>
>   $ go help testflag
>
> lists a few options that take a regexp argument, maybe something like
>
>   (mapconcat #'shell-quote-argument go-ts-mode-test-args " ")

Ah, nice!  Yeah I will update.

Thank you!




This bug report was last modified 136 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.