Re: [PATCH 1/2] winebrowser: Try other browsers on non-zero exit code.

[Vincent Povirk]

Using system is usually a bad idea for security reasons.

I also don't think waiting for the browser command to return makes
sense. It may keep winebrowser running longer than expected, and we
don't know for sure what the return value means.

[PATCH 1/2] winebrowser: Try other browsers on non-zero exit code.

[Zebediah Figura]

On 08/01/2017 12:58 PM, Vincent Povirk wrote:
> Using system is usually a bad idea for security reasons.

Will fix, thanks.
> I also don't think waiting for the browser command to return makes
> sense. It may keep winebrowser running longer than expected, and we
> don't know for sure what the return value means.

How is this different from the current behaviour?