[llvm-dev] [llvm-rc] absolute.test failing

David Greene via llvm-dev llvm-dev at lists.llvm.org
Thu Jan 10 11:42:02 PST 2019


Martin Storsjö <martin at martin.st> writes:

> On Wed, 9 Jan 2019, David Greene wrote:
>
>> I've come across a curious and pernicious problem in llvm-rc.
>> absolute.test checks that llvm-rc can accept a filename that is an
>> absolute path.  And it works just fine.  Until you run it with a file
>> that starts with "/c."
>
> Hmm, that's rather unfortunate indeed.
>
> FWIW, this test doesn't test specifically whether llvm-rc can accept
> an absolute filename as command line argument - all the llvm-rc tests
> run llvm-rc with absolute filenames as arguments. This test checks
> whether llvm-rc can handle an absolute filename reference within a rc
> file.
>
> I presume you run into the same issue on all other tests in
> test/tools/llvm-rc as well?

Actually, I don't.  Now I wonder why...

> One way of disambiguating between option and pathname for the sake of
> the tests, would be to add '--' before the path arguments, which seems
> to be handled by the LLVM options parser at least. Does that sound
> sensible to you (and others CC:d)?

I tried that and it didn't work.  Same error.

$ ./bin/llvm-rc -- /choose/me
Exactly one input file should be provided.

                        -David


More information about the llvm-dev mailing list