tvm icon indicating copy to clipboard operation
tvm copied to clipboard

[FEAT] Avoid the usage of `sudo`

Open MoisesGSalas opened this issue 2 years ago • 3 comments

Is your feature request related to a problem? Please describe. Currently tvm adds the tutor command to your PATH by symlinking the tutor_switcher script to /usr/local/bin/tutor. This requires sudo permissions, which seems excessive, and the global tutor can cause confusion when using tutor in a virtualenv outside of tvm. I would prefer, If I were to install tvm on a virtualenv, to limit the tvm managed tutor command to the scope of that virtualenv.

Describe the solution you'd like I think we can use the script option from setup.py to create a tutor-shim that calls the tutor switcher (essentially a shell script). This script will be added to the PATH automatically when installing the package and should fill the same role.

Describe alternatives you've considered None

Additional context I have not tested this, but I think it could work.

MoisesGSalas avatar Mar 09 '23 19:03 MoisesGSalas

We'll look into this in our next sprint (starting approximately a week and a half from now). Thanks for the report!

mariajgrimaldi avatar Mar 10 '23 13:03 mariajgrimaldi

Hi @MoisesGSalas We have been thinking about this, and we are planning to move the symlink to userhome/.local/share/tutor instead of /usr/local/bin/tutor, maybe that could help us to avoid the usage of sudo.

Also I remind you that we don'r recommend install tvm in a virtualenv.

We hope have good news soon.

Alec4r avatar May 10 '23 22:05 Alec4r

@santiagosuarezedunext I think we should put this again in our radar since it seems like a ggod request. What do you think?

Asespinel avatar Dec 21 '23 00:12 Asespinel