[cairo] [PATCH 0/7] Bunch of distcheck fixes
Bryce W. Harrington
b.harrington at samsung.com
Thu Mar 13 18:04:13 PDT 2014
I've gone ahead and pushed patches 1, 2, 3, 6, and 7 from this series to
master.
Patch 4 I modified as suggested by Uri and pushed, to indicate skia has
been in the codebase since 1.10 rather than 1.12.
Patch 5 I dropped and used Uri's alternate fix instead, after checking
that it resolves the gtk-doc issues.
As mentioned below, there is still some minor cruft in distcheck, but
nothing IMHO that is worth holding up the release for. However, tests
are in a poor shape right now.
I ran 'make check' comparisons between the 1.12.8 release (as tagged in
git), and master HEAD from earlier today:
1.12.8 HEAD
----------------------------------------------------------------------
Tests run: 22751 24838
Passed: 15809 15881
Failed: 4198 5385
Expected Failed: 405 330
Error: 1 1
Crashed: 227 1018
Untested: 2111 2223
Total: 22751 24838
I've attached a listing of the test cases that changed their results.
Failures seem to be rather across the board, although there does seem to
be a lot relating to clipping/masking and transform/scaling.
Bryce
On Wed, Feb 26, 2014 at 10:29:41PM +0000, Bryce W. Harrington wrote:
> With most of the build warnings squared away, here's a set of fixes for
> various problems encountered when running make distcheck.
>
> Largely these are documentation issues. I've filled in bits as best I
> could figure out, and moved things around a bit to make the doc checker
> happy, but I would certainly appreciate a close review. Some of the
> docs could benefit from a bit more elaboration by someone more familiar
> with the given code.
>
> I also had to make some judgment calls in adding Since fields to various
> API bits. I tried to find when the interface was first mentioned in
> NEWS, or lacking that tried to identify the date of introduction into
> git, and then assume it was part of the next major cairo release and
> used that number. Those numbers may be off; I don't know how critical
> these numbers are though. There may be cases where I've added versions
> but it really should be TBD; if you spot these let me know.
>
> This patch set cleans up nearly all the distcheck issues, but there's a
> few things remaining:
>
> * warning about uninitialized variables
>
> * warnings about inlining failures
>
> * warnings about ignored return values (Bugzilla #58727)
>
> * Lots and lots of test failures
>
> * rsvg_pixbuf_from_file is deprecated, so svg2png needs recoded a bit.
>
> * four unused CAIRO_HAS_*_FUNCTIONS declarations aren't included in
> cairo-sections.txt.
>
> * gtk-doc can't parse "SkBitmap::Config&" args so some skia functions
> aren't getting processed
>
> The first three we've discussed on the list and felt were pretty minor.
> The test failures need further analysis, and likely a lot more work.
> The gtk-doc issue I suspect is gtk-doc being confused by C++ code, and
> maybe not something to worry about; maybe a workaround would be enough.
> The other two issues are minor and probably more work to fix than worth.
>
>
>
> Bryce Harrington (7):
> doc: Add missing sections and symbols for public docs
> doc: Drop a couple quartz routines which distcheck claims don't exist
> Mark recently added _cairo_output_stream_print_matrix private symbol
> skia: Add section definitions and code docs for skia backend
> check-def: Fix code document errors
> surface: Make parameter naming consistent between header and impl
> Document that libglib2.0-doc is needed to avoid some xref warnings
>
> RELEASING | 4 +-
> doc/public/cairo-sections.txt | 42 +++++++++++++++++++--
> src/cairo-cache-private.h | 2 +
> src/cairo-output-stream-private.h | 2 +-
> src/cairo-raster-source-pattern.c | 2 -
> src/cairo-region.c | 16 +-------
> src/cairo-skia-surface.cpp | 21 +++++++++++
> src/cairo-surface-observer.c | 5 +++
> src/cairo-surface.c | 12 +++---
> src/cairo-types-private.h | 74 ++++++++++++++++++++-----------------
> src/cairo.h | 20 ++++++++++
> src/skia/cairo-skia-private.h | 19 ++++++++++
> 12 files changed, 158 insertions(+), 61 deletions(-)
>
> --
> 1.7.9.5
More information about the cairo
mailing list