REVIEW: http://review.gluster.org/14837 (buildld: glusterfs.spec(.in) firewalld file doesn't use %{_prefix}) posted (#1) for review on release-3.8 by Kaleb KEITHLEY (kkeithle)

REVIEW: http://review.gluster.org/14837 (build: glusterfs.spec(.in) firewalld file doesn't use %{_prefix}) posted (#2) for review on release-3.8 by Kaleb KEITHLEY (kkeithle)

REVIEW: http://review.gluster.org/14837 (build: glusterfs.spec(.in) firewalld file doesn't use %{_prefix}) posted (#3) for review on release-3.8 by Niels de Vos (ndevos)

COMMIT: http://review.gluster.org/14837 committed in release-3.8 by Niels de Vos (ndevos) 
------
commit 244decb1cc6091d441a58bcc5f9f7f85291a2f09
Author: Kaleb S KEITHLEY <kkeithle>
Date:   Thu Jun 30 11:53:51 2016 -0400

    build: glusterfs.spec(.in) firewalld file doesn't use %{_prefix}
    
    .../firewalld/services/glusterfs.xml in %server files section
    does not use %{_prefix}
    
    Other firewalld files, e.g., in the firewalld.src.rpm file use %{_prefix}
    
    N.B. the other 'hardcoded' path, "/sbin/mount.glusterfs" in the
    %files fuse section must remain as is, there is no macro for
    mount utilities to use as there is for, e.g., the %{_sbindir}
    macro for /usr/sbin. This doesn't matter for either RHEL6, where
    /sbin and /usr/sbin are distinct directories, or for Fedora
    and RHEL7, where /sbin is a symlink to /usr/sbin. E.g. see the nfs-
    utils.src.rpm where /sbin/mount.nfs is also 'hardcoded'
    
    Backport of commit c4ddfa5b76690db66fb22518d12226e2bf9cb236:
    > Change-Id: I902f47e3c589526c774d0aceb6fc2815abf86b01
    > BUG: 1350793
    > Signed-off-by: Kaleb S KEITHLEY <kkeithle>
    > Reviewed-on: http://review.gluster.org/14823
    > Smoke: Gluster Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > Reviewed-by: Milind Changire <mchangir>
    > Reviewed-by: Niels de Vos <ndevos>
    
    Change-Id: I902f47e3c589526c774d0aceb6fc2815abf86b01
    BUG: 1351711
    Signed-off-by: Kaleb S KEITHLEY <kkeithle>
    Reviewed-on: http://review.gluster.org/14837
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Niels de Vos <ndevos>
    Smoke: Gluster Build System <jenkins.org>

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.8.1, please open a new bug report.

glusterfs-3.8.1 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.packaging/156
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user