[RFC] Rename `make test` in suite.test.mk with `make regress`
NGie Cooper
yaneurabeya at gmail.com
Sun Oct 18 01:56:06 UTC 2015
> On Oct 17, 2015, at 18:50, Julio Merino <jmmv at freebsd.org> wrote:
>
> On Oct 17, 2015, at 18:03, NGie Cooper <yaneurabeya at gmail.com> wrote:
>>
>> Hi all,
>> There’s 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’s 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.
Yes, I realize that. The goal though is I want to be able to call `make <something>` from the top and it would iterate down each and every subdirectory and run tests.
Sadly, most people don’t care to figure out how kyua works enough to run `kyua test`… so I’m just trying to lower the barrier of entry in a way that will work 100% of the time.
> 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.
Unfortunately it’s been in place a lot longer than anything else, so it’s kind of the “defacto standard”. That said...
> 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.
… `make check` would be ok too. I just have to comb the tree for binaries that don’t match `check`.
Thanks!
-NGie
More information about the freebsd-testing
mailing list