https://bugzilla.redhat.com/show_bug.cgi?id=1220770
Bug ID: 1220770 Summary: Review Request: vim-fugitive - A Git wrapper so awesome, it should be illegal Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: vondruch@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://fedorapeople.org/cgit/vondruch/public_git/vim-fugitive.git/tree/vim-... SRPM URL: http://people.redhat.com/vondruch/vim-fugitive-2.2-1.fc23.src.rpm Description: fugitive.vim may very well be the best Git wrapper of all time. Fedora Account System Username: vondruch
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=9711450
https://bugzilla.redhat.com/show_bug.cgi?id=1220770
--- Comment #1 from Vít Ondruch vondruch@redhat.com --- Forgot to run rpmlint and it revealed some issues indeed.
Spec URL: https://fedorapeople.org/cgit/vondruch/public_git/vim-fugitive.git/tree/vim-... SRPM URL: http://people.redhat.com/vondruch/vim-fugitive-2.2-2.fc23.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=9711686
https://bugzilla.redhat.com/show_bug.cgi?id=1220770
Bohuslav "Slavek" Kabrda bkabrda@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |bkabrda@redhat.com Assignee|nobody@fedoraproject.org |bkabrda@redhat.com
--- Comment #2 from Bohuslav "Slavek" Kabrda bkabrda@redhat.com --- Taking this.
https://bugzilla.redhat.com/show_bug.cgi?id=1220770
Bohuslav "Slavek" Kabrda bkabrda@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review+
--- Comment #3 from Bohuslav "Slavek" Kabrda bkabrda@redhat.com --- - rpmlint output is OK (it only gives a couple of spelling error warnings which actually are false positives) - licensing is fine - everything else including %post and %postun scriptlets also seems to be fine, package builds cleanly in mock
I can't find a single problem with this package, therefore it is APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=1220770
--- Comment #4 from Vít Ondruch vondruch@redhat.com --- Appreciate your swift review. Trying request new package in pkgdb [1], lets see how it works :)
https://admin.fedoraproject.org/pkgdb/package/requests/119
https://bugzilla.redhat.com/show_bug.cgi?id=1220770
Vít Ondruch vondruch@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |vim-fugitive-2.2-2.fc23 Resolution|--- |RAWHIDE Last Closed| |2015-05-13 11:39:40
https://bugzilla.redhat.com/show_bug.cgi?id=1220770
Kamil Dudka kdudka@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |marc@marcdeop.com
--- Comment #5 from Kamil Dudka kdudka@redhat.com --- *** Bug 1236403 has been marked as a duplicate of this bug. ***
package-review@lists.fedoraproject.org