Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 17 Oct 2015 21:50:02 -0400
From:      Julio Merino <jmmv@freebsd.org>
To:        NGie Cooper <yaneurabeya@gmail.com>
Cc:        "freebsd-testing@freebsd.org" <freebsd-testing@FreeBSD.org>
Subject:   Re: [RFC] Rename `make test` in suite.test.mk with `make regress`
Message-ID:  <5C39F495-5717-450F-8460-14B69DD48AD6@freebsd.org>
In-Reply-To: <F3E9AA8C-690E-4E66-827B-B99981B8B336@gmail.com>
References:  <F3E9AA8C-690E-4E66-827B-B99981B8B336@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Oct 17, 2015, at 18:03, NGie Cooper <yaneurabeya@gmail.com> wrote:
>=20
> Hi all,
> 	There=E2=80=99s a lesser known target in suite.test.mk that runs =
`kyua test` in a similar manner to how Jenkins and other groups have =
integrated kyua into their test infrastructures.
> 	The legacy target on FreeBSD was `regress`, but the target =
created with the bsd.test.mk creation back a few years ago was `test`. =
Why change from `test` to `regress`? There are places in the tree =
(bin/test for example) that have targets named test, so in order to =
avoid clashing with a common target (name), it=E2=80=99s best to use the =
legacy target name.
> 	Would anyone have any serious heartburn over the change?

Is this only because of bin/test?  Seems like renaming a target to avoid =
that one collision is just moving the problem around.  It is possible =
that some other directory could later grow a target that conflicts with =
your new name.

Strictly speaking, "regress" is wrong.  We do not have regression tests =
only.  Also, "regress" is a pretty obscure name for a target; it does =
not appear in any other projects nor in any other build systems that I =
know of.

Have you considered "check"?  That'd be in line with what automake does, =
for example, which would homogenize the target name with a ton of other =
projects out there.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5C39F495-5717-450F-8460-14B69DD48AD6>