nautilus-open-any-terminal icon indicating copy to clipboard operation
nautilus-open-any-terminal copied to clipboard

Add flatpak support

Open polyjitter opened this issue 2 years ago • 14 comments

Added a system for flatpak support, as flatpak versions of commands are different.

This fixes #46.

polyjitter avatar Jul 03 '22 03:07 polyjitter

Doesn't actually open the working directory, not sure why. Will investigate further.

polyjitter avatar Jul 03 '22 03:07 polyjitter

I do not like the current approach. Please add a new gschema-option: use-flatpack.

In your exact scenario this would be fine. But this will introduce unknown behavior if it is installed locally and via flatpack.

Stunkymonkey avatar Jul 03 '22 08:07 Stunkymonkey

Out of scope for my skills, unfortunately.

polyjitter avatar Jul 03 '22 14:07 polyjitter

Spoke too soon!

polyjitter avatar Jul 03 '22 15:07 polyjitter

Hey, just wanted to say that this should be finished as per your requests to modify it :)

polyjitter avatar Jul 08 '22 05:07 polyjitter

Does this work with blackbox yet? i tried to set to blackbox but it seems to fallback to gnome-terminal Tested on fedora 36 with blackbox 0.10.1

ppvan avatar Jul 10 '22 13:07 ppvan

Works great!!

radical4ever avatar Jul 10 '22 14:07 radical4ever

Yep! This PR should be completed, just waiting for review.

polyjitter avatar Jul 10 '22 17:07 polyjitter

disclaimer: i'm pretty positive that this isn't a problem of this PR, but it just occurred to me trying to switch from the user-scope pip install to the installation through setup.py.

not sure how exactly compiling the dconf schema works under the hood, but i ended up with nautilus crashing because:

Settings schema 'com.github.stunkymonkey.nautilus-open-any-terminal' does not contain a key named 'use-flatpak'

thing is, i installed nautilus-open-any-terminal first using "pip install --user" and then uninstalled it using pip (leaving the user-scoped schema in place, i guess?). now to test this fork, i ran "sudo ./setup.py install", which installed the python package system-wide and asked me to recompile the system-wide dconf schema (again, leaving the user dconf schema in place). apparently, running "glib-compile-schemas ~/.local/share/glib-2.0/schemas/" fixed this for me, output was:

No schema files found: removed existing output file.

maybe this is something setup.py should handle, @Stunkymonkey?

sk22 avatar Jul 15 '22 18:07 sk22

Does this work with blackbox yet? i tried to set to blackbox but it seems to fallback to gnome-terminal Tested on fedora 36 with blackbox 0.10.1

it does for me!

sk22 avatar Jul 15 '22 18:07 sk22

maybe you need to full uninstall the old version (stunkymonkey one) and install the extension from the pull request

radical4ever avatar Jul 15 '22 20:07 radical4ever

maybe you need to full uninstall the old version (stunkymonkey one) and install the extension from the pull request

i did uninstall the stunkymonkey version using pip, which i assume left the dconf schema in place… actually, i just noticed i didn’t see the readme’s uninstall section, so my bad!

sk22 avatar Jul 15 '22 21:07 sk22

flatpak can be used in user space, so you still need to consider the key --user

flatpak --user run com.raggesilver.BlackBox

DeadMozay avatar Jul 18 '22 08:07 DeadMozay

sorry I was on holidays without any electronic device.

I think the implementation looks good. But like DeadMozay mentioned, that supporting a user mode would be nice.

Stunkymonkey avatar Jul 18 '22 17:07 Stunkymonkey

I have reworked this PR to support flatpaks system and user mode here.

As this PR has been sitting here for quiet some time, should i open a new PR?

sapphic-kitten avatar Sep 08 '22 08:09 sapphic-kitten

I have reworked this PR to support flatpaks system and user mode here.

As this PR has been sitting here for quiet some time, should i open a new PR?

No idea..

ppvan avatar Sep 08 '22 08:09 ppvan

Any way, i have made this extension This specific for Blackbox only

ppvan avatar Sep 08 '22 08:09 ppvan

I have reworked this PR to support flatpaks system and user mode here.

As this PR has been sitting here for quiet some time, should i open a new PR?

@Klapstuhl I would appreciate you opening a new PR.

Stunkymonkey avatar Sep 14 '22 19:09 Stunkymonkey

closed via #60

Stunkymonkey avatar Sep 15 '22 19:09 Stunkymonkey