Actions
Bug #5811
closedThe rudder-agent SPEC file does not handle bundled Openssl or not conditions properly
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:
Description
We currently package OpenSSL only if the condtion "use_system_openssl" is true in the package SPEC file.
Problem is, the condition to do so is broken:
# Catch-all %define use_system_openssl true ## 1 - RHEL: Bundled for pre-el5 oses ## ### Pre el5 have reached the end of production phase, ### and are thus in Extended Life phase. ### See. https://access.redhat.com/support/policy/updates/errata/ ## %if 0%{?rhel} && 0%{?rhel} < 5 %define use_system_openssl false %else %define use_system_openssl true %endif ## 3 - SLES: Bundled for pre-sles11 oses ## ### SLES 11 OSes come with OpenSSL 0.9.8h, ### which is recent enough. ## %if 0%{?sles_version} && 0%{?sles_version} < 11 %define use_system_openssl false %else %define use_system_openssl true %endifWhat happens is, on RHEL3:
- The condition RHEL is evaluated: rhel 3 < rhel 5, use_system_openssl is set to false => Good !
- The condition SLES is evaluated: Not a SLES system, use_system_openssl is set to true => FFFF@{#@{#~!!!!!
We need to just rely on the catch-all condition ( use_system_openssl = true ) and add OS-specific exceptions.
Updated by Matthieu CERDA about 10 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Matthieu CERDA to Jonathan CLARKE
- % Done changed from 0 to 100
- Pull Request set to https://github.com/Normation/rudder-packages/pull/528
Updated by Matthieu CERDA about 10 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder-packages|commit:5c8e23eb85ad3eed68c32525a14c8326a28d55df.
Updated by Jonathan CLARKE about 10 years ago
Applied in changeset rudder-packages|commit:76d7a06ca19f3d1b549fabde34eee7187831ae4b.
Updated by Vincent MEMBRÉ about 10 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 2.11.5, which was released today (16/12/14)
- Announcement 2.11
- Changelog 2.11
- Download information: https://www.rudder-project.org/site/get-rudder/downloads/
Updated by Benoît PECCATTE almost 10 years ago
- Project changed from 34 to Rudder
- Category set to Packaging
Actions