Re: Self introduction & looking for sponsor for ccls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi all,

So I've been doing some experiments with latest ccls version on COPR [1] and this is what I found:

1.- It won't build on rawhide

rawhide currently sports clang/llvm v20, and ccls simply fails to build with that (this would require changes upstream since the LLVM libs have changed).

2.- It builds in _some_ platforms with clang/llvm v17

- Builds go though in fedora40 (aarch64, x86_64), but rpm/check-rpaths complains [2] (even though I used "Requires: llvm17-libs").
- On rawhide/x86_64 the clang c++/v17 compiler seems to be broken. [3]
- On rawhide/aarch64 the LLVM17 libraries seem incompatible (even though they are good in fedora 40). [4]

So I think in order to have ccls up an running we'll have to build an older clang/llvm combination (ccls tests require clang v6). But building clang & llvm v6 just to have ccls is probably overkill.

So things look bad for ccls. I'll do some other experiments, and see if there're any plans to support LLVM20 upstream in the future. Any other ideas/suggestions to have ccls running are welcome.

Thanks and kind regards,
Antonio

[1]
https://copr.fedorainfracloud.org/coprs/vieiro/ccls/build/8747711/

[2]
ERROR 0010: file '/usr/bin/ccls' contains an empty runpath in [/usr/lib64/llvm17/lib:]

https://download.copr.fedorainfracloud.org/results/vieiro/ccls/fedora-40-aarch64/08747711-ccls/builder-live.log.gz
https://download.copr.fedorainfracloud.org/results/vieiro/ccls/fedora-40-x86_64/08747711-ccls/builder-live.log.gz

[3]
https://download.copr.fedorainfracloud.org/results/vieiro/ccls/fedora-rawhide-x86_64/08747711-ccls/builder-live.log.gz

-- Check for working CXX compiler: /usr/lib64/llvm17/bin/clang++ - broken
CMake Error at /usr/share/cmake/Modules/CMakeTestCXXCompiler.cmake:73 (message):
  The C++ compiler
    "/usr/lib64/llvm17/bin/clang++"
  is not able to compile a simple test program.

[4]
/usr/lib64/llvm17/include/llvm/ADT/SmallVector.h:582:47: error: no type named 'const_iterator' in 'llvm::SmallVectorTemplateBase<char>'
  582 |   using const_iterator = typename SuperClass::const_iterator;
      |                          ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
https://download.copr.fedorainfracloud.org/results/vieiro/ccls/fedora-rawhide-aarch64/08747711-ccls/builder-live.log.gz



El 10/3/25 a las 10:15, Antonio escribió:
Hi all,

Yes, Dan was kind enough to tell me exactly this.

I will take me some time, though, as I have some other priorities.

Thanks and kind regards,
Antonio

El 8/3/25 a las 14:23, Michel Lind escribió:
Hi Antonio,

First of all, welcome!

On Thu, Mar 6, 2025, at 7:27 AM, Antonio wrote:
Hi Dan!

This is indeed appreciated. No better sponsor!

This will be my first contribution. I've read [1] and followed all
instructions there, but I'm stuck. What should I do now?

Because the package has been retired for too long, you need to submit it for review request again as if it's a new package (base it on the retired spec with any improvement that's necessary)

Mark the issue as blocking FE-NEEDSPONSOR, and probably reply here or to Dan with the issue URL so he sees it.

Best regards,



--
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux