Bug 1094759 - taglib test suit is not run during build
Summary: taglib test suit is not run during build
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: taglib
Version: 20
Hardware: All
OS: Unspecified
unspecified
medium
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-05-06 12:34 UTC by Jakub Čajka
Modified: 2014-05-06 12:45 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-05-06 12:45:15 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch enables tests during build (1.55 KB, patch)
2014-05-06 12:34 UTC, Jakub Čajka
no flags Details | Diff

Description Jakub Čajka 2014-05-06 12:34:48 UTC
Created attachment 892851 [details]
Patch enables tests during build

Description of problem:
Build-in unit test are not run during package build %check phase.

How reproducible:
always

Steps to Reproduce:
Build package from srpm using koji or localy

Actual results:
Tests are not run during package build.

Expected results:
Tests are run during package build.

Additional info:
Tests should be run in the %check phase according to packaging guidelines. In attachment is tests enabling patch for f20 spec file.

Tests pass successfully in a koji scratch build(http://koji.fedoraproject.org/koji/taskinfo?taskID=6818034) for f20 primary architectures and build time isn't changed significantly(few seconds). 

Do you have any reasons to not enable them?  If not please enable them in all Fedora versions.

Thanks 

Jakub

Comment 1 Rex Dieter 2014-05-06 12:38:12 UTC
I don't recall why they're not enabled by default, but the easier way to do it, is to toggle
%bconf_without tests

at the top of taglib.spec

I'll do just that shortly.

Comment 2 Rex Dieter 2014-05-06 12:38:39 UTC
(minus my typo)

Comment 3 Rex Dieter 2014-05-06 12:45:15 UTC
http://pkgs.fedoraproject.org/cgit/taglib.git/commit/?id=659267308327772a7878ed37fcd9d439aed11c05

The change will filter down to stable branches when/if we do any future bugfix releases (I don't see any need to issue any updates just for this).


Note You need to log in before you can comment on or make changes to this bug.