Skip to content
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

nautilus: Add compatibility with Nautilus 43 #2115

Merged
merged 2 commits into from
Apr 20, 2023
Merged

Conversation

jbicha
Copy link
Contributor

@jbicha jbicha commented Sep 1, 2022

Nautilus 43 and nautlius-python 4 introduce major changes to the extension API. (Nautilus 43 also was ported to GTK4.)

The GNOME 43 Release Candidate is being released in the next few days and the stable release will happen on September 21.

I guess this merge proposals breaks the keyboard shortcut for Nautilus > Open in Tilix. (By the way, Ubuntu Desktop uses Ctrl+Alt+T for their systemwide Launch a Terminal feature.)

This isn't needed since
https://gitlab.gnome.org/GNOME/nautilus-python/-/commit/8c88de8da42

And interferes with being able to use this extension
on Nautilus 43+
@jbicha
Copy link
Contributor Author

jbicha commented Sep 1, 2022

For the CI, Debian Testing is known to be broken (tilix is currently unbuildable there). Sorry.

@jbicha
Copy link
Contributor Author

jbicha commented Sep 4, 2022

There were some fixes pushed to Debian but it will probably take 5 or 6 days for them to migrate to Debian Testing.

data/nautilus/open-tilix.py Outdated Show resolved Hide resolved
@ximion
Copy link
Collaborator

ximion commented Sep 6, 2022

Looks good to me!

data/nautilus/open-tilix.py Outdated Show resolved Hide resolved
@jbicha
Copy link
Contributor Author

jbicha commented Sep 6, 2022

Nautilus 43 and nautilus-python 4 have made major changes to the API

Nautilus.LocationWidgetProvider has been dropped from the API
without a replacement

https://gnome.pages.gitlab.gnome.org/nautilus-python/nautilus-python-migrating-to-4.html
@jbicha
Copy link
Contributor Author

jbicha commented Sep 10, 2022

yay, Debian Testing is fixed and the CI is passing again.

@ximion
Copy link
Collaborator

ximion commented Apr 20, 2023

Awesome, thank you for the patch!

@ximion ximion merged commit f9ef151 into gnunn1:master Apr 20, 2023
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.

3 participants