[cfe-dev] RFC: Clang test runner changes
Shantonu Sen
ssen at apple.com
Tue Aug 4 10:02:37 PDT 2009
Would it be possible for the .script file to include the PATH change
in its text so it's easier to run the .script file manually to
reproduce failures?
Shantonu Sen
ssen at apple.com
Sent from my Mac Pro
On Jul 30, 2009, at 8:37 PM, Daniel Dunbar wrote:
> On Thu, Jul 30, 2009 at 3:48 PM, Eli
> Friedman<eli.friedman at gmail.com> wrote:
>> On Mon, Jul 27, 2009 at 3:32 PM, Daniel Dunbar<daniel at zuster.org>
>> wrote:
>>> - Implement the execution of RUN lines internally to the tool. The
>>> format will probably become a bit more strict, instead of "sh"
>>> format.
>>> The runner can then start to validate the lines, for example to
>>> ensure
>>> that only certain tools are called (clang-cc, count, not, FileCheck,
>>> etc.). The runner can then also implement certain features
>>> internally
>>> (like grep).
>>
>> Hmm, actually, there's something seriously screwy with the testrunner
>> at the moment: we don't actually support not and count, but tests
>> using them still manage to pass.
>
> I'm not sure what you mean by "we don't actually support not and
> count". The test runner modifies the PATH of the subprocess to include
> LLVM's test/Scripts dir, which is the same as it used to do (although
> the mechanism changed a bit).
>
>> I was just experimenting a bit with
>> not creating .script files, and somehow reading the commands from
>> stdin changes the relevant behavior. (The other issue with reading
>> the commands from stdin is that it appears to trigger some sort of
>> thread safety bug, but that isn't really relevant if you're planning
>> on rewriting the code.)
>
> I'm not exactly sure what problem you are referring to, but yeah
> perhaps not worth worrying about until things change to not using
> /bin/sh.
>
> - Daniel
>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
More information about the cfe-dev
mailing list