Bug 58302 - Compile tests on `make check` invocation
Summary: Compile tests on `make check` invocation
Status: RESOLVED MOVED
Alias: None
Product: cairo
Classification: Unclassified
Component: general (show other bugs)
Version: 1.12.8
Hardware: Other All
: medium normal
Assignee: Chris Wilson
QA Contact: cairo-bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-14 16:24 UTC by Marvin Schmidt
Modified: 2018-08-25 13:33 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
use check_PROGRAMS to compile tests on make check (645 bytes, text/plain)
2012-12-14 16:24 UTC, Marvin Schmidt
Details

Description Marvin Schmidt 2012-12-14 16:24:10 UTC
Created attachment 71511 [details]
use check_PROGRAMS to compile tests on make check

Currently tests are treated like normal programs and are therefore built during `make`. This requires having all test dependencies installed to succeed. Among the test dependencies are packages which depend on cairo itself, causing a circular dependency.
To work around this one could compile cairo first without running tests, compile all test dependencies and then get back to running the tests. This can easily be made possible by using "check_PROGRAMS" for the tests instead of "noinst_PROGRAMS", resulting in the tests getting built when `make check` is invoked
The attached patch does just that
Comment 1 Uli Schlachter 2013-09-15 11:48:37 UTC
I was wondering about the "always build" comment on that line. It comes from the following commit:

commit b8f43617a98aeb6c10d554ed11b48a83fc9b5129
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Sat Jul 16 20:45:03 2011 +0100

    test: Always compile cairo-test-suite
    
    I'm tired of having to explicitly type make cairo-test-suite.

I know that "make check" will compile this after your patch, but thanks to newer automakes hiding test's output, I think that running the test suite through "make check" is quite useless.

In other words, the proposed patch effictively reverts the above commit and thus I don't think it is likely that this will get merged (just my 5c).
Comment 2 GitLab Migration User 2018-08-25 13:33:55 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/cairo/cairo/issues/70.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.