[PATCH] Integrate pjdfstest into the build

Garrett Cooper yaneurabeya at gmail.com
Mon Mar 3 18:22:46 UTC 2014


On Sun, Mar 2, 2014 at 3:00 PM, Julio Merino <jmmv at freebsd.org> wrote:
> On Sun, Mar 2, 2014 at 5:54 PM, Garrett Cooper <yanegomi at gmail.com> wrote:
>>> * The test files should move from tools/regression/pjdfstest/ into the
>>> new location under tests/sys/ and be renamed to follow the conventions
>>> under src/test (e.g. xx.t -> xx_test).
>>
>> Please talk to Pawel about this. These tests have been developed and are maintained (to some degree) outside FreeBSD: https://github.com/pjd/pjdfstest . I'd rather treat this as an integration of the 3rd party package, not a refactor. If you want me to install the tests like _test though, I'd be more than happy to do that.
>
> Ah great; that's what I suspected given the mentions of other OSes in the code!
>
> Then I believe this should really live under contrib/pjdfstest.  Any
> reason it's not there?  If it's external, then yes, we should keep the
> layout of the upstream sources without performing any renames (unless
> Pawel wants to do them).

    I think it should be, but I don't own pjdfstest ;)..
    (largely for jmmv's benefit) Ultimately if pjd went this route,
pjdfstest needs to be put in the vendor repo, reimported into FreeBSD
under contrib, and then linked into the tree appropriately.
Thank you :),
-Garrett


More information about the freebsd-testing mailing list