Bug 1534279 - Review Request: catch1 - A modern, C++-native, header-only, framework for unit-tests, TDD and BDD
Summary: Review Request: catch1 - A modern, C++-native, header-only, framework for uni...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Igor Gnatenko
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-01-14 19:27 UTC by Tom Hughes
Modified: 2018-01-15 22:39 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2018-01-15 22:39:21 UTC
Type: ---
Embargoed:
ignatenko: fedora-review+

Attachments (Terms of Use)

Description Tom Hughes 2018-01-14 19:27:48 UTC 
Spec URL: http://tomh.fedorapeople.org//catch1.spec
SRPM URL: http://tomh.fedorapeople.org//catch1-1.12.0-1.fc27.src.rpm

Description:
Catch stands for C++ Automated Test Cases in Headers and is a
multi-paradigm automated test framework for C++ and Objective-C (and,
maybe, C). It is implemented entirely in a set of header files, but
is packaged up as a single header for extra convenience.
Comment 1 Tom Hughes 2018-01-14 19:27:53 UTC 
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=24199891
Comment 2 Igor Gnatenko 2018-01-14 21:09:12 UTC 
This actually doesn't need review because catch is already in fedora.
Comment 3 Tom Hughes 2018-01-14 21:34:36 UTC 
Do you think we should make this conflict (as it does now) or should it install in /usr/include/catch1? and if it does should the main package move to /usr/include/catch2?
Comment 4 Igor Gnatenko 2018-01-14 21:53:06 UTC 
I think it should conflict, because people won't patch their packages given that upstream doesn't differentiate between 1.x and 2.x...
Comment 5 Gwyn Ciesla 2018-01-15 22:11:55 UTC 
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/catch1
Note You need to log in before you can comment on or make changes to this bug.