Skip to content

Commit

Permalink
Remove python27 hack
Browse files Browse the repository at this point in the history
A "temporary" hack was added six years to support forcing UIforETW to
use Python 2.7. That hack was probably never needed and is certainly
undesirable now.

This affects issue #13.
  • Loading branch information
randomascii committed Jul 12, 2021
1 parent cf87c3f commit 3aa62e6
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions UIforETW/Utility.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -846,15 +846,6 @@ std::wstring FindInPath(const std::wstring& exeName)

std::wstring FindPython()
{
const std::wstring pytwoseven = GetEnvironmentVariableString(L"python27");

// Some people, like me, (Alexander Riccio) have an environment variable
// that specifically points to Python 2.7.
// As a workaround for issue #13, we'll use that version of Python.
// See the issue: https://github.com/google/UIforETW/issues/13
if (!pytwoseven.empty())
return pytwoseven;

// First look for python.exe. If that isn't found then look for
// python.bat, part of Chromium's depot_tools
for (const auto& exeName : { L"python.exe", L"python.bat" })
Expand Down

1 comment on commit 3aa62e6

@ariccio
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

celebrates

Please sign in to comment.