Re: [waffle] [PATCH 00/29] Rework the functionality test(s)
On 14 June 2016 at 18:10, Chad Versacewrote: > On Tue 26 Apr 2016, Emil Velikov wrote: >> Humble ping ? > > And an embarassedly late merge. > > Thanks for all the cleanups. All the patches in the series, except patch > 1, are merged. I skipped patch 1 because it introduced a single symbol, > but that symbol was never used. I'm slightly leaning towards having that symbol, as with it (and a gbm cleanup) one should be able to run the waffle tests on Travis CI (which has old wayland). I won't insist on it though. Alternatively we should really bump the minimum required wayland version to 1.4. Which one would you prefer ? Thanks Emil ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
Re: [waffle] [PATCH 00/29] Rework the functionality test(s)
On Tue 26 Apr 2016, Emil Velikov wrote: > Humble ping ? And an embarassedly late merge. Thanks for all the cleanups. All the patches in the series, except patch 1, are merged. I skipped patch 1 because it introduced a single symbol, but that symbol was never used. ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
Re: [waffle] [PATCH 00/29] Rework the functionality test(s)
Humble ping ? On 6 January 2016 at 21:27, Emil Velikovwrote: > Hi all, > > Recently I had the chance to finish a lenghty series started some months > ago. Namely - rework gl_basic_tests and kick waffle_test out the door. > > It starts with a couple of symbol fixes (maint material afaict), caught > while experimenting with Travis-CI. > > Patch 3 adds a trivial --platform param and wires up the build (make > check-func) to generate and use all possible combinations. > > Patches 06 through 14 add a bunch of macros, which minimise the > duplication, and make things substantially easier/shorter add new > platform (patch 28, platform gbm). > > We might want patches 15, 16 and 17 squashed, although I chose to keep > the "import latest cmoka", "make sure it builds" and "fix the build > warning" (this one is on the cmocka ML), separate to ease review. > > With patch 18, the existing cmocka user is updated - things build even > without it, but we no longer use the deprecated API. > > With 20, we disable the functionality tests as we cannot convert the > whole file at once. Immediatelly after (21) we nuke waffle_test. > > Patch 22, introduce a handy macro (not perfect due to cmocka API > shortage), that makes the "add new platform" a ~10 line diff :-) > > The remaining patches convert the remaining platforms, one at a time, to > top up with a small cleanup. > > > The lot can be found in branch for-upstream/rework-tests over at > https://github.com/evelikov/waffle/ > > TL;DR - update cmocka, and use it over waffle_test. Cut gl_basic_test.c > from ~1.2k to 0.9 loc, while adding GBM support and platform selection > ;-) No more memory leaks with valgrind-check-func (barring driver ones). > > NOTE: I'll gladly rework any formatting and macro names. If possible, > let's avoid rebase chaos (i.e. if thinking about the test... macros) and > do them on top ? > > Thanks > Emil > ___ waffle mailing list waffle@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/waffle
[waffle] [PATCH 00/29] Rework the functionality test(s)
Hi all, Recently I had the chance to finish a lenghty series started some months ago. Namely - rework gl_basic_tests and kick waffle_test out the door. It starts with a couple of symbol fixes (maint material afaict), caught while experimenting with Travis-CI. Patch 3 adds a trivial --platform param and wires up the build (make check-func) to generate and use all possible combinations. Patches 06 through 14 add a bunch of macros, which minimise the duplication, and make things substantially easier/shorter add new platform (patch 28, platform gbm). We might want patches 15, 16 and 17 squashed, although I chose to keep the "import latest cmoka", "make sure it builds" and "fix the build warning" (this one is on the cmocka ML), separate to ease review. With patch 18, the existing cmocka user is updated - things build even without it, but we no longer use the deprecated API. With 20, we disable the functionality tests as we cannot convert the whole file at once. Immediatelly after (21) we nuke waffle_test. Patch 22, introduce a handy macro (not perfect due to cmocka API shortage), that makes the "add new platform" a ~10 line diff :-) The remaining patches convert the remaining platforms, one at a time, to top up with a small cleanup. The lot can be found in branch for-upstream/rework-tests over at https://github.com/evelikov/waffle/ TL;DR - update cmocka, and use it over waffle_test. Cut gl_basic_test.c from ~1.2k to 0.9 loc, while adding GBM support and platform selection ;-) No more memory leaks with valgrind-check-func (barring driver ones). NOTE: I'll gladly rework any formatting and macro names. If possible, let's avoid rebase chaos (i.e. if thinking about the test... macros) and do them on top ? Thanks Emil ___ waffle mailing list waffle@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/waffle