https://bugzilla.redhat.com/show_bug.cgi?id=1294523 Simone Caronni <negativo17@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |negativo17@xxxxxxxxx Blocks|177841 (FE-NEEDSPONSOR) | Assignee|nobody@xxxxxxxxxxxxxxxxx |negativo17@xxxxxxxxx --- Comment #1 from Simone Caronni <negativo17@xxxxxxxxx> --- No longer blocking FE-NEEDSPONSOR. I will sponsor Vitaly. Please remove this lines, they are not needed: > # generating empty configure script > echo '#!/bin/bash' > configure > chmod +x configure > %configure It is not mandatory to have a configure script. You can use the following to pass correct CFLAGS when building: > export CFLAGS="%{optflags}" > %make_build A personal request; can you please add these two lines to the "purple-skypeweb" pacakge? > Provides: skype4pidgin = %{version}-%{release} > Obsoletes: skype4pidgin < %{version}-%{release} I've been packaging all skype4pidgin plugins for years (including libskypeweb) until now, where skypeweb is mature enough. I would like to make the transition transparent to the official Fedora packages for my users: http://negativo17.org/skype-and-skype-pidgin-plugin/ This can be removed in a few months. Please provide again the "Spec URL" and "SRPM URL" lines with the latest changes, fedora-review needs to have them when parsing packages/spec files. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx