Skip to content

account for CLANG_VENDOR when checking for llvm version

Rodrigo Muino Tomonari requested to merge github/fork/nathanblair/clang-vendor-fix into master
Checklist

Apologies if I'm forgetting something - this is my first commit to node. I didn't think it necessary to include testing results as no src code is actually modified; only the configure script.

Please see issue #29536 (closed) for more details. I have not verified if this breaks configurations on systems like void-linux where a CLANG_VENDOR` is not specified, but it did fix the issue on alpine linux. Before this PR is merged, I'd like to investigate that it can be built in such cases.

In the meantime, it would help to possibly get other members to verify the change allows clang/llvm to be recognized by their various systems.

Merge request reports