[PATCH v2] convert /bin/sh tests over to ATF

Garrett Cooper yaneurabeya at gmail.com
Tue Apr 22 00:21:23 UTC 2014


On Sun, Mar 9, 2014 at 1:42 PM, Julio Merino <jmmv at freebsd.org> wrote:

...

> The tests need to populate /usr/tests/bin/sh/ with the
> functional_test.sh driver and all its supporting data files. As I
> understand it, your driver code assumes that the data files are
> external (i.e. not bundled into the script itself), so those files
> must be available from the same location as where the driver lives.
> Hence you still need to use FILES to ensure the data files get
> installed into /usr/tests/.
>
> ... but maybe I'm missing something from your patch; haven't reviewed
> it again to double-check this.

    I've reduced the diff quite a bit. The end result of it can be
found here: https://github.com/yaneurabeya/freebsd/pull/2 (and it will
evolve over time if needed because github is additive in pull request
reviews). FWIW, it might be better to actually review it on github
instead of using this list as patches attached to any mailing list can
become stale over time.
    You can get the diff like so:

https://github.com/yaneurabeya/freebsd/pull/2.patch

Thanks!
-Garrett


More information about the freebsd-testing mailing list