Overview

Request 211781 accepted

Added Patch-Tag-Line.
Patch submitted to gnome-db-list mailing list.

The problem this patch solves is as follows: the new graphviz
package (which is not yet in Factory, but is supposed to be
merged in the near future) will no longer provide the old
libgraph API, but only the new libcgraph API.

The libgda 5.2.1 upstream release contains a configure check
to detect which of the two APIs is supported. Unfortunately,
this configure check always fails since it just includes
without actually setting up the include path needed for graphviz.

This means the test always returns that the old API should be
used. This does in fact work correctly in current Factory,
but will break once the new graphviz package is merged.

This patch fixes the configure test to use the correct includes,
so that should now actually correctly detect the graphviz API
level supported by the system.

Note that we also need to add BuildRequires: yelp-tools,
since applying the configure.ac patch triggers an autoreconf,
which will generate an incorrect configure file if the yelp.m4
file is not available on the system.

- libgda-new-graphviz.patch: Fix detection of new Graphviz API. (forwarded request 211435 from uweigand)

Loading...
Request History
Dominique Leuenberger's avatar

dimstar created request

Added Patch-Tag-Line.
Patch submitted to gnome-db-list mailing list.

The problem this patch solves is as follows: the new graphviz
package (which is not yet in Factory, but is supposed to be
merged in the near future) will no longer provide the old
libgraph API, but only the new libcgraph API.

The libgda 5.2.1 upstream release contains a configure check
to detect which of the two APIs is supported. Unfortunately,
this configure check always fails since it just includes
without actually setting up the include path needed for graphviz.

This means the test always returns that the old API should be
used. This does in fact work correctly in current Factory,
but will break once the new graphviz package is merged.

This patch fixes the configure test to use the correct includes,
so that should now actually correctly detect the graphviz API
level supported by the system.

Note that we also need to add BuildRequires: yelp-tools,
since applying the configure.ac patch triggers an autoreconf,
which will generate an incorrect configure file if the yelp.m4
file is not available on the system.

- libgda-new-graphviz.patch: Fix detection of new Graphviz API. (forwarded request 211435 from uweigand)


Factory Auto's avatar

factory-auto added a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Factory Auto's avatar

factory-auto added a reviewer

Please review build success


Saul Goodman's avatar

licensedigger accepted review

{"approve": "license and version number unchanged: 5.2.1"}


Factory Repo Checker's avatar

factory-repo-checker accepted review

Builds for repo openSUSE_Factory


Michal Vyskocil's avatar

mvyskocil accepted review


Michal Vyskocil's avatar

mvyskocil approved review


Tomáš Chvátal's avatar

scarabeus_factory accepted request

checkin to openSUSE:Factory only

openSUSE Build Service is sponsored by