Bug 2024917
| Summary: | Review Request: pipewalker - "complete the circuit" puzzle game | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Artur Frenszek-Iwicki <fedora> |
| Component: | Package Review | Assignee: | Zbigniew Jędrzejewski-Szmek <zbyszek> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | package-review, zbyszek |
| Target Milestone: | --- | Flags: | zbyszek:
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: | 2021-12-02 01:15:08 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
Artur Frenszek-Iwicki
2021-11-19 14:02:45 UTC
> Patch0: %{name}--format-security.patch Using a macro here is *allowed*, but why would one want to do this? For me it does nothing except making it impossible to easily open the file from an editor which does not understand rpm macros. > %description It's broken at ~60-70 chars, should be 80 ;) One thing that is missing is an appdata file [1]. Guidelines say "SHOULD", so this is not a blocker, but please consider adding one, as it will make graphical installations much easier, which is probably important for the class of users who use games. [1] https://docs.fedoraproject.org/en-US/packaging-guidelines/AppData/ - package name is OK - latest version (2012-03-19, yikes!) - license is acceptable for Fedora (GPLv3+) - license is specified correctly - builds and installs correctly - BR/R/P look OK - runs OK: The window shows and reacts to clicks, sounds and templates work. rpmlint: pipewalker.x86_64: W: wrong-file-end-of-line-encoding /usr/share/doc/pipewalker/README pipewalker.x86_64: W: no-manual-page-for-binary pipewalker 4 packages and 0 specfiles checked; 0 errors, 2 warnings. Nothing important. I think it'd be nice to add some README. I couldn't figure out what the point of the game was ;) > One thing that is missing is an appdata file [1]. Guidelines say "SHOULD", so this is
> not a blocker, but please consider adding one, as it will make graphical installations
> much easier, which is probably important for the class of users who use games.
I'll probably add one later, but even then, this won't change anything, since
Gnome Software has a minimum icon size criteria (48x48 or 64x64, if I remember correctly)
and apps which don't provide large enough icons don't show up in search results.
(I guess this could be mentioned in the Guidelines, somewhere? Couldn't find it anywhere.)
Though I guess I could use hq2x or something to just scale up the 32x32 pixmap.
I haven't done an appdata file in a while… but I remember that generating something that would display properly in gnome-software wasn't easy ;) 'convert -geometry 64 input.bmp output.bmp' should work, though the result won't be very visually appealing. (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/pipewalker FEDORA-2021-1036589cf5 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2021-1036589cf5 FEDORA-2021-3099b4adb0 has been submitted as an update to Fedora 34. https://bodhi.fedoraproject.org/updates/FEDORA-2021-3099b4adb0 FEDORA-2021-3099b4adb0 has been pushed to the Fedora 34 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2021-3099b4adb0 \*` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-3099b4adb0 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. FEDORA-2021-1036589cf5 has been pushed to the Fedora 35 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2021-1036589cf5 \*` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-1036589cf5 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. FEDORA-2021-1036589cf5 has been pushed to the Fedora 35 stable repository. If problem still persists, please make note of it in this bug report. FEDORA-2021-3099b4adb0 has been pushed to the Fedora 34 stable repository. If problem still persists, please make note of it in this bug report. |