Bug 1396331 - Review Request: elfio - C++ library for reading and generating ELF files
Summary: Review Request: elfio - C++ library for reading and generating ELF files
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Igor Gnatenko
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-11-18 04:18 UTC by Orion Poplawski
Modified: 2016-11-24 01:44 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-24 01:44:38 UTC
Type: Bug
Embargoed:
ignatenko: fedora-review+


Attachments (Terms of Use)

Description Orion Poplawski 2016-11-18 04:18:42 UTC
Spec URL: https://www.cora.nwra.com/~orion/fedora/elfio.spec
SRPM URL: https://www.cora.nwra.com/~orion/fedora/elfio-3.2-1.fc26.src.rpm
Description:
ELFIO is a small, header-only C++ library that provides a simple interface for
reading and generating files in ELF binary format.

It is used as a standalone library - it is not dependant on any other product
or project. Adhering to ISO C++, it compiles on a wide variety of
architectures and compilers.

While the library is easy to use, some basic knowledge of the ELF binary
format is required. Such Information can easily be found on the Web.

Fedora Account System Username: orion

Comment 1 Igor Gnatenko 2016-11-18 08:36:54 UTC
Some issues to fix, but in general looks good.

[!]: Sources can be downloaded from URI in Source: tag
     Note: Could not download Source0:
     http://downloads.sf.net/elfio/elfio-3.2.tar.gz
     See: http://fedoraproject.org/wiki/Packaging:Guidelines#Tags

elfio-devel.noarch: W: spelling-error %description -l en_US dependant -> dependent, defendant, depend ant
elfio-devel.noarch: E: zero-length /usr/share/doc/elfio-devel/ChangeLog
elfio-devel.noarch: E: zero-length /usr/share/doc/elfio-devel/NEWS
elfio.src: W: spelling-error %description -l en_US dependant -> dependent, defendant, depend ant

Missing BuildRequires: gcc-c++

make %{?_smp_mflags} -> %make_build

you could also run some "tests" in %check. Because elfdump is compiled anyway, just run it against, let's say %{_bindir}/make.

Comment 2 Orion Poplawski 2016-11-19 19:41:45 UTC
Thanks for the review.  Issues have been fixed.

* Sat Nov 19 2016 Orion Poplawski <orion.com> - 3.2-2
- Fix source url
- Add BR gcc-c++
- Add %%check
- Other cleanup

Comment 3 Gwyn Ciesla 2016-11-20 16:43:01 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/elfio

Comment 4 Orion Poplawski 2016-11-24 01:44:38 UTC
Checked in and built.  Thanks.


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