| Re: proposition de patch pour tazwok |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/slitaz Archives
]
> Bonjour, j'avais déjà posté ici il y a quelque jours un premier jet
> d'une idée de patch pour tazwok mais sans provoquer de reaction.
Salut,
> Voici donc la version definitive de ce que j'ai imaginé :
> -------------------------------------------------------------------
> --- /usr/bin/tazwok Sun May 25 22:50:44 2008
> +++ /usr/bin/tazwoknew Sun May 25 22:50:31 2008
> @@ -184,7 +184,13 @@
> fi
> status
> # Set wanted src path.
> - src=$WOK/$WANTED/$WANTED-$VERSION
> + SRC_WANTED="`cat $WOK/$WANTED/receipt | grep
> "SOURCE=" | awk -F '"' '{print $2}'`"
> + SOURCE_WANTED="`eval echo "$SRC_WANTED"`"
> + if [ ! "$SOURCE_WANTED" = "" ]; then
> + src=$WOK/$WANTED/$SOURCE_WANTED
> + else
> + src=$WOK/$WANTED/$WANTED-$VERSION
> + fi
> _pkg=$src/_pkg
> fi
> }
> @@ -247,7 +253,7 @@
> echo "package $PACKAGE (compile)" >> $LOG
> # Set wanted $src variable to help compiling.
> if [ ! "$SOURCE" = "" ]; then
> - src=$WOK/$PACKAGE/$SOURCE-$VERSION
> + src=$WOK/$PACKAGE/$SOURCE
> else
> src=$WOK/$PACKAGE/$PACKAGE-$VERSION
> fi
> @@ -410,7 +416,7 @@
> _pkg=$src/_pkg
> fi
> if [ ! "$SOURCE" = "" ]; then
> - src=$WOK/$PACKAGE/$SOURCE-$VERSION
> + src=$WOK/$PACKAGE/$SOURCE
> _pkg=$src/_pkg
> fi
> cd $WOK/$PACKAGE
> ----------------------------------------------------------------
> Alors, en quoi consiste précisément ce patch me direz vous ?
> En fait, c'est trés simple, à l'heure actuelle, tazwok considère
> que si la variable SOURCE est remplie dans une receipt, le
> répertoire contenant les sources est $WOK/$PACKAGE/$SOURCE-$VERSION.
> Ce fonctionnement n'est pas assez flexible.
C'est vrai que ton patch est bien pensé, ça serait probablement
plus flexible.
> En effet, je me suis rendu compte en créant des receipts que
> le répertoire obtenu à la décompression du tarball peut ne pas
> s'appeler "nom-version" mais simplement "nom" ou encore
> "nom-version/build" ou bien "nom/source" et donc on se retrouve
> obliger de refaire un tarball des sources et de l'héberger soi-même
> ce qui n'a pas de sens. Avec mon patch, il suffit d'indiquer dans
> SOURCE le nom complet du répertoire créé à l'extraction du tarball.
C'est vrai que dans certains cas non standard le nom de répertoire
source peut changer, mais un simple 'mv source $PACKAGE-$VERSION' en
début de genpkg_rules suffit (ex: webkit-r).
Héberger nous même les sources permet d'être 100% GPL en fournissant les
sources des paquets utilisés et ça peut être pratique, dans certain cas
les devs en upstream ne laissent que la dernière version, hors que nous
on utilise encore un version antérieur.
> On peut mettre des sous-répertoires sous la forme
> "répertoire/sous-répertoire" ou encore des variables (ex :
> nom-$VERSION).
Bah, si les sources sont dans répertoire/sous-répertoire, tu cd pour
compiler et fini par un DESTDIR=$WOK/$PACKAGE/$PACKAGE-$VERSION/_pkg
pour avoir $_pkg lors de la génération du paquet.
> Pour que ce patch fonctionne, il faut modifier les receipts
> existantes afin d'ajouter -$VERSION au contenu des variables SOURCE.
.... et donc rebuild les pkgs, adapter la doc.
> Autre chose, il ne faut en aucun cas utiliser la variable SOURCE et
> la variable WANTED dans une même receipt. Ca peut paraître logique et
> pourtant six receipts contiennent à l'heure actuelle ces deux
> variables en même temps. Il faudra les corriger.
Ca fonctione là non ? Why change ?
> J'ai écrit un script qui pour faire des statistiques sur les paquets
> contenus dans un wok local et qui peut, le cas échéant, effectuer les
> corrections citées ci-dessus.
>
> Je vous l'enverrai si besoin et si vous acceptez mon patch.
>
> P.S. : si vous ne comprenez rien à mes explications, désolé, je
> pourrai mieux l'expliquer à l'oral je pense mais là c'est pas facile.
Un projet comme SliTaz c'est pas mal d'info à traiter, écrire ça laisse
une trace. Mai se rencontrer pour un session de hacking, c'est cool...
T'organises les SliTazDays...?
> Thomas
- Christophe
---
SliTaz GNU/Linux Mailing list.
Web site : http://www.slitaz.org/