[Bug 281522] [NEW PORT] sysutils/RyzenAdj: Power management tool for Ryzen mobile and embedded CPUs

From: <bugzilla-noreply_at_freebsd.org>
Date: Wed, 13 Nov 2024 15:39:33 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=281522

Robert Clausecker <fuz@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|New                         |Open
                 CC|                            |fuz@FreeBSD.org

--- Comment #3 from Robert Clausecker <fuz@FreeBSD.org> ---
Thank you for your submission and I'm sorry it took me so long to review this
one.  Here are some comments:

 - you don't need a DOCS option to install a README file.  In fact, I don't see
   how the README is useful at all, as it mostly contains information already
   included in the ryzenadj binary or information pertaining the installation
of
   the software.
 - instead, consider installing the examples shipped with the project
 - please extend pkg-descr to at least three lines.  The file should tell the
   user why he or she might want to install the port and what features it has.
 - GH_PROJECT defaults to PORTNAME and needs not be set explicitly in this case
 - please update the port to the most recent version (0.16.0)

Port is fine otherwise and with these issues addressed, it can be committed
quickly.

-- 
You are receiving this mail because:
You are the assignee for the bug.