[llvm] [lit] Echo full RUN lines in case of external shells (PR #66408)

Joel E. Denny via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 15:07:40 PDT 2023


jdenny-ornl wrote:

> > > > > Can the test be restructured such that it can handle this?
> > > > 
> > > > 
> > > > I'm concerned about weakening the test and missing other problems. I might use `grep -v` to throw away that line. Is that portable to windows?
> > > 
> > > 
> > > I'm not sure. I think there are some tests that still do use grep, but I think the usage of it is highly discouraged these days.
> > 
> > 
> > I grepped for grep and found a number of tests still using it. I'll push a commit with this solution for now so that (hopefully) the bot is green again. We can try to find a better solution afterward. Does that seem reasonable?
> 
> Sure, that sounds reasonable to me.

Pushed that: aa71680f2948ac177144e1089c58d55b9ac0cef2.  I'll keep an eye on the bots for any additional issues.

https://github.com/llvm/llvm-project/pull/66408


More information about the llvm-commits mailing list