Rudder

Matthieu, could you look to that one and also check for how long it does not work ?

Note: https://github.com/lfrancke/logstash-rpm/pull/8

It seems RedHat removed a Provides: statement a bit too quickly, see https://bugzilla.redhat.com/show_bug.cgi?id=1030634.

Even if jre7/java7 comes back, it seems we should not rely on it and use "java" instead

I have no mean with RPM to create as in Debian conditionnal dependencies, the best approach I have here is to depend on jre / java and ONLY for el6 depend directly on java-1.7.0-openjdk

(Note: there is no macro available to discriminate minor versions of el6 ... The best I got is rhel = 6 or something else, nothing like rhel = 6.5. Thus, I can only act based on major OS versions.

What I intend to do is:

%if 0%{?rhel}  == 6
Requires: java-1.7.0-openjdk       
%else
Requires: jre >= 1.6
%endif

Notes to clarify: Contrary to SLES, EL does not provide any OS version level as a RPM macro, as seen in the output of rpm --showrc:

# SLES 11 SP1:
rpm --showrc|grep 11
-14: suse_version    1110 <= This is a SLES 11 SP 1
# RHEL/CentOS 6.5
-14: centos    6
-14: centos_ver    6
-14: dist    .EL.6
-14: distribution    .EL.6
-14: rhel    6

As you can see, EL does not consider minor versions relevant, as packaging and versions are normally consistent on all of them. What we got here is what I felt to be an issue in both our packaging, that uses a temporary virtual package from EL6 and RedHat that savagely nerfed the virtual package on 6.5 without thinking of people using it.

• A future proof one, jre >= 1.7 for EL7 onwards
• A "best effort" one, java-1.7.0-openjdk for EL6 (the same package for all EL6, as packaging is consistent for packages on major versions
• A SLES / other RPMs one, jre >= 1.6, principally to match the JRE's provided by Oracle and NEVER the 1.5 ones provided by IBM on SuSE SLES.

I'm validating that one. The logic is OK, the package name seems ok based on what I found on the net.

And basically, without that correction, Rudder can't be installed on RHEL (any one).

So let's merge it.

Applied in changeset commit:e7f7db1b12ea617fd28247e95358ae11d8b3d0f1.