r195080 - Disable this check temporarily.

Bill Wendling isanbard at gmail.com
Wed Nov 20 23:26:23 PST 2013


On Nov 20, 2013, at 5:28 PM, Daniel Jasper <djasper at google.com> wrote:

> On Tue, Nov 19, 2013 at 1:04 AM, Manuel Klimek <klimek at google.com> wrote:
> On Tue, Nov 19, 2013 at 3:25 AM, Bill Wendling <isanbard at gmail.com> wrote:
> Author: void
> Date: Mon Nov 18 20:25:28 2013
> New Revision: 195080
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=195080&view=rev
> Log:
> Disable this check temporarily.
> 
> This is failing for me. When I run the command on my own, I get this:
> 
> Error reading /usr/local/google/home/morbo/llvm/llvm.obj/tools/clang/test/Format/Output/.clang-format: Invalid argument
> void f() {
>   int* i;
>   int j;
> }
> 
> The formatting is like this because I have the Google format version in my
> ~/.clang-format file. This test should be made independent of that.
> 
> That should already be independent, as there's a cfe/trunk/test/.clang-format file checked in, which clang-format should find (exactly for that reason). So you're saying that file exists, but clang-format still picks up the one from your home dir?
> 
> He is doing an out-of-tree build and the formatted file is actually written to the build-tree (which doesn't have any .clang-format files). We also cannot use "-assume-filename" as that option currently only works when the input comes from stdin.

What Daniel said. :-) If the ‘-assume-filename’ will fix this, then you could output the code to a temp file maybe?

-bw

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131120/8b9e8d59/attachment.html>


More information about the cfe-commits mailing list