https://bugzilla.redhat.com/show_bug.cgi?id=2348850 --- Comment #13 from Tom.Rix@xxxxxxx --- Release: 1%{?dist} - this can be %autorelease Summary: Recast Navigation is a state of the art navigation system for games and 3D applications. - rmplint complains this is tool long, I use the 'About' field in github, so can change to. - 'Industry-standard navigation-mesh toolset for games' License: zlib - license is tricky, look - https://docs.fedoraproject.org/en-US/legal/allowed-licenses/ - should be Zlib - the license check flag a few more licenses, if the files are not really used, rm them in the %prep step. - if they are needed, License: needs to be updated, and breakdown of the which files use which URL: https://github.com/recastnavigation/recastnavigation Source0: https://github.com/recastnavigation/recastnavigation/archive/refs/tags/v%{version}.tar.gz - Source0: should be prefixed with %{url}, and the 'ref/tags' may not be needed Requires: libdecor-devel - Should not need this, rpmlint complains. # Disable debug source package to avoid issues with empty debugsourcefiles.list %define debug_package %{nil} - rpmlint complains about unstripped files, so something is wrong, generally we want the libraries to be build with some level debugging for the user. %{_bindir}/Meshes/dungeon.obj %{_bindir}/Meshes/nav_test.obj %{_bindir}/Meshes/undulating.obj - These were in the tarball, it looks like the upstream was sloppy, review the RecastDemo/Bin from the tarball, and remove all/most of it in %prep %{_prefix}/bin/DroidSans.ttf - this is data, not a binary, so it needs to go somewhere like %_datadir/%{name} - likely this means any binary that needs to use it needs to be changed to look for the new location %{_bindir}/TestCases/movement_test.txt %{_bindir}/TestCases/nav_mesh_test.txt %{_bindir}/TestCases/raycast_test.txt - Similar, but maybe there needs to be an optional -test subpackage - i would like a %check to make sure it is working. -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2348850 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202348850%23c13 -- _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
