Changed by andrew_b (@aborodin) on Sep 18, 2018 at 12:41 UTC

Replying to vascom (#3936):

MC contain two file with python2 shebang:

/usr/libexec/mc/extfs.d/s3+
/usr/libexec/mc/extfs.d/uc1541

That's not true. There is no any python2 dependency in the mc.

In the source tree, shebang in those files is

The substitution of @python@ is done in configure state.
In my system I have

#! /usr/bin/python

Changed by vascom (@Vascom) on Sep 18, 2018 at 13:01 UTC (comment 2)

As I see it set in configure.ac by

AC_PATH_PROG([PYTHON], [python], [/usr/bin/python])

So if I change it to python3 path the above two vfs plugins will normal work?

Changed by andrew_b (@aborodin) on Sep 18, 2018 at 13:08 UTC (comment 2.3)

I don't know. Personally I don't use these plugins.

Changed by vascom (@Vascom) on Sep 18, 2018 at 13:12 UTC (comment 4)

Then need test plugins with python3 and rewrite or remove them.

Changed by vascom (@Vascom) on Apr 26, 2019 at 11:35 UTC (comment 5)

Hi.

Any progress?

Changed by gryf (gryf_esm@….pl) on Sep 15, 2019 at 7:54 UTC (comment 6)

As for uc1541, it supports both - python2 and python3, although I recommend to update it to the latest version (available on https://github.com/gryf/uc1541), since there are couple of tweaks and fixes.

I'd also recommend to use python in shebang, probably this form is best:

#!/usr/bin/env python

so that, default python interpreter defined in a system will be used. Some systems allows to set default interpreter to be python3 and /usr/bin/python is a link to the chosen interpreter.

According to https://legacy.python.org/dev/peps/pep-0394/ python interpreter can be used for both - python2 and python3 compatible scripts, and uc1541 currently is compatible with both versions.

Changed by andrew_b (@aborodin) on Sep 21, 2019 at 16:47 UTC (comment 7)

• Owner set to andrew_b
• Status changed from new to accepted
• Milestone changed from Future Releases to 4.8.24

Changed by andrew_b (@aborodin) on Sep 21, 2019 at 16:53 UTC (comment 8)

• Branch state changed from no branch to on review

Branch: 3936_uc1541
[dc46dc0]

Changed by andrew_b (@aborodin) on Sep 21, 2019 at 16:54 UTC (comment 9)

• Status changed from accepted to testing
• Branch state changed from on review to merged
• Votes set to committed-master
• Resolution set to fixed

Merged to master: [f788bd0].

Changed by andrew_b (@aborodin) on Sep 21, 2019 at 16:59 UTC (comment 10)

• Status changed from testing to closed

uc1541 updated.

Discussion about s3 welcome to #3904.