Bug 1463082

Summary:	Review Request: python-pocketlint - Support for running pylint against projects
Product:	[Fedora] Fedora	Reporter:	Vojtech Trefny <vtrefny>
Component:	Package Review	Assignee:	Jiri Konecny <jkonecny>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	jkonecny, package-review, ykarel
Target Milestone:	---	Flags:	jkonecny: fedora-review+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-07-06 02:22:42 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Vojtech Trefny 2017-06-20 06:04:50 UTC

Spec URL: https://raw.githubusercontent.com/rhinstaller/pocketlint/master/python-pocketlint.spec

SRPM URL: https://vtrefny.fedorapeople.org/srpms/python-pocketlint-0.15-1.fc27.src.rpm

Description: This is a re-review request after renaming "python3-pocketlint"[1] package to "python-pocketlint" -- we now need a Python 2 version of pocketlint, so we want to rename the package to "python-pocketlint" with two subpackages based on Python version. There is no need for Obsoletes/Provides in the spec, because "python3-pocketlint" package still exists -- the rename is only to rename the SRPM and package/component in PkgDB/SCM/Bugzilla.

Fedora Account System Username: vtrefny

[1] https://apps.fedoraproject.org/packages/python3-pocketlint

Comment 1 Jiri Konecny 2017-06-20 10:07:52 UTC

It would be nice to have "%global srcname pocketlint" instead of writing "pocketlint" everywhere.

I'm giving ACK here because the above is not a blocker issue.

Comment 2 Yatin Karel 2017-06-21 14:36:14 UTC

This is un-official review of the package.

+ mockbuild success for fedora 26, fedora 27

Some suggestions against the updated spec from previous version:
- python2-six, python2-polib,python2-futures can be used in place of python-six, python-polib,python-futures

Comment 3 Gwyn Ciesla 2017-06-22 11:05:44 UTC

Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/python-pocketlint

Comment 4 Fedora Update System 2017-06-27 10:35:19 UTC

python-pocketlint-0.15-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-8ce8e10468

Comment 5 Fedora Update System 2017-06-27 17:27:15 UTC

python-pocketlint-0.15-1.fc25 has been pushed to the Fedora 25 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-2017-96c9d5a464

Comment 6 Fedora Update System 2017-06-27 20:26:56 UTC

python-pocketlint-0.15-1.fc26 has been pushed to the Fedora 26 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-2017-8ce8e10468

Comment 7 Fedora Update System 2017-07-06 02:22:42 UTC

python-pocketlint-0.15-1.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 8 Fedora Update System 2017-07-11 20:54:13 UTC

python-pocketlint-0.15-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.