svn commit: r473793 - in head/devel: . cwrap nss_wrapper pam_wrapper resolv_wrapper socket_wrapper uid_wrapper
Timur I. Bakeyev
timur at freebsd.org
Tue Jul 3 13:13:51 UTC 2018
On 3 July 2018 at 08:58, Mathieu Arnold <mat at freebsd.org> wrote:
> On Tue, Jul 03, 2018 at 03:29:43AM +0000, Timur I. Bakeyev wrote:
> > Added: head/devel/nss_wrapper/Makefile
> > ============================================================
> ==================
> > --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> > +++ head/devel/nss_wrapper/Makefile Tue Jul 3 03:29:42 2018
> (r473793)
> > +
> > +CMAKE_ARGS+= -DMAN_INSTALL_DIR:PATH="${PREFIX}/man"
>
> Why +=?
>
Fixed.
> +OPTIONS_DEFINE= TEST
> > +OPTIONS_DEFAULT= TEST
> > +
> > +TEST_TARGET= test
>
> Fun fact, TEST_TARGET may not do what you think it does. What it does
> is tell the framework that the target to run when running `make test` is
> test, which will probably fail if the TEST option is not enabled. What
> you may have meant to set was TEST_TEST_TARGET, which sets TEST_TARGET
> when the TEST option is enabled.
>
Fixed.
> +TEST_CMAKE_BOOL= UNIT_TESTING
> > +TEST_BUILD_DEPENDS+= cmocka>=1.1.1:sysutils/cmocka
>
> Why +=?
>
Fixed.
> These are a lot of files, you should probably be using a pkg-plist file
> instead.
>
I don't feel that 7(8) lines is an excessive amount of lines. I'd rather
save inodes here.
> Added: head/devel/pam_wrapper/Makefile
>
> Way too many files to be using PLIST_FILES, please use a pkg-plist file.
>
> Ok, this one is excessive, moved to pkg-plist.
The rest are fixed in the same manner as nss_wrapper.
BaT
More information about the svn-ports-head
mailing list