Spec URL: https://github.com/fedora-python/python3/blob/platform_python/platform-python.spec SRPM URL: https://github.com/torsava/python3-1/blob/platform_python_srpm/platform-python-3.6.2-10.fc27.src.rpm?raw=true Patches and other files: https://github.com/fedora-python/python3/tree/platform_python Description: Python interpreter for low-level system tools, designed to work with the Platform module. Fedora Account System Username: torsava =========================================================== This is a modification of the existing `python3` package. It is designed for the [Modularity] initiative to be part of the Platform module, specifically to enable dnf. The lengths of the Python spec files are commensurate with the important role they serve, and thus we provide the [diff] between the proposed platform-python and the existing python3 package to serve as a better reviewing tool than the totality of the spec. [Modularity] https://docs.pagure.org/modularity/ [diff] https://github.com/fedora-python/python3/pull/3
I have checked the diff and it looks good to me (this is not a formal review).
Here is the corresponding Fedora Change: https://fedoraproject.org/wiki/Changes/Platform_Python_Stack
Spec URL: https://github.com/fedora-python/python3/blob/platform_python/platform-python.spec?raw=true SRPM URL: https://github.com/torsava/python3-1/blob/platform_python_srpm/platform-python-3.6.2-10.fc27.src.rpm?raw=true Adding 'raw=true' to the URL so `fedora-review` will work.
Spec URL: https://raw.githubusercontent.com/fedora-python/python3/platform_python/platform-python.spec SRPM URL: https://raw.githubusercontent.com/torsava/python3-1/platform_python_srpm/platform-python-3.6.2-10.fc27.src.rpm Apparently fedora-review drops the arguments. So here's the paths to the actual raw content.
OK, I reviewed this by inspection (because fedora-review can't handle it). In brief: things look fine to me, all licenses and legalities appear to be in order and the diff from the python3 package looks completely sane. I'm comfortable approving this.
(fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/platform-python
Cannot push to dist-git https://pagure.io/fedora-infrastructure/issue/6260
Solved. Building in https://koji.fedoraproject.org/koji/taskinfo?taskID=21258874
Thank you, Stephen, for a prompt review!