Bug 2300661 - Multiple packages: f{23..32;35}-backgrounds: FTBFS in Fedora rawhide/f41
Summary: Multiple packages: f{23..32;35}-backgrounds: FTBFS in Fedora rawhide/f41
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: f35-backgrounds
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: David Auer
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: F41FTBFS
TreeView+ depends on / blocked
 
Reported: 2024-07-29 18:06 UTC by Fedora Release Engineering
Modified: 2024-12-14 12:44 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-12-14 12:44:34 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
build.log (23.92 KB, text/plain)
2024-07-29 18:06 UTC, Fedora Release Engineering
no flags Details
root.log (32.00 KB, text/plain)
2024-07-29 18:06 UTC, Fedora Release Engineering
no flags Details
state.log (1.68 KB, text/plain)
2024-07-29 18:06 UTC, Fedora Release Engineering
no flags Details

Description Fedora Release Engineering 2024-07-29 18:06:12 UTC
f35-backgrounds failed to build from source in Fedora rawhide/f41

https://koji.fedoraproject.org/koji/taskinfo?taskID=120651593


For details on the mass rebuild see:

https://fedoraproject.org/wiki/Fedora_41_Mass_Rebuild
Please fix f35-backgrounds at your earliest convenience and set the bug's status to
ASSIGNED when you start fixing it. If the bug remains in NEW state for 8 weeks,
f35-backgrounds will be orphaned. Before branching of Fedora 42,
f35-backgrounds will be retired, if it still fails to build.

For more details on the FTBFS policy, please visit:
https://docs.fedoraproject.org/en-US/fesco/Fails_to_build_from_source_Fails_to_install/

Comment 1 Fedora Release Engineering 2024-07-29 18:06:18 UTC
Created attachment 2041070 [details]
build.log

Comment 2 Fedora Release Engineering 2024-07-29 18:06:24 UTC
Created attachment 2041071 [details]
root.log

file root.log too big, will only attach last 32768 bytes

Comment 3 Fedora Release Engineering 2024-07-29 18:06:27 UTC
Created attachment 2041072 [details]
state.log

Comment 4 David Auer 2024-11-15 22:27:02 UTC
To fix those we just need to change the build dependency to kde*4*-filesystem. I guess the macros were moved there in the f40 release. I have mock-built successfully with this change against f40 and f41. Would you like PRs to fix this?

On my Fedora 41 I have outdated builds (fc39 suffix) for a number of background packages (f25, f26, f28, f30, f31, f32, f35).

Comment 5 Luya Tshimbalanga 2024-11-16 05:40:13 UTC
(In reply to David Auer from comment #4)
> To fix those we just need to change the build dependency to
> kde*4*-filesystem. I guess the macros were moved there in the f40 release. I
> have mock-built successfully with this change against f40 and f41. Would you
> like PRs to fix this?
> 
> On my Fedora 41 I have outdated builds (fc39 suffix) for a number of
> background packages (f25, f26, f28, f30, f31, f32, f35).

Please do. Thank you.

Comment 6 David Auer 2024-11-24 22:24:17 UTC
Thanks, for f35 I triggered a build and the update should land in rawhide automatically. (https://bodhi.fedoraproject.org/updates/FEDORA-2024-24959a672f)

Shall we also merge and update this in f41 and maybe even f40? I personally would like to see it in f41 because I'm kind of on a mission to reduce the number of packages on my system that are not built against the current release. (Whether or not that is a really important mission is probably debatable, so I'd totally understand if you don't think we should create those extra updates.)

From a quick glance I'd say we need to apply the same fix for f23 .. f32, that is 10 pull requests for the rawhide updates and 10 builds. Depending on the response to my previous question it may be even 20 or 30 of those. For me it would be more efficient if I could push those directly as co-maintainer but I'm also happy to got the PR route if you prefer that. Just let me know how to proceed.

Cheers
David

Comment 7 Luya Tshimbalanga 2024-11-25 08:49:37 UTC
Let's do the merge for all affected backgrounds to reduce burden on future update. It seems you have proven packager privilege so you can push the update. If not, let know your username to add as co-maintainer.

Comment 8 David Auer 2024-11-25 13:23:16 UTC
No, I'm packager but not provenpackager. FAS is "dreua".

> Let's do the merge for all affected backgrounds to reduce burden on future update.

I'm not sure I understand you correctly here. My intention is to fix all packages at least in the rawhide/main branch so for future releases and updates everything is set. The question remains if we should also merge this back into f41 and/or f40 or just leave those with the older packages until they are eol. The packages don't appear to be broken so both is fine I guess. I'd update f41 mainly for my personal taste (and maybe others who wonder why they have packages build against f39 installed on their system, who knows?).

Thanks for making me co-maintainer, I'm ready to start updating once that is done :)

Comment 9 David Auer 2024-11-25 13:28:54 UTC
I see now where we you might have misunderstood me: When I talk about f40 and f41 specifically here, I mean builds/updates for those releases, not the f40/41-backgrounds packages ;) I always meant to do that for all fXX-backgrounds packages affected, i.e. f23 until including f32 and f35 (already updated for rawhide). Let me know if there are any more background packages affected which I might have missed.

Comment 10 Troy Dawson 2024-11-25 14:27:24 UTC
I should have spoken up back with the original pull request, but how about we fix this in a more sustainable way.
kde4-filesystem is a very old, package.  It would have been removed long ago except so many spec files use it's macros.
The macro that many people are using (including the older background rpms) is %_kde4_datadir
But guess what: %_kde4_datadir = %_datadir

How about we remove the dependency, and change the macro.

This is all it takes

-BuildRequires:  kde4-filesystem

-%{_kde4_datadir}/wallpapers/%{Bg_Name}/
+%{_datadir}/wallpapers/%{Bg_Name}/

-%{_kde4_datadir}/wallpapers/%{Bg_Name}_*/
+%{_datadir}/wallpapers/%{Bg_Name}_*/

Comment 11 David Auer 2024-11-25 15:26:48 UTC
Thats good to know, it's not too late ;)
I don't know the backgrounds of these packages or why it was done like this in the first place but your suggestion looks good to me.
There is also "Requires: kde-filesystem" should that stay as is or can that also be removed or changed to something better?

Comment 12 Troy Dawson 2024-11-25 15:29:23 UTC
oh, I missed that. Yes, that can be removed.

Comment 13 Luya Tshimbalanga 2024-11-27 08:50:43 UTC
(In reply to David Auer from comment #8)

> 
> Thanks for making me co-maintainer, I'm ready to start updating once that is
> done :)

I added you as co-maintainer for the remaining fedora backgrounds needed an update. Thanks for taking care. =)

Comment 14 David Auer 2024-11-27 23:52:34 UTC
My packaging dashboard is quite red, so thank you :) It looks like you missed f25 and f35 though, maybe you could add those so I can do it all in one batch.

Since no other opinions came up here, I'll create upgrades for rawhide and the f41 release. I assume there will be still some people upgrading from f40 which might have the older background packages installed and so they get a fresh build.

@Troy: I had another look at that and found out that the kde-filesystem owns the "/usr/share/wallpapers" directory which would be used unowned otherwise. While not strictly necessary I think it is should stay as runtime requirement.

Comment 15 Luya Tshimbalanga 2024-11-28 16:19:02 UTC
(In reply to David Auer from comment #14)
> My packaging dashboard is quite red, so thank you :) It looks like you
> missed f25 and f35 though, maybe you could add those so I can do it all in
> one batch.
> 
You are now added on both f25 and f35 background. =)

Comment 16 Fedora Update System 2024-12-02 23:33:21 UTC
FEDORA-2024-f18232d800 (f35-backgrounds-35.0.1-12.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-f18232d800

Comment 17 Fedora Update System 2024-12-04 03:48:30 UTC
FEDORA-2024-f18232d800 has been pushed to the Fedora 41 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-f18232d800`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-f18232d800

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 18 David Auer 2024-12-05 13:02:04 UTC
The builds for rawhide and f41 are done, just created the update: https://bodhi.fedoraproject.org/updates/FEDORA-2024-bb1677ad00

Once that hits stable I'm happy and done with the stuff I came here for. I have now some scripts in place to automate the repetitive tasks, so let me know if you want me to remove the kde4-filesystem dependency from the remaining packages I have access to or if someone wants me to update the older backgrounds. (Those with release names instead of numbers. I'd need co-maintainer access first of course and ideally a list of those names.)

Comment 19 Fedora Update System 2024-12-12 01:33:43 UTC
FEDORA-2024-f18232d800 (f35-backgrounds-35.0.1-12.fc41) has been pushed to the Fedora 41 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 20 David Auer 2024-12-14 12:44:34 UTC
I believe this is all done, thanks everyone!


Note You need to log in before you can comment on or make changes to this bug.