aosucas499 / minino-TDE

ISO modificada del sistema operativo Minino para su uso en dispositivos antiguos. #TDE
GNU General Public License v3.0
8 stars 2 forks source link

Incluir update-minino en listado sudoers #75

Closed jasvazquez closed 3 years ago

jasvazquez commented 3 years ago

Tras "pelear" en la tarea #32 con los permisos y tener que estar introduciendo contraseñas para poder actualizar minino cada vez que aparezca una nueva release, nos planteamos la posibilidad de incluir el script /usr/bin/update-minino dentro del fichero sudoers para evitar al usuario tener que hacerlo.

aosucas499 commented 3 years ago

Archivo añadir sudoers: commit 3609990

function sudoersUpdate: commit c239ba

añadido sudo a updateMinino.desktop: commit 71b008c

aosucas499 commented 3 years ago

He añadido /usr/bin/update-minino a sudoers creando una función para que lo añada en update-minino. He modificado también tu código que creaba el archivo update-minino.desktop de autoinicio, simplemente añadiendo sudo, ya que sin él seguiría pidiendo contraseña en cada inicio. Con esto resolvemos el problema.

Por ahora en cada inicio siempre ejecuta todo el update-minino, supongo que tendré que crear una release para que compruebe que es la actual y no ejecute en cada inicio hasta que sea diferente.

aosucas499 commented 3 years ago

Creo que con esto tenemos solucionado el problema.

Cada destacar que update-minino se ejecuta en cada inicio. No lo veo un gran problema sino fuese porque sale la ventana de xterm siempre.

Una fotografía de la primera vez que detecta una release nueva:

VirtualBox_minino TDE_27_04_2021_21_37_22

Y esta otra de cada reinicio aunque no haya release nueva:

VirtualBox_minino TDE_27_04_2021_21_38_55

aosucas499 commented 3 years ago

Solucionado el problema anterior con la release 1.2.4 modificando otra vez el .desktop de autoinicio y eliminando xterm, que supongo pusiste como una manera de visualizar su ejecución. Muy inteligente!

Hay que destacar que la función autostartupdateminino no detecta el ambio en el archivo .desktop, por lo que lo he borrado manualmente para que se actualizara con update-minino e incluyese los cambios de borrar el comando xterm -e y el añadido de sudo a update-minino.

jasvazquez commented 3 years ago

Parece que te estorba xterm pero no me gusta la idea de que el sistema esté haciendo cosas por detrás y no sepamos qué ocurre ni (de momento) tengamos forma de notificar al usuario el resultado del proceso.

Había optado por usar xterm precisamente para poder interactuar con el script y, aunque esté en la lista de sudoers (buen trabajo) creo que debería seguir visible mientras no tengamos una alternativa 🤔

Abro tarea temporalmente hasta resolver la duda y/o tomar una decisión 👍

aosucas499 commented 3 years ago

Update-minino está agregado a sudoers con éxito. Se ejecuta en cada inicio y no pide clave. Si te gusta por ahora que lo tengamos un tiempo mostrando la terminal lo vuelvo añadir. 5929b096bb8af8cdebe520ceef3a41f29367ee74

jasvazquez commented 3 years ago

Preferiría (mientras no tengamos otra forma) dejarlo así: sale "barato" y nos ahorramos "sustos" 😉

aosucas499 commented 3 years ago

Cierro tras comprobar que funciona perfectamente en release 1.2.5. Se ejecuta en cada inicio sin pedir contraseña y se ve gráficamente con xterm.

jasvazquez commented 3 years ago

Buen trabajo, compi 👍🏻