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

LinGui: add support for running a command after an encode completes #6029

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

samhutchins
Copy link
Contributor

@samhutchins samhutchins commented May 6, 2024

Description of Change:

My lack of C and glib/gtk experience may very well be showing in full force here...

macOS and Windows can send the encoded file to an external application after the encode completes, this change adds that functionality for the Linux GUI as well. I'm not sure if this is the best way to invoke an external command

Tested on:

  • Windows 10+ (via MinGW)
  • macOS 10.13+
  • Fedora Linux

@robxnano
Copy link
Contributor

Looks good, but have you considered making this more useful in the Flatpak version? Users will probably expect the command to run outside the sandbox. You could do this by prepending flatpak-spawn --host -- to the argv of the command when running in the Flatpak environment, which you can detect with if (g_access("/.flatpak-info", F_OK) == 0). Note that this would also need permission to talk to the org.freedesktop.Flatpak D-Bus name.

@samhutchins
Copy link
Contributor Author

I had not, I've never really used the flatpak version. Thanks for the pointers on where to start, I'll look into it

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

Successfully merging this pull request may close these issues.

None yet

2 participants