Needlessly creating the executable path preference for the ufraw-batch executable on linux caused the value to be saved as the startup directory, which in turn caused a crash when trying to extract the embedded jpg.

This commit is contained in:
Bill Ferguson
2020-11-17 18:53:54 -05:00
parent d172ac7049
commit b5b49245fa

View File

@@ -204,24 +204,26 @@ end
to see if the executable is there. to see if the executable is there.
]] ]]
local executable = "ufraw-batch" if dt.configuration.running_os ~= "linux" then
local ufraw_batch_path_widget = dt.new_widget("file_chooser_button"){ local executable = "ufraw-batch"
title = _("Select ufraw-batch[.exe] executable"), local ufraw_batch_path_widget = dt.new_widget("file_chooser_button"){
value = df.get_executable_path_preference(executable), title = _("Select ufraw-batch[.exe] executable"),
is_directory = false, value = df.get_executable_path_preference(executable),
changed_callback = function(self) is_directory = false,
if df.check_if_bin_exists(self.value) then changed_callback = function(self)
df.set_executable_path_preference(executable, self.value) if df.check_if_bin_exists(self.value) then
df.set_executable_path_preference(executable, self.value)
end
end end
end }
} dt.preferences.register("executable_paths", "ufraw-batch", -- name
dt.preferences.register("executable_paths", "ufraw-batch", -- name "file", -- type
"file", -- type _('multi_os: ufraw-batch location'), -- label
_('multi_os: ufraw-batch location'), -- label _('Installed location of ufraw-batch. Requires restart to take effect.'), -- tooltip
_('Installed location of ufraw-batch. Requires restart to take effect.'), -- tooltip "ufraw-batch", -- default
"ufraw-batch", -- default ufraw_batch_path_widget
ufraw_batch_path_widget )
) end
--[[ --[[
Add a button to the selected images module in lighttable Add a button to the selected images module in lighttable