Skip to content

Remove cairo-sphinx

Uli Schlachter requested to merge psychon/cairo:remove-sphinx into master

I tried to use cairo-sphinx do to something useful. I failed.

The cairo-sphinx binary was added in commit 6e0b3be9 in 2009 by ickle with the explanation:

Add cairo-sphinx utility for regression analysis

sphinx is an alternate version of the current cairo-test-trace. It's
purpose is to replay a live application (which may just be a replay of a
trace) against a backend and its reference. The improvement over the
original cairo-test-trace is that the reference backend may be from an
older version of cairo.

Ever since this commit, there were only build fixes to this code (for example to fix compilation on AIX or to fix compilation after cairo-tee.h became optional). So, either this code was perfect from the beginning or it was not much used.

Next, I tried to reverse-engineer a bit how to actually use this. The non-existing documentation didn't help much. The result? Well... something happened, but I am not quite sure what. Then the server process just exited.

So, in this commit I am removing cairo-sphinx. The hope is that someone speaks up and complains. That person then hopefully can provide some explanation on how to use this. Alternatively, no one speaks up and we no longer have to maintain this piece of code.

Here is an alternative to !509 (merged). Let's see if anyone speaks up and wants to keep this code.

Merge request reports