keeferrourke / capitaine-cursors

An x-cursor theme inspired by macOS and based on KDE Breeze. Designed to pair well with my icon pack, La Capitaine.
Other
1.06k stars 88 forks source link

Fedora/EPEL packaging issues #43

Open MrPointer opened 4 years ago

MrPointer commented 4 years ago

After upgrading to the latest version from the tcg copr repo (full system spec given below) and re-enabling the cursor using gnome-tweaks, the session immediately crashes and logs out. After that, I'm not able to log in again, unless I disable the cursors from a classic GNOME session. Restarting the system also didn't help. However, reverting back to version 3.5 solves the issue.

I also should mention that I have some custom themes installed:

I also have some gnome extensions enabled, but I'll list them only if you'd ask.

System specs

Computer Type: Laptop Laptop Model: Dell XPS 13 9380 OS: Fedora 31 GNOME: Version 3.34.3 Kernel: Linux version 5.4.17-200.fc31.x86_64 (mockbuild@bkernel04.phx2.fedoraproject.org) (gcc version 9.2.1 20190827 (Red Hat 9.2.1-1) (GCC)) #1 SMP Sat Feb 1 19:00:13 UTC 2020

keeferrourke commented 4 years ago

That is... unexpected. How did you install the cursors?

I built and tested them on my Fedora 31 machine, and have not experienced any crashes.

Can you attach a crash log?

MrPointer commented 4 years ago

I tried installing now directly from the pling package, and apparently it worked just fine. Before that though, I've installed it through the tcg copr repo as mentioned in the issue.

I can try and attach a crash log if you'd like, but how can I do that?

keeferrourke commented 4 years ago

I've installed it through the tcg copr repo as mentioned in the issue.

Heh. My bad, hadn't had a coffee yet when I read this 😓

I don't own the tcg copr repo, so can't officially support it. Maybe try filing an issue with @LaurentTreguier here?

I can try and attach a crash log if you'd like, but how can I do that?

IIRC Gnome's bug reporter should catch the crash and show a log in its GUI? I'll leave this bug open for now though and try to reproduce the crash on my Fedora machine.

LaurentTreguier commented 4 years ago

I probably missed some changes in the build process since version 3. I'll go check and get back to this issue

LaurentTreguier commented 4 years ago

Although it's not crashing GNOME for me (3.34.4 on Fedora 31), it shows an ugly default X11 cursor, so it's definitely broken

LaurentTreguier commented 4 years ago

I can see those 3 error messages during the build:

Do you have any idea what could cause them ? Fedora is packaging the beta version of Inkscape 1.0.0, which probably has some breaking changes compared to 0.92 (or whichever version is the current stable)

keeferrourke commented 4 years ago

Definitely an error in the build.sh on master. Working on a fix right now!

keeferrourke commented 4 years ago

@LaurentTreguier merged the fix. Should be able to build using either inkscape 0.9x or 1.0.x now.

For your package, you probably want to build the biggest versions:

./build.sh -d xxxhd
./build.sh -d xxxhd -t light
LaurentTreguier commented 4 years ago

Thanks! I just tried to build the latest commit, but my Fedora is acting up and started freezing when starting a container to build the package, so there might be a few days before this is fixed in my COPR repo

LaurentTreguier commented 4 years ago

So, I've managed to build the package without Fedora freezing, and the latest commit seems to fix it, the cursor shows up normally. I've launched a new build in COPR, which should be finished soon enough

MrPointer commented 4 years ago

@LaurentTreguier Great, I'll test it tomorrow then and update if it works

Update: @LaurentTreguier Have you renamed the package? dnf only seems to find the 3-5 version... I tried both explicitly and implicitly

LaurentTreguier commented 4 years ago

I haven't renamed the package. I've never put out a 3-5 version as far as I remember, is that package really coming from my COPR ?

MrPointer commented 4 years ago

Huh, interesting... it seems that updates also contains this package, and it's probably what I've installed from the beginning :laughing:

However, I see now that there's a 4-2 version as well - What's the difference and should I install it?

LaurentTreguier commented 4 years ago

Interesting indeed. Both la-capitaine-icon-theme and la-capitaine-cursor-theme are officially available in Fedora now. For the cursors, 3-5 is actually the first version to have been published. The latest version I have put out is 4-2.20200223, this should be a functional one

keeferrourke commented 4 years ago

Huh. News to me! lol

MrPointer commented 4 years ago

@LaurentTreguier I'm probably misusing dnf, but when I try to explicitly install the package, i.e. sudo dnf install la-capitaine-cursor-theme-4-2.20200223.fc31 - I get the following message:

All matches were filtered out by exclude filtering for argument:

Googling it didn't help either, I haven't found any helpful resource. Any help?

BTW, trying to install the general la-capitaine-cursor-theme results in the 3-5 version supplied by the updated repo to be listed for installation

LaurentTreguier commented 4 years ago

Does manually refreshing the repos with sudo dnf distro-sync --refresh help ?

MrPointer commented 4 years ago

@LaurentTreguier Nope

LaurentTreguier commented 4 years ago

That's strange. Have you tried removing and re-enabling the repo ? I know this sounds stupid, but the "turn it off and on again" solution actually tands to work out more often than it should. Other than this, I have no idea what could be the problem. I haven't done anything special when updating the package

MrPointer commented 4 years ago

The same result, unfortunately... I honestly don't know what went wrong on my behalf Strangely enough, the la-capitaine-icon-theme package is detected successfully and can be installed flawlessly

yan12125 commented 1 year ago

(From https://github.com/keeferrourke/capitaine-cursors/issues/43#issuecomment-590098233)

Cannot parse integer value ?60.0? for -w

FWIW, I can get rid of such warnings by forcing integers from bc:

--- a/build.sh  2022-10-18 00:54:24.319840199 +0800
+++ b/build.sh  2022-10-18 00:54:33.839454372 +0800
@@ -104,8 +104,8 @@
 function render {
   name="x$1"
   variant="$2"
-  size=$(echo "$SVG_DIM*$1" | bc)
-  dpi=$(echo "$SVG_DPI*$1" | bc)
+  size=$(echo "$SVG_DIM*$1/1" | bc)
+  dpi=$(echo "$SVG_DPI*$1/1" | bc)

   mkdir -p "$BUILD_DIR/$variant/$name"
   find "$SRC/svg/$variant" -name "*.svg" -type f \

The trick is inspired by https://unix.stackexchange.com/a/320703