Bonsoir à tous,

C'est mon second paquet, mais avant d'ouvrir une véritable review request interminable, je viens demander votre avis.
Je viens de m'atteler à la tache de faire un rpm pour Polari, le nouveau client IRC de Gnome.
Si quelqu'un trouve 5mins l'histoire de me dire si mon affaire est viable 🙂

Le paquet se construit correctement dans mock.
L'output du rpmlint:
polari.x86_64: W: devel-file-in-non-devel-package /usr/lib64/polari/libpolari-1.0.so
polari.x86_64: W: no-manual-page-for-binary polari
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

Le spec: http://nobrakal.fedorapeople.org/polari.spec
Le srpm: http://nobrakal.fedorapeople.org/polari-0.0.1-1.fc20.src.rpm

Merci d'avance,

Alexandre
Je ne suis pas expert mais comme l'indique l'erruer, le fichier /usr/lib64/polari/libpolari-1.0.so devrait être dans le paquet -devel. Par contre est-il nécessaire au binaire généré ?
ldd $(which polari)
if echo "getconf LONG_BIT" == 64;then 
./autogen.sh --exec-prefix=/usr/ --prefix=/usr/ --libdir=/usr/lib64/

elif echo "getconf LONG_BIT" == 32;then
./autogen.sh --exec-prefix=/usr/ --prefix=/usr/ 
fi
Pourquoi pas simplement
./autogen.sh --exec-prefix=%[_prefix} --prefix=%[_prefix} --libdir=%[_libdir}
En fait ce qui me gene, c'est l'autogen qui visiblement lance le "configure".

La macro %configure permet de passer les bon paramètres (dossiers).

Donc voir si l'autogen est vraiment obligatoire, ou dispose d'une option pour ne pas faire le configure.
Le bibliothèque est visiblement privé (pas d'entete). Donc pas besoin de -libs ou -devel.

Par contre il faut la filtrer pour qu'elle n'apparaisse pas dans les dép.
rm -rf $RPM_BUILD_ROOT
make install DESTDIR=%{buildroot} INSTALL="/usr/bin/install -p"
1/ le rm -rf ... est inutile.

2/ on ne mélange pas $RPM_BUILD_ROOT et %{buildroot}
Source0:        https://git.gnome.org/browse/polari/snapshot/polari-master.tar.gz
Il faut une archive "reproductible", c'est à dire basée sur un tag ou une référence de commit.
Release:        1%{?dist}
Il s'agit visiblement d'une pré-release (snapshot), donc il faut que cela soit visible: 0.1.%{gitrevision}%{?dist}
Bonjour,

quelques remarques :
  1. Tu packages une version SNAPSHOT de polari (il n'y a pas encore eu, dans mon souvenir, de release de polari). Or il n'y a aucun moyen d'identifier le commit GIT correspondant à ton archive source (le lien vers Source0 pointe toujours sur le dernier commit du tronc de polari). Dans un tel cas de figure, je te renvoie aux guidelines suivantes :
  2. Histoire de contourner le lancement automatique de configure lorsqu'autogen.sh est invoqué, je te suggère de modifier ta section %build ainsi :
    %build
    # Ne lancer autogen.sh que s'il n'y a pas de configure
    # La variable NOCONFIGURE permet de désactiver le lancement de configure par autogen.sh si elle n'est pas vide
    [ -f configure ] || NOCONFIGURE=1 ./autogen.sh
    %configure
    make %{?_smp_mflags}
    
  3. Il est peut-être prématuré de packager polari, même pour Rawhide, et surtout en l'absence de release. Si tu pensais le proposer pour Fedora 20, tu ne pourras guère le maintenir longtemps, son développement devrait très vite cibler GNOME 3.12, avec des dépendances qui ne pourront plus être satisfaites sous GNOME 3.10.
Moyen d'avoir un RPM pour les leecher 😉
7 jours plus tard
Bon, en attendant, je vais essayer de garder une version à jour (environ tous les 2/3 soirs) de polari sur mon fedorapeople.

Pour rappel: http://nobrakal.fedorapeople.org/

Si ça intéresse quelqu'un, (hein, llaumgui...) 🙂

Alexandre