Bug 1342507

Thanks, fixed. 
BTW, the shebang fix commands in the given link are incorrect and need a few changes to work correctly:

find -type f -exec sed -i '1s=^#!/usr/bin/\(python\|env python.*\)$=#!%{__python3}=' {} \;

Hi Hedayat!

Thank you very much for the heads up! The %{__python3} macro was a later addition, so I did not notice the clash with my favourite separator—the underscore. It should be fixed now!

The linu thus should be:

find -type f -executable -exec sed -i '1s=^#!/usr/bin/\(python\|env python\)=#!%{__python3}=' {} +

Notice it's a bit different than the one you posted, because this one works also with e.g. the shebang `#!/usr/bin/python --some-flag`, which is important. Your sed line only allows for flags following `/usr/bin/env python` as you put the `.*` inside it's branch.

Thanks again for the bug report!

Hi & thank you for the description.
However, there is a problem with your version too: it modifies the following shebangs too:

/usr/bin/env python3
/usr/bin/python3

to
/usr/bin/env python33
/usr/bin/python33

so you should exclude the cases where python is followed by 3! :)

Ah, good point! How about this?

find -type f -exec sed -i '1s=^#!/usr/bin/\(python\|env python\)[23]\?=#!%{__python3}=' {} +

That should work every time!

Yes, it looks great :)