Overview
Request 1156747 accepted
- New package based on version 18.1.1.
* For details, see the release notes:
- https://releases.llvm.org/18.1.0/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/tools/clang/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/tools/clang/tools/extra/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/projects/libcxx/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/tools/lld/docs/ReleaseNotes.html
* New LLVM tool: llvm-readtapi, LLVM TAPI file reader and
transformer.
* Removed LLVM tools: llvm-tapi-diff, llvm-remark-size-diff.
* New LLDB tool: lldb-dap debug adapter.
* Removed LLDB tool: lldb-vscode.
- Rename libomp-devel symbolic name to libomp-devel-provider.
We want to introduce a metapackage named libomp-devel.
- Rebase patches:
* assume-opensuse.patch
* link-clang-tools-extra-shared.patch
* llvm_build_tablegen_component_as_shared_library.patch
* llvm-do-not-install-static-libraries.patch
* llvm-fix-find-gcc5-install.patch
* llvm-normally-versioned-libllvm.patch
* llvm-suse-implicit-gnu.patch
- Add patches to fix tests:
* clang-fix-modules-test-riscv.patch
* clang-fix-openmp-test-non-x86.patch
* clang-fix-openmp-test.patch
* llvm-fix-cov-test-i586.patch
- Restructure version macros in specfile for new versioning scheme.
- Fix packaging of ld.lld as ld alternative.
- Disable LLDB Python bindings on SLE/Leap. They now require Swig 4
which isn't available there.
- Created by aaronpuchert
- In state accepted
Request History
aaronpuchert created request
- New package based on version 18.1.1.
* For details, see the release notes:
- https://releases.llvm.org/18.1.0/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/tools/clang/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/tools/clang/tools/extra/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/projects/libcxx/docs/ReleaseNotes.html
- https://releases.llvm.org/18.1.0/tools/lld/docs/ReleaseNotes.html
* New LLVM tool: llvm-readtapi, LLVM TAPI file reader and
transformer.
* Removed LLVM tools: llvm-tapi-diff, llvm-remark-size-diff.
* New LLDB tool: lldb-dap debug adapter.
* Removed LLDB tool: lldb-vscode.
- Rename libomp-devel symbolic name to libomp-devel-provider.
We want to introduce a metapackage named libomp-devel.
- Rebase patches:
* assume-opensuse.patch
* link-clang-tools-extra-shared.patch
* llvm_build_tablegen_component_as_shared_library.patch
* llvm-do-not-install-static-libraries.patch
* llvm-fix-find-gcc5-install.patch
* llvm-normally-versioned-libllvm.patch
* llvm-suse-implicit-gnu.patch
- Add patches to fix tests:
* clang-fix-modules-test-riscv.patch
* clang-fix-openmp-test-non-x86.patch
* clang-fix-openmp-test.patch
* llvm-fix-cov-test-i586.patch
- Restructure version macros in specfile for new versioning scheme.
- Fix packaging of ld.lld as ld alternative.
- Disable LLDB Python bindings on SLE/Leap. They now require Swig 4
which isn't available there.
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
staging-bot added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:23"
staging-bot accepted review
Picked "openSUSE:Factory:Staging:adi:23"
anag+factory added factory-staging as a reviewer
Being evaluated by group "factory-staging"
anag+factory accepted review
Unstaged from project "openSUSE:Factory:Staging:adi:23"
anag+factory set openSUSE:Factory:Staging:H as a staging project
Being evaluated by staging project "openSUSE:Factory:Staging:H"
anag+factory accepted review
Picked "openSUSE:Factory:Staging:H"
licensedigger accepted review
The legal review is accepted preliminary. The package may require actions later on.
dimstar accepted review
anag+factory accepted review
Staging Project openSUSE:Factory:Staging:H got accepted.
anag+factory approved review
Staging Project openSUSE:Factory:Staging:H got accepted.
anag+factory accepted request
Staging Project openSUSE:Factory:Staging:H got accepted.
mrueckert wrote (1156747) (https://build.opensuse.org/request/show/1156747) if you do not set any %attr() on %ghost it will set the the attr to
compared to the ls output
it is advisable to set the expected permissions via
%attr(777,root,root)
My understanding is that these symlinks are installed by
update-alternatives
, and the%ghost
files are just to track the symlinks as belonging to a package. After installation, the permissions seem to be as expected, andrpm -V
doesn't complain about them.This is in fact unchanged from previous LLVM versions. I could change it, but then it should probably be changed for all versions.
The Wiki also recommends plain
%ghost
without%attr
. Also, I wonder why RPM can't just default to777
for symlinks? Regular files don't need an explicit755
as well. Or is the apparent000
just meaningless for%ghost
files?@aaronpuchert, it looks like this change broke build for PostgreSQL versions up to 15 (but not 16). The PostgreSQL server that gets started as part of the regression test suite crashes with a segfault. It does not happen when I disable LLVM support or build the affected PostgreSQL versions with LLVM 17.
Each time the PostgreSQL server segfaults the kernel logs this:
Do you have any idea what could be causing (or triggering) this?
I don't know what's causing this, but I assume it simply doesn't support LLVM 18 yet. PostgreSQL 15.6 for example came out on February 8, but the first release of LLVM 18 was tagged on March 6. So there might be changes in LLVM 18 that PostgreSQL hasn't adapted to yet.
For now I'd just go back to LLVM 17 and try again with the next version. If that still doesn't work, we should investigate, but for now it's not surprising.