Génération de fichiers STR - Page 2

./25: j'afficherais un warning plutôt qu'une erreur, et on peut ajouter un switch '-f' pour forcer la génération de chaînes contenant des 0x00.

Ça sonne raisonnable, tu vas nous faire ça pour la prochaine version?

Il vaudrait mieux que je fasse ça *avant* la prochaine version, pour que Martial puisse l'utiliser rapidement

Ce qui bloque la prochaine version, ce sont les trois entrées restantes dans la todo list qui résume le licensing audit que tu avais fait:
* "Please license under the WxWindows Library Licence: src/pctools/unpack.c, src/calctools/ttunpack.h". Vu que je l'ai marqué "partiellement fait", il faut que je vérifie où ça en est (peut-être que c'est fait et que je n'ai pas mis la todo list à jour);
* "Please remove (copyright violation): examples/tribute". C'est un des rares exemples de TTArchive, donc je n'ai pas envie de le supprimer purement et simplement... mais il faudrait trouver un jeu cohérent d'images (licence CC ?), qui se traduisent bien en B/W ou 4 grayscales, pour remplacer;
* "Needs contacting original authors: src/calctools/ttbatt.c, src/calctools/tthedit.c, src/calctools/ttrow.c". C'est tout juste commencé (j'ai saisi l'occasion où Loïc Minier, mainteneur du package libmpeg2 dans Debian, a posté sur libmpeg2-devel une review de quelques-uns de mes patches à libmpeg2), mais toujours pas fini, parce que je pense qu'il y a beaucoup de choses plus importantes que de s'occuper de problèmes de licence.

http://www.mirari.fr/cGXC est un patch à appliquer avec `patch -p1` dans src/pctools, pour:
* corriger le crash reporté en ./15 (double fclose);
* ajouter un warning (affiché une seule fois) seulement si on n'a pas passé au programme l'option -f;
* ajouter l'info de licence au fichier: même licence que les autre tt*. ttbin2str avait été poussé hors de mon arborescence des fichiers de TIGCC Tools Suite pendant l'été 2006, après le snapshot 20060628 (fait par roms & Kevin). Ce qui rend explicite la permission d'utiliser et d'embarquer ttbin2str comme c'est prévu dans le source.

Merci à vous deux de faire avancer les choses au niveau de la TT Suite

Ton patch ne corrige pas le plantage avec _FORTIFY_SOURCE.

    if (calctype == CALC_TI89) strcpy(header.signature,SIGNATURE_TI89);
    else                       strcpy(header.signature,SIGNATURE_TI92P);

déborde d'un octet. (Oui, ça déborde dans fill1 que tu écris juste après, mais _FORTIFY_SOURCE détecte quand-même le débordement et arrête le programme avec une erreur.)

Le code dans ttbin2oth (bin2oth.c):

    if (calctype == CALC_TI89) strncpy(psh->signature,SIGNATURE_TI89,8);
    else                       strncpy(psh->signature,SIGNATURE_TI92P,8);

est correct.

Lionel Debroux (./33) :
* "Please license under the WxWindows Library Licence: src/pctools/unpack.c, src/calctools/ttunpack.h". Vu que je l'ai marqué "partiellement fait", il faut que je vérifie où ça en est (peut-être que c'est fait et que je n'ai pas mis la todo list à jour);

Vu que, à ma connaissance, les auteurs en question ont déjà donné leur permission, il suffit de mettre à jour les headers.

* "Please remove (copyright violation): examples/tribute". C'est un des rares exemples de TTArchive, donc je n'ai pas envie de le supprimer purement et simplement... mais il faudrait trouver un jeu cohérent d'images (licence CC ?), qui se traduisent bien en B/W ou 4 grayscales, pour remplacer;* "Needs contacting original authors: src/calctools/ttbatt.c, src/calctools/tthedit.c, src/calctools/ttrow.c". C'est tout juste commencé (j'ai saisi l'occasion où Loïc Minier, mainteneur du package libmpeg2 dans Debian, a posté sur libmpeg2-devel une review de quelques-uns de mes patches à libmpeg2), mais toujours pas fini, parce que je pense qu'il y a beaucoup de choses plus importantes que de s'occuper de problèmes de licence.

Ben, sors un tarball avec ces 2 problèmes encore marqués "pending", comme ça je pourrais packager une version sans les fichiers en question dans le dépôt calcforge-free - de toute façon ce n'est pas la peine de packager les exemples et les outils on-calc pour Fedora, ils ne dépendent pas de l'OS qu'on utilise sur l'ordinateur.

./36: corrigé en local.

plom plom plom