Bug 1685029

Summary: `llvm-config --bindir' points to /usr/lib64/llvm/clang
Product: [Fedora] Fedora Reporter: Martin Stransky <stransky>
Component: llvmAssignee: serge_sans_paille <sguelton>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 30CC: dmalcolm, jakub, jistone, scottt.tw, sguelton, siddharth.kde, tstellar
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: llvm-8.0.0-0.4.rc3.fc30 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-03-29 19:11:06 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Martin Stransky 2019-03-04 08:29:09 UTC
Description of problem:
When building Firefox, we get failure at configure. It worked with previous version (7.0 AFAIK).

 0:07.64 checking for llvm-config... /usr/bin/llvm-config
 0:07.74 ERROR: The file /usr/lib64/llvm/clang returned by `llvm-config --bindir` does not exist.
 0:07.74 clang is required to build Firefox.  Please install the
 0:07.74 necessary packages, or run `mach bootstrap`.


Version-Release number of selected component (if applicable):
llvm                        x86_64  8.0.0-0.2.rc2.fc30            build  4.2 M
llvm-devel                  x86_64  8.0.0-0.2.rc2.fc30            build  2.7 M

Comment 1 Martin Stransky 2019-03-04 08:29:54 UTC
Failed FF build task: https://koji.fedoraproject.org/koji/taskinfo?taskID=33121406

Comment 2 serge_sans_paille 2019-03-04 10:11:56 UTC
According to the documentation,

   [...]
   --bindir          Directory containing LLVM executables.

And llvm-related binaries are indeed installed in that location.

I'll probably make sure clang also install symlinks to that location so that everything is consistent.

Comment 3 Martin Stransky 2019-03-06 09:50:43 UTC
(In reply to serge_sans_paille from comment #2)
> According to the documentation,
> 
>    [...]
>    --bindir          Directory containing LLVM executables.
> 
> And llvm-related binaries are indeed installed in that location.
> 
> I'll probably make sure clang also install symlinks to that location so that
> everything is consistent.

Before 8.0.0 update the --bindir pointed to /usr/bin. It's returns /usr/lib64/llvm/ now where the clang is missing.
Does that mean the configure script is broken and the llvm-config --bindir should not be used for clang location?
Thanks.

Comment 4 serge_sans_paille 2019-03-06 18:16:04 UTC
I've discussed with @tstellar and we'll keep the previous behavior. Everything should be back to normal in rawhide by llvm-8.0.0-0.4.rc3.fc31 and clang-8.0.0-0.5.rc3.fc31. Can you confirm its okay for you? if so I'll apply the same patches to f30.

Comment 5 Martin Stransky 2019-03-07 10:44:35 UTC
(In reply to serge_sans_paille from comment #4)
> I've discussed with @tstellar and we'll keep the previous behavior.
> Everything should be back to normal in rawhide by llvm-8.0.0-0.4.rc3.fc31
> and clang-8.0.0-0.5.rc3.fc31. Can you confirm its okay for you? if so I'll
> apply the same patches to f30.

Seems to be functional in rawhide, please backport to F30. Thanks!

Comment 6 Fedora Update System 2019-03-08 08:06:05 UTC
clang-8.0.0-0.5.rc3.fc30 llvm-8.0.0-0.4.rc3.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-ba6de71571

Comment 7 serge_sans_paille 2019-03-08 08:10:27 UTC
https://bodhi.fedoraproject.org/updates/FEDORA-2019-ba6de71571 should fix the issue for f30

Comment 8 Fedora Update System 2019-03-08 19:45:25 UTC
clang-8.0.0-0.5.rc3.fc30, llvm-8.0.0-0.4.rc3.fc30 has been pushed to the Fedora 30 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-ba6de71571

Comment 9 Fedora Update System 2019-03-29 19:11:06 UTC
clang-8.0.0-0.5.rc3.fc30, llvm-8.0.0-0.4.rc3.fc30 has been pushed to the Fedora 30 stable repository. If problems still persist, please make note of it in this bug report.