#8465 closed defect (fixed)
llvm-3.9.0: make cmake Required instead of Recommended
| Reported by: | Jeremy Henty | Owned by: | Samuel |
|---|---|---|---|
| Priority: | normal | Milestone: | 8.0 |
| Component: | BOOK | Version: | SVN |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Previous versions of llvm could be built with cmake or autotools, and therefore cmake was described as Recommended rather than Required. (See my earlier ticket http://wiki.linuxfromscratch.org/blfs/ticket/7953 for some discussion about this.)
However, llvm-3.9.0 has removed support for building with autotools. Running ./configure does nothing except return an error message telling the user to use cmake. Since cmake is now the only way to build llvm, it should be Required instead of Recommended.
Attachments (1)
Change History (6)
comment:1 by , 8 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
by , 8 years ago
| Attachment: | llvm_cmake_dependency.patch added |
|---|
comment:3 by , 8 years ago
Can someone please apply the patch? I don't know if Jeremy can commit so someone else might need to.
comment:4 by , 7 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixed at revision 17927.
One vert small change to the patch: Removing an entry from recommended patches left the sequence 'a, and b'. I removed the comma.
I have done something like this (see ticket #8247) so I'll take this ticket and do it tonight, and post the patch here and on blfs-dev.
I believe this is not the same as #7953 and should not be marked wontfix.