935c2e1
@@ -0,0 +1,3 @@
+ # Drop the following option after debugedit adds support to DWARF-5:
+ # https://sourceware.org/bugzilla/show_bug.cgi?id=28728
+ -gdwarf-4 -g0
@@ -94,6 +94,9 @@
%if %{without compat_build}
Source5: macros.%{name}
%endif
+ # This file is still needed because We still build on F38 in the
+ # upstream-snapshot branch.
+ Source6: clang.cfg
# Patches for clang
Patch1: 0001-PATCH-clang-Make-funwind-tables-the-default-on-all-a.patch
We still need this file in Fedora 38 compat packages, which is still used during the daily snapshots.
Keep in mind that %SOURCE6is expected to be available in line 526. It would be great if compat packages in rawhide branch and non-compat in upstream-snapshot used the same mechanism. See https://src.fedoraproject.org/rpms/clang/pull-request/223.
%SOURCE6
Build succeeded. https://fedora.softwarefactory-project.io/zuul/buildset/1bc18276ab2a4f0994861ffb59dad782
I think we need to add -g0 here so it doesn't enable debuginfo unconditionally.
I'm slightly confused on what the target for this change is. I believe we still have this in upstream-snapshot, why do we need it in rawhide? Or is this part of an attempt to sync rawhide and upstream-snapshot?
@nikic because the rawhide branch is also used to build on f38, e.g. https://copr.fedorainfracloud.org/coprs/g/fedora-llvm-team/llvm-compat-packages/builds/
rawhide
rebased onto 5fbeaee
I've just rebased this PR on top of rawhide and included -g0 as requested.
-g0
Build succeeded. https://fedora.softwarefactory-project.io/zuul/buildset/28d164727bdb4b4ba4775592a522a7b4
LGTM.
rebased onto 935c2e1
Pull-Request has been merged by tuliom
Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci https://fedora.softwarefactory-project.io/zuul/buildset/7bcbdba4bf6f402690151c660016d8bd
tuliom
commented 2 months ago
We still need this file in Fedora 38 compat packages, which is still
used during the daily snapshots.