Todd, Thanks very much for this!
On Sun Jun01'25 09:09:55PM, Todd Zullinger wrote: > From: Todd Zullinger <[email protected]> > Date: Sun, 1 Jun 2025 21:09:55 -0400 > To: [email protected] > Reply-To: Community support for Fedora users <[email protected]> > Subject: Re: F41 -> F42 (openbox+slim: X does not get started) > > Ranjan Maitra via users wrote: > > [As a note, upstream is in trouble because the maintainer > > is quite unwell, judging by the ticket responses on the > > slimfork site. I hope he recovers well and soon, at least > > for his sake.] > > Yes, I hope they get well soon. > > > So, I think that there are two options: one, perhaps a > > long-term one would be to add code that allows for reading > > .jxl files. That is doable and a lot of work, but might > > only be transient as Fedora moves to something else down > > the road. > > > > So, I was thinking of using magick to convert the file to > > a png while the rpm was being built. Might not be kosher > > anyway, but I brought in ImageMagick as a BuildRequire, > > and replaced: > > > > ln -s ../../../backgrounds/f%{?fedora}/default/f%{?fedora}-01-day.png > > %{buildroot}%{_datadir}/%{name}/themes/default/background.png > > > > with: > > > > magick ../../../backgrounds/f%{?fedora}/default/f%{?fedora}-01-day.jxl > > convert %{buildroot}%{_datadir}/%{name}/themes/default/background.png > > > > I don't know why this creates the following error: > > > > magick ../../../backgrounds/f42/default/f42-01-day.jxl > > /home/localuser/rpmbuild/BUILD/slim-1.4.0-build/BUILDROOT/usr/share/slim/themes/default/background.png > > > > magick: unable to open image > > '../../../backgrounds/f42/default/f42-01-day.jxl': No such file or > > directory @ error/blob.c/OpenBlob/3596. error: Bad exit status from > > /var/tmp/rpm-tmp.62ipD8 (%install) > > > > RPM build errors: > > Bad exit status from /var/tmp/rpm-tmp.62ipD8 (%install) > > > > I do not quite understand why I get this error, especially > > because the "ln -s" appears to find this path. > > 'ln -s' works because it doesn't actually need the target to > create the link. If you did a build in mock and then looked > for the target file in the chroot, it's not there because > there's no BuildRequires for f%{?fedora}-backgrounds-base, > only a Requires. Oh, I see. I put in the BuildRequires now, but also made the changes below. I am not sure I needed to put in the BuildRequires but I did it anyway. Btw, it appears to me that the Requires should no longer be needed, since the image that is the only thing that is needed from the backgrounds package is being created and stored essentially in /usr/share/slim/themes/default/background.png > > That ensures it is there at install time, so the symlink > works. You'd need to add a BuildRequires to get > f%{?fedora}-backgrounds-base installed in the package > buildroot where you can use magick to convert it. > > Once you're not making a symlink, I think using the absolute > path makes things clearer. Having to keep track of where > you are relative to the backgrounds directory is a minor > hassle, which has no real benefit when you're converting the > file and saving it at another path. In other words: I see, thank you for the explanation. > > magick %{_datadir}/backgrounds/f%{?fedora}/default/f%{?fedora}-01-day.jxl > \ > %{buildroot}%{_datadir}/%{name}/themes/default/background.jpg > > You don't need to specify convert in the magick command. Yeah, right! Old habits (when convert was the command) die hard. > Using magick as above will convert the file from jxl to jpg > (or png if you change the extension). > > It may be worth comparing the size and quality of jpg versus > png. It takes about 4 times as long to convert to png as > jpg (2.8 vs 0.6 seconds) and is about that many times larger > as well (6.4M vs 1.4M). Indeed, I decided to stick with jpg. I do not think the quality will make that much of a difference and we can see if anyone else does not like the drop from PNG to JPG. > > Neither the time nor size for the png is too much. I just > noticed it while doing this in a container to test. The > input jxl file is under 1M, FWIW. Right, and I was wondering earlier the real value of this change to jxl. But it does reduce the size a bit, and I guess smaller is always better. I submitted the updated RPM to F42 testing. I will see if anyone has comments on it, assuming there exists anyone else who uses slim, and then move to looking into 1.4.1 (perhaps next week after 1.4.0 moves to stable). As it is, it is not clear to me that anyone other than me is waiting with bated breath for SLiM:-) Thank you again for all your explanations and all your help! I am quite at sea with .spec files, one of the reasons being, of course, that I end up dabbling in them rather infrequently, so it is good to have explanations. Best wishes, Ranjan -- _______________________________________________ users mailing list -- [email protected] To unsubscribe send an email to [email protected] 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/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
