¿Creeis que sería buena idea meter los updaters en un repositorio de control de versiones (git, subversion...)?
Así lo mismo hacemos todo el proceso más "limpio"




Saludos

mmmm  ,  creo que puedo hacerlo , ...le he cogido la idea a gemnis_demon  , de copiar los updaters a un directorio temporal ... y mutilar todo lo que no necesito , es dceir todas las llamadas a las funciones que no necesito ...todas menos


F_variables


despues ejecutarlos todos y mostrar por pantalla los campos que quiera de dicha funcion.


los ultimos son todos muy estanbdard , pero los primeros algunos tienen funciones que solo existen alli , asi que estoy en proceso de identificarlas todas para anularlas en el tmpo.

se me ocurren cosas ,para prevenir futuras funciones "inventadas" , que no esten en esa lista de seds , al final.


todo updater acaba las funciones y despues ya tiene


###################################
## BLOQUE PRINCIPAL DE EJECUCION ##
###################################
# Si se cierra el script inesperadamente, ejecutar la funcion de limpieza
trap f_exitmode SIGHUP SIGINT


molaria , identificar la linea esa azul ...ELIMINAR TODO LO QUE HAYA DETRAS DE ELLA ...que son todas las llamadas a funciones...  y por ultimo  insertar la unica llamada que me interesaria

F_variables


pero mi bash , es el que es 

Este script va recorriendo los updaters uno por uno y mostrando la versión obtenida, y notificando los updaters que no han conseguido obtener la versión online.

#!/bin/bash

find /opt/wifislax-updaters -type f -iname "*_updater"|\
while read LINE; do

unset WEB VERSION
egrep "PRGNAM=|WEB=|RAMA=|DOWNLOAD=|LAGRANWEB=|ELGRANARCHIVO=|VERSION=|VERSION_REAL=|FICHERO=" "$LINE" >/tmp/$$.tmp
source /tmp/$$.tmp 2>/dev/null; rm -f /tmp/$$.tmp
[ "$VERSION" ] && echo "$PRGNAM-$VERSION"||echo "${LINE/*\/} ERROR"
done

Lo que he hecho es grepear las variables que se necesitan para obtener la versión, como se puede ver en esta línea:

egrep "PRGNAM=|WEB=|RAMA=|DOWNLOAD=|LAGRANWEB=|ELGRANARCHIVO=|VERSION=|VERSION_REAL=|FICHERO=" "$LINE" >/tmp/$$.tmp
Aunque fallará con los updaters que no cumplan el estandar, como los del kernel, o como firefox-multi que necesita interacción del usuario para obtener la versión.

jajaja ok 

Me gusta la forma que has usado para comprobar la versión, mola porque no depende de las variables como el mio, y aunque cambien los nombres de las variables seguirá funcionando 

Un par de cosillas, este bucle no cumple ninguna función, así que mejor quitarlo para no liar al que lea el código 

for updater in $UPDATERS; do
cp -rf $UPDATERS $TMP &>/dev/null
rm -rf $TMP/Driver_ATI_updater &>/dev/null
rm -rf $TMP/linux* &>/dev/null
rm -rf $TMP/funciones_updater &>/dev/null
rm -rf $TMP/wifislax-updaters_updater &>/dev/null
rm -rf $TMP/last_update &>/dev/null
rm -rf $TMP/firefox_multi_updater &>/dev/null
done

osea quedaría así:

cp -rf $UPDATERS $TMP &>/dev/null
rm -rf $TMP/Driver_ATI_updater &>/dev/null
rm -rf $TMP/linux* &>/dev/null
rm -rf $TMP/funciones_updater &>/dev/null
rm -rf $TMP/wifislax-updaters_updater &>/dev/null
rm -rf $TMP/last_update &>/dev/null
rm -rf $TMP/firefox_multi_updater &>/dev/null

Y otra cosa, las funciones solo son útiles cuando vas a usarlas mas de una vez en el mismo script, para evitar repetir el mismo código en diferentes partes del script, pero en este caso todo el código se ejecuta una sola vez, por tanto no hace falta crear funciones, funciona igual así:

#!/bin/bash

clear

#BUSQUEDA
TMP=/tmp/Updaters_Test
mkdir -p $TMP
cd /opt/wifislax-updaters
UPDATERS=`find . -type f -name "*_updater"`
echo
echo "Copiando updaters a directorio temporal ..."
echo
for updater in $UPDATERS; do
cp -rf $UPDATERS $TMP &>/dev/null
rm -rf $TMP/Driver_ATI_updater &>/dev/null
rm -rf $TMP/linux* &>/dev/null
rm -rf $TMP/funciones_updater &>/dev/null
rm -rf $TMP/wifislax-updaters_updater &>/dev/null
rm -rf $TMP/last_update &>/dev/null
rm -rf $TMP/firefox_multi_updater &>/dev/null
done

#TEMPORAL
cd $TMP
TEMPUPDATERS=`find . -type f -name "*_updater"`
echo
echo "Modificando copias temporales de updaters ..."
echo
for tempupdater in $TEMPUPDATERS; do
LINEA=`grep -r "f_exitmode" -n $tempupdater | tr -d [a-zA-Z#_.:/]`
FINAL=`cat $tempupdater | wc -l`
ELIMINAR=`expr $FINAL + 1`
sed -i "${LINEA},${ELIMINAR}d" $tempupdater
echo F_variables >> $tempupdater
echo 'if [ -z $VERSION ]; then' >> $tempupdater
echo 'VERSION="Fallido"' >> $tempupdater
echo 'fi' >> $tempupdater
echo 'echo "Programa:$PRGNAM" "VERSION:$VERSION"' >> $tempupdater
done

#MOSTRAR
echo
echo "Testeando updaters..."
echo
sleep 3
for tempupdater in $TEMPUPDATERS; do
sh -c "$TMP/$tempupdater"
done

#LIMPIAR
echo
echo "Limpiando directorio temporal ..."
echo
echo "Bye Bye"
echo



code editado a las 6:10  


he metido directamente los colores en los echo >> , para no tener que llamar a cada vez al funciones_updater , COSA NECESARIA por que si no , los updaters no mostraban colorines ...


 

ahora va super-rapido  ....


se me acaba de pasar por la cabeza, aunque no le veo demasiado sentido , por que esta tool la van a usar 4 gatos que somos los que escribimos updaters y estamos en desarrollo del live.


pero tal vez , deberia antes , mirar si los updaters tienen actualizacion en el ftp , mas por vosotros que por mi , puede que vuestra version local falle, y yo en el ftp ya lo tenga corregido.


o tal vez, MEJOR AUN , se me ocurren cosas mientras escribo ...  

que revise si hay mejoras en el ftp , al acabar , usando la lista de updaters fallidos ... los arreglos siempre son por ese motivo de no pillar la VERSION , asi que , con comparar los que fallaron en el test , por si hay uno nuevo en el ftp , seria suficiente.

Perdon por la ausencia.

La idea de este script era precisamente, para los 4 gatos que hacemos los updaters 

De todas formas, quería proponer una revision de todos los scripts para estanderizar su estructura. Concretamente:

-Añadir 2 variables estdandard, $ARCHIVO (el nombre del archivo que se va a descomprimir) y $DOWNLOAD (URL final que se va a usar para descargar).

para evitar cosas como $PRGNAM-$VERSION-$ARCH-1sg.txz o $WEB/$VERSION/$ARCH/$PRGNAM-$VERSION-$ARCH-1sg.txz

-Añadir una funcion a parte de descomprimir. Es decir, que descomprimir el archivo descargado se encuentre fuera de la funcion F_compilar.

Aqui podría entrar en juego una funcion generica f_descomprimir ,que autodetectase la extension del archivo y usase el programa correspondiente (aunque todavia tengo mis dudas respecto a eso)

Salu2

Ok, lo de descomprimir se puede obviar si lo que pretendes es reducir las funciones.

Respecto a lo otro, es cierto que ARCHIVO y DOWNLOAD existen. Pero existen en algunos. No es algo génerico.

Crear esas variables es una tontería. Sólo hay que juntar el conjunto de variables en una sola. Pero así simplificamos el código.

Salu2

Bueno, si te hace ilusión dividir el código en funciones aya tu  pero ya te digo que no tiene sentido.

Pero plis quita ese bucle que me duele la vista al verlo xd

Es como si pones:

echo "hola hola hola" >/dev/null