[PATCH] D112759: [llvm] [Debuginfo] Add llvm-debuginfod-find tool and end-to-end-tests.

[Noah Shutty via Phabricator via llvm-commits]

noajshu added a comment.

In D112759#3196640 <https://reviews.llvm.org/D112759#3196640>, @uabelho wrote:

> In D112759#3195363 <https://reviews.llvm.org/D112759#3195363>, @noajshu wrote:
>
>> Hi Mikael,
>>
>> Could you check which version of python you are running? (python --version)
>>
>> Thanks,
>> Noah
>
> Hi,
>
> We're using python 3.8:
>
>   -- Found Python3: /app/vbuild/RHEL7-x86_64/python/3.8.0/bin/python3.8 (found suitable version "3.8.0", minimum required is "3.6") found components: Interpreter
>
> and
>
>   > app/vbuild/RHEL7-x86_64/python/3.8.0/bin/python3.8 --version
>   Python 3.8.0
>
> However, I noticed now that it passes if I do a clean build, so I have no idea what actually happened when I got the failure yesterday.
> Could there be some problem so binaries are not rebuilt as they should?
>
> Apologies for sending you on a wild-goose chase. :/
>
> The test passes for me also with https://reviews.llvm.org/D115814 applied, but as I said, that patch doesn't seem to be needed by us at least.

@uabelho thanks so much for confirming it works after you clean your build folder, and no worries about the flake. 
Although the cmake caching behavior is mostly correct, I have seen at least one exception (e.g. when implicit tools are added).
Glad to hear it now passes the test, and thanks for running it!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112759/new/

https://reviews.llvm.org/D112759