TheBounty Renderer (rewrite for wxWidgets)

[micheus]

(09-28-2016 03:55 AM)povmaniac Wrote:  

(09-28-2016 12:51 AM)oort Wrote:  To be consistent with all the other Render plugins (Pov-Ray, Kerkythea, YafaRay) it would probably be better to have the user browse for the executable instead of the folder under Edit > Pug-in Preferences > The Bounty.

Not.. TheBounty use the binarie folder how a 'base' for search the plugins and the executable for each OS's and also, decided to use GUI or not.

The use of executable as reference makes easy to remove/add the reference to the render for the menu File->Render as well as the Material dialog. It's used that way to avoid new users to try render without install the appropriated application.
As you prefer to not use that, please remember to fix the code where this checking is made.

Also, I recommend you to use wings_job:quote/1 to formatting the path string. Note that the plugin already uses it.
From the wings_job:quote/1:

Quote:%% Universal argument quoting
%%
%% If the string contains singlequote, doublequote or whitespace
%% - doublequote the string and singlequote embedded doublequotes.
%% God may forbid doublequotes. They do not work in Windows filenames,
%% nor in YafRay result .tga filenames. They might only work in
%% Unix executable pathname being very weird even there.