Re: ceph 0.47 libs3 install problem

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 21, 2012 at 1:29 PM, Yehuda Sadeh <yehuda@xxxxxxxxxxx> wrote:
> On Mon, May 21, 2012 at 1:22 PM, Tommi Virtanen <tv@xxxxxxxxxxx> wrote:
>> On Mon, May 21, 2012 at 1:14 PM, Nick Bartos <nick@xxxxxxxxxxxxxxx> wrote:
>>> I'm trying to install ceph 0.47, but --prefix isn't getting passed to
>>> the bundled libs3.  I installed libs3 separately, but even when using
>>> --with-system-libs3 (as ./configure suggests) or
>>> --with-system-libs3=yes (just for the hell of it), it still installs
>>> libs3, and outside of the prefix it was given.
>>>
>>> Anyone know of a way to either get ceph's libs3 to be installed in
>>> --prefix, or to get it to use the system one instead?
>>
>> Hmm. I just noticed this: src/Makefile.am has
>>
>> SUBDIRS = ocf libs3
>> ...
>> if WITH_SYSTEM_LIBS3
>> rest_bench_LDADD += -ls3
>> else
>> rest_bench_LDADD += libs3/build/lib/libs3.a -lcurl -lxml2
>> rest_bench_CXXFLAGS += -I$(top_srcdir)/src/libs3/inc
>> SUBDIRS += libs3
>> endif
>>
>> I'm guessing the top SUBDIRS assignment should not have libs3 there,
>> and that's the source of your trouble. Yehuda, can you confirm &
>> commit a fix?
>
> It does look like a possible suspect. I'll verify that.
>

This is the culprit, however, it doesn't solve the case where we
actually need libs3 (when rest-bench is enabled). We don't really need
to make install anyway, as we link statically to that library.

I pushed two fixes, one to ceph.git
(3a2dc969fffe6ee6f3a51583444998aba6449b7a) that removes the
unconditional appending of libs3 to SUBDIRS, and the second one to
libs3.git (9dc3a9c683385abfe4ad92b7c6ff30719acc3c13) that overrides
the install rule.

Yehuda

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


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux