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.

Sign up for GitHub

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

Jump to bottom

Another way to fix #697 #702

Open

DarthGandalf wants to merge 1 commit into shutter-project:master from DarthGandalf:run3

Member

DarthGandalf commented Sep 11, 2024

No description provided.


          Another way to fix shutter-project#697

8028d6c

DarthGandalf mentioned this pull request

Fix #697: a crash in the Watermark plugin if screenshot name includes spaces #698

Open

Member

Photon89 commented Dec 30, 2024

I had another look at it today. With the system call I get the following behavior:

The first time that apply_effect() is executed (when launching the plugin dialog), it runs without error.
The second time that it is executed (by pressing the reload or save button), the system call silently fails.
It still works in the first run and fails on subsequent ones when all the variables in the argument are substituted by fixed values (besides of $tmpfilename).

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

Labels

None yet