[lld] r261682 - Use a different error in test.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 24 14:52:55 PST 2016


Fair enough.

Fixed in r261802.

Cheers,
Rafael


On 24 February 2016 at 13:00, Rui Ueyama <ruiu at google.com> wrote:
> On Wed, Feb 24, 2016 at 6:56 AM, Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>>
>> On 23 February 2016 at 18:38, Rui Ueyama <ruiu at google.com> wrote:
>> > "No such file or directory" part of the error message is
>> > system-dependent
>> > because (I believe) it is what strerror returned. It is probably better
>> > to
>> > remove ": No such file or directory" from this test.
>>
>> This test is already disable on windows because of its use of echo, so
>> we should be fine, no?
>
>
> Maybe, but are the error strings part of POSIX or something? It may not
> worth fixing if it's working fine now, but I'd have been on safer side just
> by not including that part of the string in the test.


More information about the llvm-commits mailing list