Terminal command doesn't work if x-terminal-emulator is unavailable

[OPNA2608]

Terminal command doesn't work if x-terminal-emulator is unavailable

[AlanGriffiths]

@mattkae

            for (auto const& terminal : POSSIBLE_TERMINALS)
            {
                if (external_client_launcher.launch({terminal}) > 0)
                    break;
            }

Doesn't do what you think. external_client_launcher.launch() does a fork()/exec() and returns the pid of the forked process. The fork() will always succeed, even if the exec() doesn't.

The approach I took with Miriway was to allow the user to configure the terminal command (and provide a fallback to a script that tries to identify an installed terminal).

[mattkae]

Doesn't do what you think. external_client_launcher.launch() does a fork()/exec() and returns the pid of the forked process. The fork() will always succeed, even if the exec() doesn't.

Oh that's unfortunate

The approach I took with Miriway was to allow the user to configure the terminal command (and provide a fallback to a script that tries to identify an installed terminal).

I think that I will follow this same approach

[mattkae]

Hi agan @OPNA2608 👋 Would you give this PR try when you get a chance: #40

It should work in both a snap and non-snap context. You should be able to install the AMD snap from this page: https://github.com/mattkae/miracle-wm/actions/runs/8038897957?pr=40. I'll add the fix to a release/0.1.1 later this week if all is good.

Thanks for finding the bug!

[mattkae]

Also, I am testing out #39 now. I will cherry-pick it into release/0.1.1 alongside the terminal fix once approved 👍