Fedora-Fr - Communauté francophone Fedora - Linux

Communauté francophone des utilisateurs de la distribution Linux Fedora.

  

Dernière news : Stand Fedora-Fr sur Paris Open Source Summit 2019

#1 18/02/2010 12:21:35

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

contribution: Cover thumbnailer

Bonjour,
J'aurais besoin d'avis sur mon premier rpm :

Résumé du test : cover-thumbnailer
Résumé de la description : Afficher les pochettes des albums de musique dans nautilus
URL Spec : http://tmp.fgsp.org/fedora/rpm/cover-thumbnailer.spec
URL SRPM : http://tmp.fgsp.org/fedora/rpm/cover-th … 11.src.rpm

Description :
Cover thumbnailer est un petit script développé en Python qui permet d'afficher les pochettes des albums de musique dans nautilus et de prévisualiser les images contenues dans un dossier.
Le script s'intègre dans nautilus comme n'importe quel autre thumbnailer de la GNOME thumbnail factory ; il n'est donc pas nécessaire de le lancer manuellement pour générer les miniatures.

Screenshot :
screenshot_more02_thumb.png

Site web :
http://software.flogisoft.com/cover-thumbnailer/

smile

Dernière modification par FLOZz (23/02/2010 13:25:07)

Hors ligne

#2 18/02/2010 15:56:00

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

Alors quelques points:
- Pas besoin de tes define en haut, le simple fait de mettre quelque chose en face de Name te permettra d'utiliser la macro %{name}
  - au passage il faut utiliser %global à la place de %define maintenant: https://fedoraproject.org/wiki/Packagin … _.25define
- Pour post je pense que tu devrais lire ceci: https://fedoraproject.org/wiki/Packagin … pets#GConf
- En général on mets la section %files à la fin juste au dessus du changelog mais ça c'est une broutille


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

#3 18/02/2010 16:01:59

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

Tu pourrais aussi ajouter la license aux sources (ce d'autant plus que ton fichier python ne précise pas la version alors que ton spec si)


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

#4 18/02/2010 16:09:05

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Merci pour tes suggestion pingou, je vais apporter les modifs dans mon spec :)

Pour ce qui est de la licence, elle y est :
dans le fichier Python : « either version 3 of the License » (ligne 19)
et dans les sources : le fichier COPYING

Merci encore :)

Hors ligne

#5 18/02/2010 16:11:07

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

oups désolé j'avais regardé le fichier en haut soit la plus vieille version des sources big_smile


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

#6 18/02/2010 16:49:47

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Voilà ! j'ai modifié mon .spec :)

Sortie de rpmlint sur le srpm :

[fedora@fc11vm rpm]$ rpmlint SRPMS/cover-thumbnailer-0.7-1.fc11.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Sortie de rpmlint sur le rpm :

[fedora@fc11vm rpm]$ rpmlint RPMS/noarch/cover-thumbnailer-0.7-1.fc11.noarch.rpm 
cover-thumbnailer.noarch: W: non-conffile-in-etc /etc/gconf/schemas/cover-thumbnailer.schemas
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

:)

Hors ligne

#7 18/02/2010 17:11:36

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Sortie de MOCK :

[fedora@fc11vm rpm]$ mock -r "fedora-11-i386" --arch noarch rebuild SRPMS/cover-thumbnailer-0.7-1.fc11.src.rpm 
INFO: mock.py version 1.0.2 starting...
State Changed: init plugins
State Changed: start
INFO: Start(SRPMS/cover-thumbnailer-0.7-1.fc11.src.rpm)  Config(fedora-11-i386)
State Changed: lock buildroot
State Changed: clean
State Changed: init
State Changed: lock buildroot
Mock Version: 1.0.2
INFO: Mock Version: 1.0.2
INFO: enabled root cache
State Changed: unpacking root cache
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: setup
State Changed: build
INFO: Done(SRPMS/cover-thumbnailer-0.7-1.fc11.src.rpm) Config(fedora-11-i386) 1 minutes 7 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-11-i386/result

Hors ligne

#8 18/02/2010 22:57:06

Pikachu_2014
Retraité
Lieu : Boulogne-Billancourt
Inscription : 16/06/2005
Messages : 11 334
Site Web

Re : contribution: Cover thumbnailer

Bonjour,

j'ai parcouru le .spec, un excellent travail smile. Juste quelques petites remarques cependant.

1) Le Requires sur python est superflu pour deux raisons :
- la première est que python-imaging est déjà déclaré en Requires et celui-ci dépend évidemment de python, d'où un Requires superflu.
- python est automatiquement détecté et ajouté en tant que Requires de toute façon au moment de la construction du RPM.

2) Petit détail également mais sans importance : il me semble que ta description ne dépasse pas les 80 caractères par lignes comme imposé par les Guidelines, mais j'ai l'impression que tu t'es même limité à 60 caractères ^^. Ça pourrait donner un affichage étriqué sur la console de yum ou même dans PackageKit ou yumex, je te suggère d'élargir un peu plus ton texte.

Là s'arrêtent les remarques adressées au packager ^^. Maintenant les remarques --- ou plutôt souhaits --- au développeur (puisque que tu sembles être le développeur de ce chouette programme).
L'utilisation de fakeroot était plus qu'approprié pour un script d'installation qui définit ses chemins en dur dans ton .spec. Cependant, à terme, peut-on espérer une option « --prefix » ou approchant, peu importe la forme, pour éviter le recours à fakeroot et customiser ses chemins d'installation ? As-tu d'ailleurs envisagé la possibilité d'utiliser les setuptools de Python ?


« …elle excitait si puissamment le désir, que je devins alors très incrédule sur sa vertu. »
À propos de Fœdora, dans la Peau de Chagrin (Balzac) roll

Hors ligne

#9 18/02/2010 23:11:05

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Pikachu_2014 a écrit :

Bonjour,

1) Le Requires sur python est superflu pour deux raisons :
- la première est que python-imaging est déjà déclaré en Requires et celui-ci dépend évidemment de python, d'où un Requires superflu.
- python est automatiquement détecté et ajouté en tant que Requires de toute façon au moment de la construction du RPM.

2) Petit détail également mais sans importance : il me semble que ta description ne dépasse pas les 80 caractères par lignes comme imposé par les Guidelines, mais j'ai l'impression que tu t'es même limité à 60 caractères ^^. Ça pourrait donner un affichage étriqué sur la console de yum ou même dans PackageKit ou yumex, je te suggère d'élargir un peu plus ton texte.

Ok je vais améliorer ça smile

Pikachu_2014 a écrit :

Là s'arrêtent les remarques adressées au packager ^^. Maintenant les remarques --- ou plutôt souhaits --- au développeur (puisque que tu sembles être le développeur de ce chouette programme).
L'utilisation de fakeroot était plus qu'approprié pour un script d'installation qui définit ses chemins en dur dans ton .spec. Cependant, à terme, peut-on espérer une option « --prefix » ou approchant, peu importe la forme, pour éviter le recours à fakeroot et customiser ses chemins d'installation ?

Bah quand j'ai commencé à faire le paquet debian, je me suis dis que ce serait bien pratique alors j'ai ajouté ça (option -p) seulement... j'ai oublié de permettre la copie des fichiers sans être root quand on utilise cette option (ce qui sera modifié dans la prochaine version) donc... ^^'

Pikachu_2014 a écrit :

As-tu d'ailleurs envisagé la possibilité d'utiliser les setuptools de Python ?

Bah heu non... c'est très bien à ce qu'il parait... mais faut que je me penche dessus smile

Merci pour tes remarques :D

Hors ligne

#10 18/02/2010 23:50:40

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Vala j'ai modifié le spec smile

Hors ligne

#11 18/02/2010 23:55:38

Pikachu_2014
Retraité
Lieu : Boulogne-Billancourt
Inscription : 16/06/2005
Messages : 11 334
Site Web

Re : contribution: Cover thumbnailer

Parfait smile

Une dernière pour la route : juste un petit commentaire avant « fakeroot » pour expliquer le pourquoi de la commande, histoire de faciliter la tâche à celui qui en fera la revue.
Sinon ton paquet est, je pense, prêt à être soumis aux dépôts.


« …elle excitait si puissamment le désir, que je devins alors très incrédule sur sa vertu. »
À propos de Fœdora, dans la Peau de Chagrin (Balzac) roll

Hors ligne

#12 19/02/2010 00:44:13

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Hors ligne

#13 21/02/2011 15:48:43

Roultabie
Le jour ou FC11 a changé ma vie
Inscription : 11/06/2009
Messages : 265

Re : contribution: Cover thumbnailer

Déterrage de post...

Bonjour,

Ou en est ce paquet?

Je vois qu'ici: http://fedoraproject.org/wiki/Package_m … s_wishlist

il n'apparait pas et que depuis bien des versions sont passées.

Comme je débute la création de paquets, je me propose pour reprendre celui-ci si besoin est.

Si ça vous dit.


Config: Asus U53SD, Intel Core i5-2410M, DDR3 4G, NVIDIA GeForce GT520M, Fedora 16 x86_64 Archlinux, Optimus avec Bumblebee.

Hors ligne

#14 21/02/2011 15:54:11

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

À priori si la personne qui a ouvert le bug ne répond pas rapidement, Tiibs va fermé le bug.

Tu auras alors la possibilité de soumettre le tiens

N'hésite pas à nous faire relire tes specs ici, ça permet un premier regard sur ton travail (tu peux aussi donner le lien vers la revue bugzilla)


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

#15 21/02/2011 16:01:11

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

Actuellement j'ai pas trop le temps et mes mails s'empilent et j'ai pas eu le temps de répondre sur le tracker... Donc si tu veux prendre en charge le paquet rpm moi je suis pas contre ^^'

Si t'as besoin d'aide n'hésite pas par contre :)

Hors ligne

#16 21/02/2011 16:06:00

Roultabie
Le jour ou FC11 a changé ma vie
Inscription : 11/06/2009
Messages : 265

Re : contribution: Cover thumbnailer

Merci pour ces réponses rapides.

Je vais donc travailler su ce paquet. (principalement le lundi et le dimanche) donc pour les résultat, ce ne sera pas au jour le jour.

Je vous tiens au courant de la suite des évennements (je travaille dessus aujourd'hui).


Config: Asus U53SD, Intel Core i5-2410M, DDR3 4G, NVIDIA GeForce GT520M, Fedora 16 x86_64 Archlinux, Optimus avec Bumblebee.

Hors ligne

#17 21/02/2011 18:05:58

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

Ferme alors toi même la revue sur le bugzilla, ça permettra à FLOZz de soumettre la sienne


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

#18 21/02/2011 18:10:40

Roultabie
Le jour ou FC11 a changé ma vie
Inscription : 11/06/2009
Messages : 265

Re : contribution: Cover thumbnailer

Ce n'est pas à FLOZz de fermer la sienne pour que je crée la mienne?

Dernière modification par Roultabie (21/02/2011 18:35:53)


Config: Asus U53SD, Intel Core i5-2410M, DDR3 4G, NVIDIA GeForce GT520M, Fedora 16 x86_64 Archlinux, Optimus avec Bumblebee.

Hors ligne

#19 21/02/2011 18:21:51

FLOZz
Membre
Lieu : Lyon
Inscription : 18/02/2010
Messages : 13
Site Web

Re : contribution: Cover thumbnailer

@pingou:
C'est l'inverse... :)

Bug fermé !

Hors ligne

#20 21/02/2011 18:36:32

Roultabie
Le jour ou FC11 a changé ma vie
Inscription : 11/06/2009
Messages : 265

Re : contribution: Cover thumbnailer

Je vous présenterai mon paquet demain et ouvrirai ensuite un bug.

Bonne soirée.


Config: Asus U53SD, Intel Core i5-2410M, DDR3 4G, NVIDIA GeForce GT520M, Fedora 16 x86_64 Archlinux, Optimus avec Bumblebee.

Hors ligne

#21 22/02/2011 09:37:11

Roultabie
Le jour ou FC11 a changé ma vie
Inscription : 11/06/2009
Messages : 265

Re : contribution: Cover thumbnailer

Bonjour a tous.

Je vous présente mes travaux du jour (déjà bien aidé par l'excellent travail de FLOZz)

donc le fichier spec: cover-thumbnailer.spec

et le src.rpm: cover-thumbnailer-0.8.2-1.fc14.src.rpm

Voici les modifications effectuées:

- Retrait du mv des shemas dans le spec car plus besion dans cette version de cover-thumbnailer.

- Création d'un patch de l'installer car celui-ci copie la doc manuellement dans le répertoire /usr/share/doc/cover-thumbnailer/ en plus du spec qui le copie dans /usr/share/doc/cover-thumbnailer-0.8.2/

ce qui pose problème lors de la création de RPM.

Ce patch supprime simplement les lignes suivantes de l'installeur:

#/usr/share/doc/cover-thumbnailer
mkdir -pv "$1"/usr/share/doc/cover-thumbnailer 1>> $LOGFILE 2>> $LOGFILE || error=1
cp -v ./README "$1"/usr/share/doc/cover-thumbnailer 1>> $LOGFILE 2>> $LOGFILE || error=1

En espérant ne pas avoir oublié des choses.

Dernière modification par Roultabie (22/02/2011 09:55:46)


Config: Asus U53SD, Intel Core i5-2410M, DDR3 4G, NVIDIA GeForce GT520M, Fedora 16 x86_64 Archlinux, Optimus avec Bumblebee.

Hors ligne

#22 22/02/2011 09:45:40

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

À la limite plutôt que de patcher tu peux juste virer le dossier à la fin de l'%install

> mkdir -p $RPM_BUILD_ROOT%{_sysconfdir}/gconf/schemas/
Je ne pense pas qu'elle soit nécessaire cette ligne là

ton %changelog est pas correct, il doit refléter des modifications que tu as apporté au spec (ce que tu nous dit ici quoi)


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

#23 22/02/2011 10:13:35

Roultabie
Le jour ou FC11 a changé ma vie
Inscription : 11/06/2009
Messages : 265

Re : contribution: Cover thumbnailer

Spec mis à jour.

Par contre, pour le patch.

La documentation précise de ne pas ajouter de ligne dans le spec mais bien de créer des patchs afin que ce soit plus clair pour les mainteneurs.

Ou est la limite pour créer des patchs?

(Et désolé d'avance pour mon anglais dans le changelog, je le travaille ;) )


Config: Asus U53SD, Intel Core i5-2410M, DDR3 4G, NVIDIA GeForce GT520M, Fedora 16 x86_64 Archlinux, Optimus avec Bumblebee.

Hors ligne

#24 22/02/2011 10:16:37

madko
Contributeur Fedora et Linuxé depuis 1994
Modérateur
Lieu : Noisy the Great (9³)
Inscription : 22/12/2006
Messages : 7 368
Site Web

Re : contribution: Cover thumbnailer

Dans la section files:

%{_datadir}/applications/*
%{_datadir}/cover-thumbnailer/*

Je trouve ça moyen
il y a pas un rep cover-thumbnailer dans %{_datadir}/applications?
Bref je metterais plutôt:

%{_datadir}/applications/cover-thumbnailer
%{_datadir}/cover-thumbnailer

Hors ligne

#25 22/02/2011 10:22:01

pingou
Fedora Addict
Lieu : Toulouse
Inscription : 30/03/2006
Messages : 3 843
Site Web

Re : contribution: Cover thumbnailer

Pour le patch c'est une question de taille de changement.

Si c'est juste changer une ligne je passe par sed, si c'est retirer des fichiers je les vire à la fin de la section %install, si c'est changé un gros bout de l'installeur pour qu'il fasse son boulot correctement, je patch.
En gros tout est faisable et le choix est laissé au mainteneur (ta solution ne bloquera pas la revue).

Ensuite quelque soit la solution choisie il faut en effet la documenter (les commentaires sont autorisé dans le spec) pour se rappeler de ce qui est fait.


On a pas inventé l'électricité en cherchant à améliorer la bougie...
-- Si c'est pas sur le bugzilla, c'est pas un bug ! --

Hors ligne

Pied de page des forums