Skip to content
This repository has been archived by the owner on Mar 19, 2021. It is now read-only.

Commit

Permalink
Fix to bugreport #840
Browse files Browse the repository at this point in the history
To allow end users to choose the MPM way of running php7 the package
should only require httpd (work also for RHEL related) and one php
engine.

It can recommend mod_php but this will not last long in the future.

Note:

SUSE's RPM supports the Recommends tag, but it was not available
upstream until very recently. As a result, packages with Recommends will
fail to build for OBS targets Fedora_18, RHEL_6 and CentOS_6, for
example. A workaround is to exclude the Recommends tag from non-SUSE
targets. Alternatively the Recommends tag could become a Requires tag so
that a recommended package (foo) gets installed in any case on systems
that do not support RPM Recommends.

Fixes #840: bareos-webui openSUSE RPM has hard require to apache2_mod-php (5 or 7)
  • Loading branch information
fbergkemper committed Aug 7, 2017
1 parent 399c35c commit c8521c0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions packaging/obs/bareos-webui.spec
Expand Up @@ -62,8 +62,8 @@ BuildRequires: mod_php_any
%define _apache_conf_dir /etc/apache2/conf.d/
%define daemon_user wwwrun
%define daemon_group www
Requires: apache
Requires: mod_php_any
Requires: apache
Recommends: mod_php_any
%else
#if 0#{?fedora} || 0#{?rhel_version} || 0#{?centos_version}
BuildRequires: httpd
Expand Down

0 comments on commit c8521c0

Please sign in to comment.