Make it clear to the user that llvm and lld is required to be installed for OSS VM Clang

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 8.2.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • DevProd Build
    • Fully Compatible
    • ALL
    • None
    • 3
    • Hour
    • None
    • None
    • None
    • None
    • None
    • None
    • 0

      One of the engineers used the --local_clang_compiler flag where the setup failed. This was due to not displaying the requirements ahead of time that lld is need it also

      Furthermore, the path of lld is a symlink and we should be reading the realpath of it. Bazel repository_ctx.path(lld_path).exist did not detect that the path exists.

      The error message of not finding the installed lld needs to be updated as it prints the following: "The LLD_PATH does not exist: xx/yy/zz". LLD_PATH was an environment variable and we are no longer using it.

            Assignee:
            Patrice Arruda
            Reporter:
            Patrice Arruda
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: