[Bugs] [Bug 1351711] build: remove absolute paths from glusterfs spec file

bugzilla at redhat.com bugzilla at redhat.com
Thu Jul 7 10:57:15 UTC 2016


https://bugzilla.redhat.com/show_bug.cgi?id=1351711



--- Comment #4 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14837 committed in release-3.8 by Niels de
Vos (ndevos at redhat.com) 
------
commit 244decb1cc6091d441a58bcc5f9f7f85291a2f09
Author: Kaleb S KEITHLEY <kkeithle at redhat.com>
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 at redhat.com>
    > Reviewed-on: http://review.gluster.org/14823
    > Smoke: Gluster Build System <jenkins at build.gluster.org>
    > CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    > NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    > Reviewed-by: Milind Changire <mchangir at redhat.com>
    > Reviewed-by: Niels de Vos <ndevos at redhat.com>

    Change-Id: I902f47e3c589526c774d0aceb6fc2815abf86b01
    BUG: 1351711
    Signed-off-by: Kaleb S KEITHLEY <kkeithle at redhat.com>
    Reviewed-on: http://review.gluster.org/14837
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Niels de Vos <ndevos at redhat.com>
    Smoke: Gluster Build System <jenkins at build.gluster.org>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7OgVGbdqzW&a=cc_unsubscribe


More information about the Bugs mailing list