Bug 1581831 - Review Request: libsmbios-cpp - Libsmbios C++ shared libraries
Summary: Review Request: libsmbios-cpp - Libsmbios C++ shared libraries
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: Package Review
Version: epel7
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Stephen John Smoogen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-05-23 17:38 UTC by Carl George
Modified: 2018-06-22 15:05 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2018-06-22 15:05:06 UTC
Type: ---
Embargoed:
smooge: fedora-review+


Attachments (Terms of Use)
Review done by user (7.88 KB, application/octet-stream)
2018-05-31 16:35 UTC, Stephen John Smoogen
smooge: review+
Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1562440 1 None None None 2021-12-10 15:53:48 UTC
Red Hat Bugzilla 1581722 0 high CLOSED libsmbios-devel: remove c++ pkgconfig file and headers 2021-03-01 07:36:34 UTC

Internal Links: 1562440 1581722

Description Carl George 2018-05-23 17:38:31 UTC
Spec URL: https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/libsmbios-cpp/epel-7-x86_64/00758209-libsmbios-cpp/libsmbios-cpp.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/carlwgeorge/libsmbios-cpp/epel-7-x86_64/00758209-libsmbios-cpp/libsmbios-cpp-2.3.3-1.el7.src.rpm
COPR: https://copr.fedorainfracloud.org/coprs/carlwgeorge/libsmbios-cpp/

Description:
Libsmbios is a library and utilities that can be used by client programs to get
information from standard BIOS tables, such as the SMBIOS table.

This package provides the C++-based libsmbios library, with a C++ interface.
It is not actively maintained, but provided for backwards compatibility.  New
programs should use the libsmbios C interface.

Fedora Account System Username: carlwgeorge

Justification for this package is explained in further detail in these EPEL meeting notes (from 18:03:43 to 18:27:48).

https://meetbot.fedoraproject.org/fedora-meeting/2018-05-16/epel.2018-05-16-18.00.log.html

Comment 1 Carl George 2018-05-23 17:39:27 UTC
This package is for EPEL7 only.

Comment 2 R P Herrold 2018-05-25 13:34:50 UTC
I get this

# -- rpmlint starts -- 
rpmlint version 1.5 Copyright (C) 1999-2007 Frederic Lepied, Mandriva
libsmbios-cpp.src:58: W: configure-without-libdir-spec
libsmbios-cpp.src:62: W: configure-without-libdir-spec
libsmbios-cpp.src:63: W: configure-without-libdir-spec
libsmbios-cpp.src: W: patch-not-applied Patch0: disable-unwanted-components.patch
./holding/libsmbios-cpp.spec:58: W: configure-without-libdir-spec
./holding/libsmbios-cpp.spec:62: W: configure-without-libdir-spec
./holding/libsmbios-cpp.spec:63: W: configure-without-libdir-spec
./holding/libsmbios-cpp.spec: W: patch-not-applied Patch0: disable-unwanted-components.patch
1 packages and 1 specfiles checked; 0 errors, 8 warnings.
# -- rpmlint ends --

Comment 3 Carl George 2018-05-25 13:45:42 UTC
> libsmbios-cpp.src:58: W: configure-without-libdir-spec
> libsmbios-cpp.src:62: W: configure-without-libdir-spec
> libsmbios-cpp.src:63: W: configure-without-libdir-spec

I think these are false positives based on searching for the string "./configure".  Lines 58, 62, and 63 all happen to contain that string, but aren't actually executing the configure script.  Besides, that section of %build was lifted directly from the RHEL spec file.

> libsmbios-cpp.src: W: patch-not-applied Patch0: disable-unwanted-components.patch

Another false positive.  Patch0 is applied via the %autosetup macro.  Fedora's rpmlint is aware of this and doesn't return this warning.

Comment 4 Stephen John Smoogen 2018-05-31 16:35:02 UTC
Created attachment 1446357 [details]
Review done by user

This covers the files needed to show that a review was done and that the items which fedora-review flagged were checked and confirmed to be ok for EL-7

Comment 5 Stephen John Smoogen 2018-05-31 16:44:28 UTC
After much searching and clicking.. I found the correct review button.

Comment 6 Gwyn Ciesla 2018-06-06 15:50:11 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/libsmbios-cpp

Comment 7 Fedora Update System 2018-06-06 17:52:31 UTC
libsmbios-cpp-2.3.3-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-b03a9eeb2f

Comment 8 Fedora Update System 2018-06-07 11:59:44 UTC
libsmbios-cpp-2.3.3-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-b03a9eeb2f

Comment 9 Fedora Update System 2018-06-22 15:05:06 UTC
libsmbios-cpp-2.3.3-1.el7 has been pushed to the Fedora EPEL 7 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.