New package: rivalcfg-4.17.0#60987
Conversation
uggwar
commented
Jun 10, 2026
- I tested the changes in this PR: YES
- This new package conforms to the package requirements: YES
- I built this PR locally for my native architecture, (x86_64-glibc)
|
Are you sure it's licensed under Apache-2.0 and not WTFPL? Their github page says it's the latter, in which case, you should probably write WTFPL and do |
|
Auch, you are correct. Missed that one! Thanks, will fix tonight! |
ahesford
left a comment
There was a problem hiding this comment.
We don't vendor content like those udev rules, especially when they can be automatically generated by the program.
| homepage="https://github.com/flozz/rivalcfg" | ||
| distfiles="https://github.com/flozz/rivalcfg/archive/v${version}.tar.gz" | ||
| checksum=9ef0512b097ce46b77d0c74eb9c84411250c91a4dea1c77469a41bfacbe8705d | ||
| make_check=no # no tests available |
There was a problem hiding this comment.
What are these tests? https://github.com/flozz/rivalcfg/tree/master/test
There was a problem hiding this comment.
I used another project as a template and this slipped through. Will remove and test!
| maintainer="Christer Stenbrenden <christer@uggwar.net>" | ||
| license="WTFPL" | ||
| homepage="https://github.com/flozz/rivalcfg" | ||
| distfiles="https://github.com/flozz/rivalcfg/archive/v${version}.tar.gz" |
There was a problem hiding this comment.
Fetching from $PYPI_SITE should be preferred, unless the GitHub tarball offers content (like tests) that are not shipped in the PyPI distribution.
There was a problem hiding this comment.
Thanks, didn't know. Will change this and see how it goes.
There was a problem hiding this comment.
I really hate that hard coded path for the udev rules that it checks every time it runs. I removed the patch that changes this path, since it probably is better to just go with the application defaults. What do you think?