Re: [PATCH] Delete test/ directory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 3, 2012 at 11:30 AM, Lucas De Marchi
<lucas.demarchi@xxxxxxxxxxxxxx> wrote:
> Hi Dan,
>
> On Fri, Feb 3, 2012 at 3:23 PM, Dan McGee <dan@xxxxxxxxxxxxx> wrote:
>> Apparantly my previous patch is in error, as:
>> 1) test/ isn't even used anymore, but it is still present in git for who
>>   knows what reason, and
>
> Reason is that testsuite was merged recently. All tests should migrate
> to testsuite so we can test all of them with testsuite. I added some,
> fixed bugs, added more, but there are still pending tests on test dir.
>
> Instead of just deleting all files at once, lets delete them once they
> are migrated to testsuite?
>
> Could you help migrating them?

If I have time (which I don't right now).

I was simply wondering why `make check` wasn't working in my kmod git
checkout and came across this. It looks like the make rules have
already been removed for test/ (not to mention the .gitignore rules
were dropped in your "testsuite: add skeleton" commit, 80f9e02382).
This lead me and probably others to believe test/ was not being used
whatsoever- nothing is ever built in that directory anymore.

Can you please outline what the difference between test/ and
testsuite/ is, and why migration entails? There is no
testsuite/README, or really any documentation on how to write or run
the tests at all (other than guessing the standard `make check` would
run them).

-Dan
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux