Fedora-Fr - Communauté francophone Fedora - Linux

Communauté francophone des utilisateurs de la distribution Linux Fedora.

  

Dernière news : Venez tester Fedora 30 Beta !

#1 18/02/2007 16:09:04

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

pypar2

Bonjour à tous,

Voilà le passage un peu intensif à la fac est fini.
Je reviens avec mon petit rpm.

Petit rappel :
PyPar2 (http://pypar2.silent-blade.org/) est une interface graphique (en python) à par2cmdline.
Il permet de créer des fichiers de redondances et ainsi que de vérifier l'intégrité de fichiers télécharger.

Je voudrais juste que les experts fassent une petite revue de mon paquet.

Vous trouverez le spec, le rpm et le srpm ici : http://carron.maxime.free.fr/fedora/rpms/

rpmlint me retourne une erreur et un warning, mais je ne pense pas que je puisse les enlever.

[builder@localhost SPECS]$ rpmlint ../RPMS/i386/pypar2-1.2-1.fc6.maxca.i386.rpm 
E: pypar2 no-binary
W: pypar2 symlink-should-be-relative /usr/bin/pypar2 /usr/share/pypar2/src/main.py

L'objectif c'est de le proposer sur extras.

TODO :
Il y a un truc qui cloche, c'est l'emplacement du raccourci dans le menu fedora.
Il existe un utilitaire complementaire pour integrer pypar2 dans nautilus (nautilus-action). Je ferai le rpm tres bientot.

Voilà, j'attends vos remarques.
Puis si qq'un veut etre mon sponsor...

A+


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#2 18/02/2007 17:10:23

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

Re : pypar2

Salut,

- je suppose que ce programme ne consiste «qu'en» de scripts Python : dans ce cas, remplace ta clause «BuildArch:      i386» par «BuildArch:      noarch» ;
-pour ton problème de lien, peut-être que ceci :
ln -s ../..%{_datadir}/pypar2/src/main.py %{buildroot}%{_bindir}/pypar2
à la place de cela :
ln -s %{_datadir}/pypar2/src/main.py %{buildroot}%{_bindir}/pypar2
passera outre le warning.

P.S. : j'ai ces petits warnings quand j'appelle rpmlint sur ton RPM source :
- W: pypar2 no-%build-section
- W: pypar2 mixed-use-of-spaces-and-tabs (spaces: line 5, tab: line 19)
Des détails, mais à corriger wink

Dernière modification par Pikachu_2014 (18/02/2007 17:12:22)


« …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

#3 18/02/2007 18:28:05

SmootherFrOgZ
Rédacteur Wiki
Rédacteur Wiki
Lieu : fedoraprojet.org
Inscription : 12/07/2006
Messages : 348
Site Web

Re : pypar2

Je te ferais la review wink

Hors ligne

#4 18/02/2007 18:31:18

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Bon, j'ai plus que ces deux warnings :

[builder@localhost result]$ rpmlint pypar2-1.2-1.fc6.noarch.rpm 
W: pypar2 symlink-should-be-relative /usr/bin/pypar2 /usr/share/pypar2/src/main.py
[builder@localhost result]$ rpmlint pypar2-1.2-1.fc6.src.rpm 
W: pypar2 no-%build-section

Le second sur le src, c'est normal, il n'y a pas de make à faire, donc...
Je sais pas si je laisse une section build vide.

Le premier est selon moi normal aussi.
Il faut que mon lien symbolique soit placer à cette endroit.
Sinon laisse faire le Makefile, le lien (qui  permet de lancer le programme se trouve dans %{buildroot}/%{datadir} et donc n'est pas accessible une fois installé.

Voilà...
j'ai mis à jour les nouveaux spec, rpm et srpm sur http://carron.maxime.free.fr/fedora/rpms/


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#5 18/02/2007 20:04:47

SmootherFrOgZ
Rédacteur Wiki
Rédacteur Wiki
Lieu : fedoraprojet.org
Inscription : 12/07/2006
Messages : 348
Site Web

Re : pypar2

alors,

* Remplace pypare2 par sa macros %{name}.
* Rajoute le timestamps INSTALL="install -p -D" à ton make install.
* > rm %{buildroot}%{_bindir}/pypar2
  > ln -s %{_datadir}/pypar2/src/main.py %{buildroot}%{_bindir}/pypar2 --> il faut commenter les modifications que tu apportes pour une meilleur review.
  ce lien est déjà généré par le Makefile, donc pourquoi tu l'écrase ?
* "--add-category X-Fedora \" ne doit pas être appliqué.
* Ton paquet ne contient pas de licence. c'est pas bon ça ! vois avec le dev de l'app pour c'la.
* Tu n'es pas obligé d'entrer tous les fichiers dans %files, le chemin suffit.
  %{_datadir}/%{name}/pix
  %{_datadir}/%{name}/res
  %{_datadir}/%{name}/src   --> ou juste %{_datadir}/%{name}  suffit.
* Placer le disttag .fc6 n'est pas nécessaire, rpmlint l'ignore lors du check. Par contre si tu le laisse rpmlint pourrai se plaindre si t'a buildé pour une autre version (fc-devel  par exemple).
* Rajoute un point pour "initiale rpm release.

Voilà, j'attends vos remarques.
Puis si qq'un veut etre mon sponsor...

Il faut faire ces preuves pour c'la wink

Hors ligne

#6 18/02/2007 20:17:17

remi
Crazy PHP packages monkey... !
Rédacteur Wiki
Lieu : Champagne...
Inscription : 16/10/2004
Messages : 5 563
Site Web

Re : pypar2

Oui :

%{_datadir}/%{name}

Sinon aucun paquet ne sera le propriétaire de ce répertoire.

A+

Hors ligne

#7 19/02/2007 09:41:41

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Me revoilà!!

Voici donc le specfile : http://carron.maxime.free.fr/fedora/rpm … .spec.html
Les autres fichiers sont toujours dispo sur http://carron.maxime.free.fr/fedora/rpms/

Deux petites questions supplémentaires :
- pourquoi lorsque je fais un yum remove pypar2, par2cmdline n'est pas supprimé aussi?
- SmootherFrog m'a dit qu'il n'y avait pas de licence dans mon paquet. Indiquer la licence dans le specfile ne suffit pas? Il faut mettre le contrat de licence dans le paquet?

Merci


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#8 19/02/2007 19:52:10

drpixel
Membre FedoraProject
Membre FedoraProject
Lieu : Maromme (76)
Inscription : 28/07/2004
Messages : 4 324
Site Web

Re : pypar2

- SmootherFrog m'a dit qu'il n'y avait pas de licence dans mon paquet. Indiquer la licence dans le specfile ne suffit pas? Il faut mettre le contrat de licence dans le paquet?

oui dans %doc

Hors ligne

#9 19/02/2007 20:26:45

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

ok.
Le probleme, c'est que le texte n'est pas dans le source.
J'ai demandé à l'auteur c'est bien de la GPLv2.

Mais comment ajouter un fichier exterrieur?


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#10 19/02/2007 20:59:35

SmootherFrOgZ
Rédacteur Wiki
Rédacteur Wiki
Lieu : fedoraprojet.org
Inscription : 12/07/2006
Messages : 348
Site Web

Re : pypar2

Ajoute le comme source.

Hors ligne

#11 19/02/2007 21:02:48

drpixel
Membre FedoraProject
Membre FedoraProject
Lieu : Maromme (76)
Inscription : 28/07/2004
Messages : 4 324
Site Web

Re : pypar2

Source1 : monfichier

%doc %{SOURCE1}

monfichier doit se trouver dans ton dossier de sources.

Hors ligne

#12 19/02/2007 23:22:43

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Ouais Sereinity m'a dit ca aussi.

Je voyais un truc plus compliqué avec un wget dans une section %pre, puis unn ajout dans la section %file.
C'etait plus drole aussi.

Bon, je fais ça, puis ca devrait etre bon.

Merci à tous.


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#13 20/02/2007 08:40:55

drpixel
Membre FedoraProject
Membre FedoraProject
Lieu : Maromme (76)
Inscription : 28/07/2004
Messages : 4 324
Site Web

Re : pypar2

ouais Pingoomax ! lol
avec un
BuildRequires: Internet-Connection
BuildRequires: wget
^^

Hors ligne

#14 20/02/2007 09:05:59

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Ho putain, j'avais pas pensé à celle là!
J'ai encore du chemin à faire. smile

En tout cas ca fait des blagues à balancer sur la ml d'extras big_smile


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#15 20/02/2007 09:21:51

slobberbone
Ancien Ambassadeur Fedora
Rédacteur Wiki
Lieu : Caen
Inscription : 20/02/2006
Messages : 1 265
Site Web

Re : pypar2

j'adore lire ce genre de post, je sens à quel point j'ai encore à apprendre pcq là je pige rien de ce que vous dites !! ^^ MDR

Faudrait que je m'y mette ... qd j'aurrai du temps ! hammer

Hors ligne

#16 20/02/2007 14:07:00

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Gné...
Ca marche pô :'(

J'ai donc ajouté :
Source1:    http://www.gnu.org/licenses/gpl.txt

puis :
%doc doc/whatsnew.txt %{SOURCE1}
qui me rend l'erreur :

Erreur de construction de RPM:
    Ne peut mélanger un %doc spécial avec d'autre formes: /home/builder/rpmbuild/SOURCES/gpl.txt

J'ai donc mis ca en deux lignes :
%doc doc/whatsnew.txt
%doc %{SOURCE1}

Erreur de construction de RPM:
    Fichier non trouvé: /var/tmp/pypar2-1.2-1.fc6.maxca-root-builder/home/builder/rpmbuild/SOURCES/gpl.txt

Any idea?


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#17 20/02/2007 15:34:26

drpixel
Membre FedoraProject
Membre FedoraProject
Lieu : Maromme (76)
Inscription : 28/07/2004
Messages : 4 324
Site Web

Re : pypar2

mets ton fichier gpl.txt dans le dossier ~/rpmbuild/SOURCES
Source1: gpl.txt

Hors ligne

#18 20/02/2007 16:13:14

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

ouais, ca, ca va marcher, mais normalement, je ne suis pas censé mettre l'adresse du fichier que j'ajoute?
Sinon comment les gars qui reconstruisent le paquet vont faire?


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#19 20/02/2007 16:50:03

drpixel
Membre FedoraProject
Membre FedoraProject
Lieu : Maromme (76)
Inscription : 28/07/2004
Messages : 4 324
Site Web

Re : pypar2

Il sera dans le SRPM.

Hors ligne

#20 20/02/2007 17:01:54

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Ha?

Cooooool!!
smile


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#21 20/02/2007 17:31:36

drpixel
Membre FedoraProject
Membre FedoraProject
Lieu : Maromme (76)
Inscription : 28/07/2004
Messages : 4 324
Site Web

Re : pypar2

Love

Nous aussi on t'aime Pingoomax ^^

Hors ligne

#22 21/02/2007 14:07:04

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Ben en fait, mais là ca en marche pas :

Erreur de construction de RPM:
    Fichier non trouvé: /var/tmp/pypar2-1.2-1.fc6.maxca-root-builder/home/builder/rpmbuild/SOURCES/COPYING

Mon spec ressemble à :
...
Source0:    http://pypar2.silent-blade.org/uploads/ … on}.tar.gz
Source1:    COPYING
...
%files -f %{name}.lang
%defattr(-,root,root,-)
%doc doc/whatsnew.txt
%doc %{SOURCE1}
...


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#23 21/02/2007 14:15:16

remi
Crazy PHP packages monkey... !
Rédacteur Wiki
Lieu : Champagne...
Inscription : 16/10/2004
Messages : 5 563
Site Web

Re : pypar2

Effectivement, l'URL est nécessaire uniquement pour la Source0.

%doc %{SOURCE1}

Je ne suis pas sur que cette ligne soit très conforme

J'ai plutôt l'habitude de quelque chose du genre

Source1:    pypar-COPYING
...
%prep
...
cp %{SOURCE1} COPYING
...
%files
%defattr(-,root,root,-)
%doc doc/whatsnew.txt COPYING

De cette manière tous les fichiers Sources sont différenciés lorsque tu as beaucoup de RPM

Dans la phase de préparation, tu le copie (dans le répertoire de construction) sous le nom que tu veux.

Il y a actuellement des bugs pour les constructions lorsqu'il n'y a pas de build. Donc il est préférable de conserver une section vide et commentée

%build
# Nothing to do

A+

Hors ligne

#24 21/02/2007 15:02:05

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Ok.

Merci remi.

Je fais ca.


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

#25 21/02/2007 16:45:30

pingoomax
Retraité
Lieu : Paris
Inscription : 24/09/2004
Messages : 1 833
Site Web

Re : pypar2

Voilà, cette fois, il me semble que tout est bon :

Mon specfile (en version lisible en ligne): http://carron.maxime.free.fr/fedora/rpm … .spec.html
Et le reste des fichiers utiles : http://carron.maxime.free.fr/fedora/rpms/

Si vous voulez bien jetter encore un coup d'oeil, je veux bien.
Apres, je le proposerai sur extras.

Une derniere question, dans le changelog, je n'ai pas besoin d'expliquer pourquoi le patch et l'ajout du texte de la GPL?


bouton-april-80x15.png Je suis membre de l'APRIL, et vous?
Vous aussi adhérez.

Hors ligne

Pied de page des forums