Bug 624690
| Summary: | Enable and fix testsuite in graphviz | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 6 | Reporter: | Michal Nowak <mnowak> | ||||||||
| Component: | graphviz | Assignee: | Jaroslav Škarvada <jskarvad> | ||||||||
| Status: | CLOSED ERRATA | QA Contact: | BaseOS QE - Apps <qe-baseos-apps> | ||||||||
| Severity: | medium | Docs Contact: | |||||||||
| Priority: | low | ||||||||||
| Version: | 6.0 | CC: | jskarvad, lzachar, ohudlick, rvokal | ||||||||
| Target Milestone: | rc | ||||||||||
| Target Release: | --- | ||||||||||
| Hardware: | All | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | graphviz-2.26.0-5.el6 | Doc Type: | Bug Fix | ||||||||
| Doc Text: |
The graphviz test suite was disabled on the PowerPC, 64-bit PowerPC and SPARC64 architectures due to unexpected terminations with segmentation faults. The test code used in the test suite did not set the TextLayout plugin correctly, which led to the crash of the test suite. This has been fixed on all architectures.
|
Story Points: | --- | ||||||||
| Clone Of: | |||||||||||
| : | 645703 (view as bug list) | Environment: | |||||||||
| Last Closed: | 2011-07-20 12:27:30 UTC | Type: | --- | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | --- | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Michal Nowak
2010-08-17 13:56:54 UTC
Fixed SIGSEGVs and several other problems. But testsuite results are highly dependant on system configration / package versions used. The source data are processed by specially created test executable and the results are diffed with upstream supplied samples. Only one pixel shift will cause the test to fail. For now all tests are failing because the upstream samples seems to be a little outdated. Created attachment 455022 [details]
Proposed SIGSEGVs fix
Fix segfaults caused by wrong initialization.
Created attachment 455023 [details]
Proposed rtest fix
Fix for counting differences even in case of error / warning output.
Created attachment 455024 [details]
Proposed spec patch
Fix failure of testsuite caused by missing fonts in buildroot.
Cloned to rawhide: Bug #645703 Forwarded upstream. Upstream bugzilla: http://graphviz.org/bugs/b2063.html After application of proposed patches the testsuite is valuable for detection of layout failures caused by dot crashes. These failures are marked as "Layout failures" in testsuite output. But the usability for regression testing (output figures comparison) is questionable even if we provide updated samples. One pixel shift in output figures (caused by e.g. fontconfig update) will cause tests to fail. These failures are marked as "Changes" in testsuite output. There is probably nothing we can easily do with these false positive failures (not counting testsuite rewrite). It seems the testsuite is designed to be run in "conserved" environment. This issue was proposed for RHEL 6.1 FasTrack but did not get resolved in time. It has been moved to RHEL 6.2 FasTrack. This issue was proposed for RHEL 6.1 FasTrack but did not get resolved in time. It has been moved to RHEL 6.2 FasTrack.
Technical note added. If any revisions are required, please edit the "Technical Notes" field
accordingly. All revisions will be proofread by the Engineering Content Services team.
New Contents:
The graphviz test suite was disabled on the PowerPC, 64-bit PowerPC and SPARC64 architectures due to unexpected terminations with segmentation faults. The test code used in the test suite did not set the TextLayout plugin correctly, which led to the crash of the test suite. This has been fixed and the test suite passes on all architectures.
Technical note updated. If any revisions are required, please edit the "Technical Notes" field
accordingly. All revisions will be proofread by the Engineering Content Services team.
Diffed Contents:
@@ -1 +1 @@
-The graphviz test suite was disabled on the PowerPC, 64-bit PowerPC and SPARC64 architectures due to unexpected terminations with segmentation faults. The test code used in the test suite did not set the TextLayout plugin correctly, which led to the crash of the test suite. This has been fixed and the test suite passes on all architectures.+The graphviz test suite was disabled on the PowerPC, 64-bit PowerPC and SPARC64 architectures due to unexpected terminations with segmentation faults. The test code used in the test suite did not set the TextLayout plugin correctly, which led to the crash of the test suite. This has been fixed on all architectures.
An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on therefore solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHBA-2011-0965.html |