Fedora-Fr

Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

Demande de Review: Polari - An IRC client for GNOME

Nobrakal

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

MarbolanGos

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)

remi

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}

remi

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.

remi

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.

remi

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}

remi

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.

remi

Release:        1%{?dist}

Il s'agit visiblement d'une pré-release (snapshot), donc il faut que cela soit visible: 0.1.%{gitrevision}%{?dist}

Pikachu_2014

Bonjour,

quelques remarques :

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 :
- https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages (pour versionner ton paquet) ;
- http://fedoraproject.org/wiki/Packaging:SourceURL#Using_Revision_Control (pour décrire la manière dont ton archive source a été construite à partir d'un identifiant de commit).

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}

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.

Nobrakal

Salut,

Merci de vos réponses 🙂

@remi, j'ai corrigé tous les problèmes, mais je doute pour encore pour le Release...

@Pikachu_2014: Dommage pour l'impossibilité de packager... Mais bon, en attendant, on a toujours les dépendances de gnome 3.10, on peut encore s'amuser 🙂

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

Alexandre

llaumgui

Moyen d'avoir un RPM pour les leecher 😉

Nobrakal

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