Bug 1318668 - new rpm requires: are they expected?
Summary: new rpm requires: are they expected?
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Software Collections
Classification: Red Hat
Component: curl
Version: httpd24
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: alpha
: 2.2
Assignee: Kamil Dudka
QA Contact: BaseOS QE - Apps
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-03-17 13:27 UTC by Ondřej Pták
Modified: 2016-03-21 12:40 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-03-21 12:40:17 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Ondřej Pták 2016-03-17 13:27:04 UTC
Description of problem:
There are several new rpm requires in updated httpd, please review them if all are expected. 

httpd24-curl-7.47.1-1.1.el6 [rhel-6]
 librt.so.1()(64bit)
 librt.so.1(GLIBC_2.2.5)(64bit)

httpd24-libcurl-7.47.1-1.1 [rhel-6+7]
 libidn.so.11()(64bit)
 libidn.so.11(LIBIDN_1.0)(64bit)
 libldap-2.4.so.2()(64bit)
 librt.so.1()(64bit) [rhel-6]
 librt.so.1(GLIBC_2.2.5)(64bit)) [rhel-6]
 libssh2(x86-64) >= 1.4.2
 libssh2.so.1()(64bit)

httpd24-libcurl-devel-7.47.1-1.1 [rhel-6+7]
 /opt/rh/httpd24/root/usr/share/aclocal

Comment 1 Kamil Dudka 2016-03-18 08:20:28 UTC
httpd24-curl has been added to the collection.  These dependencies are expected.

Comment 2 Ondřej Pták 2016-03-21 09:41:46 UTC
Thanks for feedback. So all of them should be added to whitelist?
This one looks a little bit suspicious to me: /opt/rh/httpd24/root/usr/share/aclocal

Comment 3 Kamil Dudka 2016-03-21 09:58:46 UTC
(In reply to Ondřej Pták from comment #2)
> Thanks for feedback. So all of them should be added to whitelist?

New packages come with new dependencies.  If you need to put them to some white list, then so be it.

> This one looks a little bit suspicious to me:
> /opt/rh/httpd24/root/usr/share/aclocal

It is a dependency of libcurl-devel to make sure that the directory is installed before installing libcurl.m4 in there.

Comment 4 Ondřej Pták 2016-03-21 12:40:17 UTC
Thanks for clarification, Kamil. I updated test to expect all these requires.


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