Tips acerca del proceso de revisión
- Para los empaquetadores, reportes útiles
- Incluya la salida rpmlint en su solicitud de revisión Bugzilla
- Los nombres cortos de licencias deben usarse consistentemente
- Cuando modifique el spec, cambie Release y changelog
- Deje que debug_info.sh genere el paquete debuginfo
- No reclame propiedad sobre archivos provistos por otros paquetes
- Uso del macro name y version en source url
Para los empaquetadores, reportes útiles¶
- Nuevas solicitudes de revisión : todas las peticiones no-merge sin revisor asignado, ordenados por fecha. Los colores en verde requieren un patrocinante.
- Solicitudes que necesitan patrocinante : todas las peticiones donde se requiere patrocinante ordenadas por quien reporta.
- Solicitudes de revisión merge : todas las solicitudes de revisión merge.
- Hidden tickets : peticiones que han sido ocultadas por alguna razón. Estas peticiones ya sea dependen de otras solicitudes que aún no han sido cerradas, o que no son revisables por algún motivo.
Incluya la salida rpmlint en su solicitud de revisión Bugzilla¶
$ rpmlint -i fwsnort-1.0.6-1.fc12.src.rpm fwsnort.spec 1 packages and 1 specfiles checked; 0 errors, 0 warnings.
Los nombres cortos de licencias deben usarse consistentemente¶
man fwsnort >> se refiere a GPL
spec >> se refiere a GPLv2 : esta referencia es la correcta ya que GPL sin version no es un
: nombre corto de licencia aceptado por Fedora, pero el uso de los
: nombres debe ser consistentemente entre el man y el spec
Cuando modifique el spec, cambie Release y changelog¶
Los revisores podrán hacerle observaciones a su spec/srpm, cuando haga las modificaciones para corregir, no olvide hacer los cambios correspondiente, puede usar rpmdev-bumpspec para ello.
$ rpmdev-bumpspec --comment="- documentation included, %doc LICENSE VERSION README CREDITS TODO SNORT-RULES-VERSION" --userstring="Guillermo Gómez <ggomez@neotechgw.com>" SPECS/fwsnort.spec
Resultado:
...
Version: 1.0.6
Release: 2%{?dist} <<<< se incrementó
License: GPLv2
...
%changelog
* Wed Feb 03 2010 Guillermo Gómez <ggomez@neotechgw.com> - 1.0.6-2 << Nuevos cambios
- documentation included, %doc LICENSE VERSION README CREDITS TODO
SNORT-RULES-VERSION
* Sat Jan 2 2010 Guillermo Gómez <ggomez@neotechgw.com> - 1.0.6-1
- First Fedora spec compliant version, several modifications
- No deps included
- Free snort rules included
Deje que debug_info.sh genere el paquete debuginfo¶
Ej: clamsmtp-debuginfo-1.10-1.fc12.i686.rpm.
%{__strip} %{buildroot}%{_sbindir}/clamsmtpd << esto provoca un debuginfo vacío.
No reclame propiedad sobre archivos provistos por otros paquetes¶
Ej. En el siguiente ejemplo, %{_sysconfdir}/logorotate.d se está reclamando la propiedad de dicho directorio cuando en realidad dicho directorio es provisto por el paquete logorotate. No reclame la propiedad de esta forma y a cambio incluya Requires logrotate para que se suministre el directorio como se espera.
%files
...
%dir %{_sysconfdir}/logrotate.d
Aún asi, en la instalación puede que necesite crear el directorio para la construcción del paquete:
%install ... mkdir -p $RPM_BUILD_ROOT/etc/logrotate.d install -p -m 644 logrotate.fwsnort $RPM_BUILD_ROOT/etc/logrotate.d/fwsnort
Uso del macro name y version en source url¶
... Source0: http://www.net.princeton.edu/software/dhcp_probe/dhcp_probe-1.3.0.tar.gz ...
...
Source0: http://www.net.princeton.edu/software/dhcp_probe/%{name}-%{version}.tar.gz
...
use version macro in source url