search

eduNEXT / tvm

Tutor Version Manager and Tutor enVironment Manager
https://tvm.docs.edunext.co/
GNU Affero General Public License v3.0
24 stars 0 forks source link

[FEAT] Avoid the usage of `sudo` #67

Open MoisesGSalas opened 1 year ago

MoisesGSalas commented 1 year ago

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.

mariajgrimaldi commented 1 year ago

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

Alec4r commented 1 year ago

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.

Asespinel commented 10 months ago

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