[kate] [Bug 456307] Expanding variables in External Tools's Executable field

[Dominik Haumann]

https://bugs.kde.org/show_bug.cgi?id=456307

Dominik Haumann <dhaumann at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|REPORTED                    |CONFIRMED

--- Comment #2 from Dominik Haumann <dhaumann at kde.org> ---
Code to check for the exe:
https://invent.kde.org/utilities/kate/-/blob/master/addons/externaltools/kateexternaltool.cpp#L127

The variable hasexec is used here:
https://invent.kde.org/utilities/kate/-/blob/master/addons/externaltools/kateexternaltoolsview.cpp#L71

And hasexec is set to checkExec on load here:
https://invent.kde.org/utilities/kate/-/blob/master/addons/externaltools/kateexternaltool.cpp#L154

That is a bug. Caching in in hasexec is a bad idea. Instead, checkExec should
expand the variable on the fly.

-- 
You are receiving this mail because:
You are watching all bug changes.