candlerb commented on issue #5369: "#!/usr/bin/env python" doesn't work on Ubuntu 18.04 / python3 URL: https://github.com/apache/pulsar/issues/5369#issuecomment-548330379 Actually, I believe the shebang lines in these python source files are spurious, since they do not have the eXecute bit set anyway: ``` $ tar -tvzf apache-pulsar-2.4.1-bin.tar.gz | grep python_instance -rw-r--r-- jia/staff 10165 2019-08-28 08:59 apache-pulsar-2.4.1/instances/python-instance/python_instance_main.py -rw-r--r-- jia/staff 3384 2019-08-28 08:59 apache-pulsar-2.4.1/instances/python-instance/tests/test_python_instance.py -rw-r--r-- jia/staff 20104 2019-08-28 08:59 apache-pulsar-2.4.1/instances/python-instance/python_instance.py ``` Shebangs are only needed if you `exec()` the file directly. So I'm guessing pulsar is explicitly running the interpreter, like `/usr/bin/python <path-to-file>.py`, but I can't find that code right now. If it's true, then I think it would be good idea to remove all the shebang lines to avoid confusion.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
