Bug 458607 - dependency on the cppunit testing framework
Summary: dependency on the cppunit testing framework
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: subtitleeditor
Version: rawhide
Hardware: All
OS: Linux
medium
low
Target Milestone: ---
Assignee: Martin Sourada
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-10 21:08 UTC by Lubos Stanek
Modified: 2008-09-25 00:27 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-09-25 00:08:57 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Lubos Stanek 2008-08-10 21:08:56 UTC 
Description of problem:
The package build dependencies include cppunit-devel. The application is built with the unit testing code included but the build script does not check the code in any way. Should the end user check it?

Version-Release number of selected component (if applicable): 0.21.3

How reproducible: See the .spec file.

Additional info:
The unit testing code is designed for developers to verify the code. I do not think that it should be included in the release build.
Comment 1 Lubos Stanek 2008-08-19 11:26:01 UTC 
What info do you want?

The unit testing code is designed for the developer to check whether the application works as designed with different subtitle formats. It uses sample test files from the source subdirectory. The internal debug functions are more than necessary for testing.

Building the application with testing code is possible but it is useless. No end user will ever use it. It does not work without test files and they are not packaged in. It only expands the size of the target executable.

Is it enough information for the decision to remove the offending configure switch from the spec file?
Comment 2 Martin Sourada 2008-08-19 11:44:22 UTC 
Ah, OK, I'll fix this in next release (I am waiting for infrastructure guys to get everything working again). Thanks for filling the bug and providing the info :) IIRC, I initially added the switch as per suggestion from the package reviewer...
Comment 3 Lubos Stanek 2008-08-19 20:51:35 UTC 
Good to read that. :-)
Do not forget to remove both the configure switch and the appropriate BuildRequires.

It would be useable when the unit test code would be built as a standalone binary and used in the make check. Probably it was the reviewer's idea.

Thanks for packaging it for Fedora users.
Comment 4 Fedora Update System 2008-08-25 16:09:42 UTC 
subtitleeditor-0.22.3-1.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/subtitleeditor-0.22.3-1.fc8
Comment 5 Fedora Update System 2008-09-10 06:55:54 UTC 
subtitleeditor-0.22.3-1.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update subtitleeditor'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F8/FEDORA-2008-7582
Comment 6 Fedora Update System 2008-09-10 07:12:18 UTC 
subtitleeditor-0.22.3-1.fc9 has been pushed to the Fedora 9 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update subtitleeditor'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2008-7399
Comment 7 Fedora Update System 2008-09-25 00:08:54 UTC 
subtitleeditor-0.22.3-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 8 Fedora Update System 2008-09-25 00:27:57 UTC 
subtitleeditor-0.22.3-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.
Note You need to log in before you can comment on or make changes to this bug.