-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
fix(x11/cinnamon): use desktopappinfo from giounix not gio #26744
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@sabamdarif If you cherry-pick this PR, does it work as an alternative solution? |
|
Another X11 GUI affected by an extremely similar problem is |
i will try |
|
yes @robertkirkman it did work, i understand now that is the real issue thanks |
|
very good, and should it be called "from" or "form"? I thought it should be "from" |
|
o yes wait |
from |
|
In order to pass the CI it will need to have |
|
That should have already been caught by the linter. termux-packages/scripts/lint-packages.sh Lines 124 to 142 in b982d53
Why wasn't it... |
oh yes i forgot about that |
|
@TomJo2000 I don't fully understand how the linter works, but just looking at the part you highlighted, where does the |
That's some git-rev syntax1. Footnotes |
|
@TomJo2000 ok that's great but it doesn't work on my computer (a Termux right now) when I try to test it. How is it meant to work in CI, do I have it set up wrong? Also, can this condition detect a problem if |
|
It's supposed to make a diff of the entire branch using
|
|
If you can see my screenshot, do you see as well how my |
I'm guessing there's no |
Maybe the CI doesn't have the correct information configured in |
|
I'll throw a |
could you edit the commit to change that? |
ok |
|
i use git commit --amend |
|
oh my bad i understand it wrong , wait |
|
Ok wonderful, now it's ready! I will merge it in 24 hours and I guess I will also merge my similar |
close #26742