search

JamesRezo / spip_svn_loader

Installer et mettre à jour SPIP avec SVN
MIT License
6 stars 2 forks source link

Mettre en parametre la fonction qui calcule le md5 du repertoire courant #9

Closed tech-nova closed 8 years ago

tech-nova commented 8 years ago

La fonction "md5" n'est pas installée par défaut sous Debian. J'ai donc utilisé md5sum qui calcule le md5 d'une fonction ou du flux en entrée Il faudrait peut-être multiplier ce type de paramétrage sur les fonctions de base (curl, en particulier)

JamesRezo commented 8 years ago

OK pour md5. Mais md5sum n'existe pas sur Mac.

Pour assurer la compat mac/linux, j'ai fait ça https://github.com/JamesRezo/spip_svn_loader/blob/master/spip_svn_loader#L170

Il faudrait faire le même test ici https://github.com/JamesRezo/spip_svn_loader/blob/master/spip_svn_loader#L286

alternativement, on pourrait utiliser shasum ? : 

echo `pwd`"/spip" | shasum | cut -d" " -f1

Enfin, je ne comprends pas ce que tu veux dire à propos de curl ?

tech-nova commented 8 years ago

Pour curl, j'avais eu une erreur indiquant que la fonction n'était pas trouvée. un apt-get install a corrigé le problème. Il faut dire que j'utilise plutot wget

tech-nova commented 8 years ago

Et pour être plus secure encore que shasum, on peut aussi utiliser sha512sum, qui fait parti du même dépôt coreutils

echo `pwd`"/spip" | sha512sum | cut -d" " -f1
JamesRezo commented 8 years ago

On va uniquement traité de cette histoire de md5 et sha dans cette PR, si tu veux bien.

La commande sha512sum n'existe pas sur Mac. Et ce n'est pas une question de sécurité, c'est histoire d'avoir une chaine pseudo aléatoire.

tech-nova commented 8 years ago

Pas de problème, ça me convient.

2016-03-17 20:37 GMT+01:00 James notifications@github.com:

On va uniquement traité de cette histoire de md5 et sha dans cette PR, si tu veux bien.

La commande sha512sum n'existe pas sur Mac. Et ce n'est pas une question de sécurité, c'est histoire d'avoir une chaine pseudo aléatoire.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/JamesRezo/spip_svn_loader/pull/9#issuecomment-198049172

JamesRezo commented 8 years ago

fermé par 0f196832185e2cab8ec412388f4b46933da1b85d et 0f196832185e2cab8ec412388f4b46933da1b85d