diff mbox

Possible bug in src/Makefile.am

Message ID 51D8F96B.5040902@catalyst.net.nz (mailing list archive)
State New, archived
Headers show

Commit Message

Mark Kirkwood July 7, 2013, 5:15 a.m. UTC
I noticed when building with prefix=/usr/local that the install step 
produced an usr/local/sbin hierarchy *under* /usr/local (i.e 
/usr/local/usr/local/sbin) with ceph_disk and friends (i.e 
ceph_sbin_SCRIPTS) therein. I am guessing that these should actually be 
installed in /usr/local/sbin (i.e prefix/sbin).

The attached patch does this.

Regards

Mark

Comments

Mark Kirkwood July 8, 2013, 6:51 a.m. UTC | #1
It seems this has been noted previously (just...):

http://tracker.ceph.com/issues/5492

Blast, I was just a bit slow :-) , should have posted when I first 
noticed this a week or so ago!

Regards

Mark

On 07/07/13 17:15, Mark Kirkwood wrote:
> I noticed when building with prefix=/usr/local that the install step
> produced an usr/local/sbin hierarchy *under* /usr/local (i.e
> /usr/local/usr/local/sbin) with ceph_disk and friends (i.e
> ceph_sbin_SCRIPTS) therein. I am guessing that these should actually be
> installed in /usr/local/sbin (i.e prefix/sbin).
>
> The attached patch does this.
>
> Regards
>
> Mark

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/src/Makefile.am b/src/Makefile.am
index c1a7e80..21f30dd 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -28,7 +28,7 @@  bin_PROGRAMS =
 bin_DEBUGPROGRAMS =
 sbin_PROGRAMS =
 # like sbin_SCRIPTS but can be used to install to e.g. /usr/sbin
-ceph_sbindir = $(prefix)$(sbindir)
+ceph_sbindir = $(sbindir)
 ceph_sbin_SCRIPTS = \
 	ceph-disk \
 	ceph-disk-prepare \