Skip to content

Upgrade gnome* from 44 -> 46#155

Open
rj-xy wants to merge 1 commit intotom-james-watson:masterfrom
rj-xy:rj-xy-gnome-46
Open

Upgrade gnome* from 44 -> 46#155
rj-xy wants to merge 1 commit intotom-james-watson:masterfrom
rj-xy:rj-xy-gnome-46

Conversation

@rj-xy
Copy link
Copy Markdown

@rj-xy rj-xy commented Jun 1, 2024

image

Tested on Kubuntu 24.04

image

@tom-james-watson
Copy link
Copy Markdown
Owner

Hi - thanks for this! What OSes/versions have you tried this build on? Any issues?

@rj-xy rj-xy force-pushed the rj-xy-gnome-46 branch from dbfa613 to 8a83fad Compare June 3, 2024 19:58
@rj-xy rj-xy closed this Jun 3, 2024
@rj-xy rj-xy reopened this Jun 3, 2024
Comment thread README.md
```bash
sudo apt install xdotool libgtk-3-dev libgirepository1.0-dev python3-venv gir1.2-keybinder-3.0 libkeybinder-dev desktop-file-utils
# Debian/Ubuntu
sudo apt install xdotool libgtk-3-dev libgirepository1.0-dev python3-venv gir1.2-keybinder-3.0 libkeybinder-dev desktop-file-utils pipenv flatpak-builder libcairo2-dev libxt-dev
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added pipenv flatpak-builder libcairo2-dev libxt-dev

Comment thread README.md
If so, run the following command:

```bash
sudo pip3 install pipenv
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pipenv moved to line 89

Comment thread meson.build
@@ -1,5 +1,5 @@
project('emote',
version: '4.0.2',
version: '4.1.0',
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This must have updated auto-magically with the build steps I followed

Comment thread Pipfile

[requires]
python_version = "3.11"
python_version = "3.12"
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to revert this, and the Pipfile.lock - I just used the version of Python I had installed locally

@rj-xy rj-xy force-pushed the rj-xy-gnome-46 branch from 8a83fad to d78bbc8 Compare June 3, 2024 20:21
@rj-xy rj-xy force-pushed the rj-xy-gnome-46 branch from d78bbc8 to 4729bd0 Compare June 3, 2024 20:23
@rj-xy
Copy link
Copy Markdown
Author

rj-xy commented Jun 3, 2024

Ready for review @tom-james-watson

@rj-xy
Copy link
Copy Markdown
Author

rj-xy commented Jun 5, 2024

@tom-james-watson when you have a moment to review, just let me know what I have to revert/fix/change, and I'll sort it out for you.

I've been using the local build version for 3 days without issues

@calejvaldez
Copy link
Copy Markdown

@tom-james-watson Another user here! I really encourage this PR to be merged. The flatpak version keeps telling us to update, and it's getting a bit annoying atp.

(By the way, thank you for the PR, @rj-xy!! ❤️ )

@tom-james-watson
Copy link
Copy Markdown
Owner

This was fixed by a flathub maintainer directly: flathub/com.tomjwatson.Emote#6.

He then made a PR with the relevant changes back to this repo. Do you mind updating this PR to include the changes he's making on that PR? For example his PR is failing to build the flatpak via the GitHub action, I think because it's missing changes you have here.

When you update the PR, I can kick off the GitHub action to ensure everything is working!

Sorry for the delay on this - life has got in the way, plus I'm not on Linux that much these days.

@rj-au
Copy link
Copy Markdown

rj-au commented Jul 9, 2025

@tom-james-watson can you close this PR and use #158 instead please.

If you think it's still worth including the other changes I made in this PR, let me know and I'll grab the changes from #158 and update everything I can (this PR dates back to June 2024)

Lol, I'm still on that DEV build 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants