Date: Sat, 17 Oct 2015 18:56:04 -0700 From: NGie Cooper <yaneurabeya@gmail.com> To: Julio Merino <jmmv@freebsd.org> Cc: "freebsd-testing@freebsd.org" <freebsd-testing@FreeBSD.org> Subject: Re: [RFC] Rename `make test` in suite.test.mk with `make regress` Message-ID: <E4FD723B-256D-4EF1-B1B5-C3303E77C59B@gmail.com> In-Reply-To: <5C39F495-5717-450F-8460-14B69DD48AD6@freebsd.org> References: <F3E9AA8C-690E-4E66-827B-B99981B8B336@gmail.com> <5C39F495-5717-450F-8460-14B69DD48AD6@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Oct 17, 2015, at 18:50, Julio Merino <jmmv@freebsd.org> wrote: >=20 > 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? >=20 > 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=E2=80=99t care to figure out how kyua works = enough to run `kyua test`=E2=80=A6 so I=E2=80=99m 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=E2=80=99s been in place a lot longer than anything = else, so it=E2=80=99s kind of the =E2=80=9Cdefacto standard=E2=80=9D. = 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. =E2=80=A6 `make check` would be ok too. I just have to comb the tree for = binaries that don=E2=80=99t match `check`. Thanks! -NGie=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?E4FD723B-256D-4EF1-B1B5-C3303E77C59B>