https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Bug ID: 1294852 Summary: Review Request: kf5-libksane - SANE Library interface for KDE Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: rdieter@math.unl.edu QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://rdieter.fedorapeople.org/rpms/kf5/kf5-libksane.spec SRPM URL: https://rdieter.fedorapeople.org/rpms/kf5/kf5-libksane-15.12.0-1.fc23.src.rp... Description: SANE Library interface for KDE Fedora Account System Username: rdieter
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1294838 (digikam-5.0), | |656997 (kde-reviews) Alias| |kf5-libksane
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=656997 [Bug 656997] kde-related package review tracker https://bugzilla.redhat.com/show_bug.cgi?id=1294838 [Bug 1294838] digikam-5.0 tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #1 from Upstream Release Monitoring upstream-release-monitoring@fedoraproject.org --- rdieter's scratch build of kf5-libksane-15.12.0-1.fc23.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12349465
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Pier Luigi Fiorini pierluigi.fiorini@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |pierluigi.fiorini@gmail.com Assignee|nobody@fedoraproject.org |pierluigi.fiorini@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Pier Luigi Fiorini pierluigi.fiorini@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(pierluigi.fiorini | |@gmail.com)
--- Comment #2 from Rex Dieter rdieter@math.unl.edu --- ping, do you plan on continuing reviewing this package?
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Pier Luigi Fiorini pierluigi.fiorini@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(pierluigi.fiorini | |@gmail.com) |
--- Comment #3 from Pier Luigi Fiorini pierluigi.fiorini@gmail.com --- Yes I will do it tonight. Sorry for the delay.
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #4 from Pier Luigi Fiorini pierluigi.fiorini@gmail.com --- name: ok
source: ok SHA1 0c69697a66d6d0908d3822ecebedb08c2c424942 libksane-15.12.0.tar.xz
license: ok
post/postun ldconfig: ok
filelist: ok
APPROVE
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Mattia Verga mattia.verga@tiscali.it changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mattia.verga@tiscali.it
--- Comment #5 from Mattia Verga mattia.verga@tiscali.it --- Pier Luigi, I don't think this way of conducting reviews are acceptable. You do them in a too simplified way.
You should follow the entire checklist on https://fedoraproject.org/wiki/Packaging:ReviewGuidelines?rd=Packaging/Revie... or you can use fedora-review tool to automate that.
For example, in this review the .src.rpm file refers to libksane-15.12.0 while the .spec file is for libksane-15.12.1 and using fedora-review the check fails.
I know Rex is a "trusted" packager, but for the future please make reviews with better quality.
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #6 from Pier Luigi Fiorini pierluigi.fiorini@gmail.com ---
Pier Luigi, I don't think this way of conducting reviews are acceptable. You do > them in a too simplified way.
You should follow the entire checklist on https://fedoraproject.org/wiki/Packaging:ReviewGuidelines?rd=Packaging/Revie... or you can use fedora-review tool to automate that.
For simplicity I didn't put all the review check list items in the report here. Thanks for suggesting fedora-review.
For example, in this review the .src.rpm file refers to libksane-15.12.0 while > the .spec file is for libksane-15.12.1 and using fedora-review the check fails.
That was changed after I picked up the review but didn't come to my mind to download the files again. Next time I'll be more accurate.
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #7 from Rex Dieter rdieter@math.unl.edu --- Pier, you said approve in comment #4, but did not set the fedora-review flag to + , can you do so please?
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Pier Luigi Fiorini pierluigi.fiorini@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |RAWHIDE Last Closed| |2016-03-29 10:02:52
--- Comment #8 from Rex Dieter rdieter@math.unl.edu --- imported, thanks
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #9 from Fedora Update System updates@fedoraproject.org --- kf5-libksane-15.12.3-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-d0ceed9b2b
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #10 from Fedora Update System updates@fedoraproject.org --- kf5-libksane-15.12.3-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-a2f57ba56a
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |ON_QA Resolution|RAWHIDE |--- Keywords| |Reopened
--- Comment #11 from Fedora Update System updates@fedoraproject.org --- kf5-libksane-15.12.3-1.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-d0ceed9b2b
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #12 from Fedora Update System updates@fedoraproject.org --- kf5-libksane-15.12.3-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-a2f57ba56a
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #13 from Fedora Update System updates@fedoraproject.org --- kf5-libksane-15.12.3-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed|2016-03-29 10:02:52 |2016-04-03 13:57:28
https://bugzilla.redhat.com/show_bug.cgi?id=1294852
--- Comment #14 from Fedora Update System updates@fedoraproject.org --- kf5-libksane-15.12.3-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.
package-review@lists.fedoraproject.org