https://bugzilla.redhat.com/show_bug.cgi?id=2275305
Bug ID: 2275305 Summary: Review Request: python-glfw - A Python ctypes-based wrapper for GLFW3 Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Assignee: nobody@fedoraproject.org Reporter: t.imarthurlee@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://github.com/timarthurlee/uml-redhat-capstone-2023/raw/main/rpmbuild/S... SRPM URL: https://github.com/timarthurlee/uml-redhat-capstone-2023/raw/main/rpmbuild/S... Description: This module provides Python bindings for GLFW < (on GitHub: glfw/glfw < It is a ctypes wrapper which keeps very close to the original GLFW API. Fedora Account System Username: timarthurlee This is one of my first packages and I will need a sponsor. I am submitting this as a part of the 2023 Friendly Fedora capstone project team at UMass Lowell.
The rpm was successfully built in koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=116458826
https://bugzilla.redhat.com/show_bug.cgi?id=2275305
Sandro gui1ty@penguinpee.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? Assignee|nobody@fedoraproject.org |gui1ty@penguinpee.nl CC| |gui1ty@penguinpee.nl Doc Type|--- |If docs needed, set a value Status|NEW |ASSIGNED
--- Comment #1 from Sandro gui1ty@penguinpee.nl --- I'm taking this. Seems I might need it for another package I'm working on.
Glancing over the spec file, I have the following preliminary remarks:
In `setup.py` upstream defines extra `preview`. That should be packaged as such. See:
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#Extras
However, since glfw-preview [1] is not yet packaged for Fedora, you can leave out the extra. On the other hand, `glfw-preview` uses the same source as `glfw`, so it seems reasonable to package that as well. It should probably be a package of its own, but I'll check on that.
You should use the modern Python (pyproject) macros described in [2], unless there's a compelling reason for using the older macros [3]. Using the modern macros, you should not need to specify any Provides or Requires. They will be detected and added automatically. You have `Requires: python3dist(glfw-preview)`. That seems wrong since only the optional `preview` extra requires it. It also causes the package not to install since nothing provides `python3dist(glfw-preview)`.
The modern macros will also make it easy to populate the `%files` section (see [4]).
Lastly, please consider defining a macro for the description instead of repeating the content. See the example spec file [4] for inspiration. It will make maintaining the spec file easier.
[1] https://pypi.org/project/glfw-preview/ [2] https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/ [3] https://docs.fedoraproject.org/en-US/packaging-guidelines/Python_201x/ [4] https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_example_s...
https://bugzilla.redhat.com/show_bug.cgi?id=2275305
Sandro gui1ty@penguinpee.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(t.imarthurlee@gma | |il.com)
--- Comment #2 from Sandro gui1ty@penguinpee.nl --- Ping?
Tim, are you still interested in getting this packaged for Fedora?
https://bugzilla.redhat.com/show_bug.cgi?id=2275305
--- Comment #3 from Sandro gui1ty@penguinpee.nl --- This review appears to be stalled. It's been four weeks since I took it. I'll wait another week as per stalled review policy [1] before closing this as a dead review
[1] https://docs.fedoraproject.org/en-US/fesco/Package_review_policy/#submitter_...
https://bugzilla.redhat.com/show_bug.cgi?id=2275305
Sandro gui1ty@penguinpee.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | |needinfo?(t.imarthurlee@gma | |il.com) | Blocks| |201449 (FE-DEADREVIEW) Status|ASSIGNED |CLOSED Resolution|--- |DUPLICATE Last Closed| |2024-07-04 10:41:02
--- Comment #4 from Sandro gui1ty@penguinpee.nl ---
*** This bug has been marked as a duplicate of bug 2295768 ***
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug.
package-review@lists.fedoraproject.org