Print

Print


Yes, I can fix this - I just misunderstood what configure.classic was doing.  I thought lack of fuse disabled XrdFfs.

There is an issue with the commit you just made regardless, which can be fixed with the following:

--- a/src/XrdPss/Makefile.am
+++ b/src/XrdPss/Makefile.am
@@ -15,7 +15,7 @@ xrootdpssinc_HEADERS = XrdPss.hh
 
 INCLUDES = -I$(top_srcdir)/src
 
-libXrPss_la_SOURCES = \
+libXrdPss_la_SOURCES = \
         XrdPss.cc  XrdPss.hh \
         XrdPssAio.cc \
         XrdPssConfig.cc

Brian

On Nov 29, 2010, at 1:21 PM, Andrew Hanushevsky wrote:

> Hi Brian,
> 
> Hmmm, looks like one needs to call AC_CHECK_FUNC in the right way in configure.ac. Do you know how to use these autotool config functions?
> 
> Andy
> 
> On Mon, 29 Nov 2010, Brian Bockelman wrote:
> 
>> 
>> Hm, that's not exactly how I implemented it; if fuse.h isn't available, the check fails and XrdFfs won't be compiled.  I can look into this.
>> 
>> However, I do think we're fast approaching the due date for pushing a release to the ROOT team.  Let's make sure autotools deficiencies don't delay pushing this out the door.
>> 
>> Brian
>> 
>> On Nov 29, 2010, at 12:52 PM, Andrew Hanushevsky wrote:
>> 
>>> Hi Lukasz,
>>> 
>>> Not at all. Where do you see the dependency? XrdPosix doesn't reference XrdFfs. XrdPss references both but even then, XrdFfs is not dependent on Fuse. The XrdFfs package compiles whether or not Fuse exists. It just builds a dummy fuse loadable if Fuse doesn't exist.
>>> 
>>> I will agree, that we should probably make --disable-fuse the default since it's not normally installed on almost all RH releases, but the include file is actually there, sigh. I suppose we can check whether the fuse library is also there.
>>> 
>>> Andy
>>> 
>>> On Mon, 29 Nov 2010, Lukasz Janyst wrote:
>>> 
>>>> Hi Andy,
>>>> 
>>>> this has made the posix stuff dependent on fuse... Is this what you intended?
>>>> 
>>>> Lukasz
>>>> 
>>>> On Sun, Nov 28, 2010 at 7:53 PM, Andrew Hanushevsky
>>>> <[log in to unmask]> wrote:
>>>>> This is an automated email from the git hooks/post-receive script. It was
>>>>> generated because a ref change was pushed to the repository containing
>>>>> the project "xrootd".
>>>>> 
>>>>> The branch, master has been updated
>>>>>      via  2f7c3128d6bcd3a9771fcf7ba088c32d3fd5e06a (commit)
>>>>>      via  d11f09d46ff94cc90bfd2bc8b831d30ed3fa6055 (commit)
>>>>>      via  7a32ac1158538c3fb4eaa47bb6ac2a4be7aaf68c (commit)
>>>>>      via  cb4e338c119eb13d35fdd16d0353349cbe0f6629 (commit)
>>>>>     from  68cef315596dc6e32c4c3df1120c6bd22449501d (commit)
>>>>> 
>>>>> Those revisions listed above that are new to this repository have
>>>>> not appeared on any other notification email; so we list those
>>>>> revisions in full, below.
>>>>> 
>>>>> - Log -----------------------------------------------------------------
>>>>> commit 2f7c3128d6bcd3a9771fcf7ba088c32d3fd5e06a
>>>>> Author: Andrew Hanushevsky <[log in to unmask]>
>>>>> Date:   Sun Nov 28 09:52:46 2010 -0800
>>>>> 
>>>>>   Fix autotools make to reflect changes in making XrdPss.
>>>>> 
>>>>> commit d11f09d46ff94cc90bfd2bc8b831d30ed3fa6055
>>>>> Author: Andrew Hanushevsky <[log in to unmask]>
>>>>> Date:   Sun Nov 28 09:46:19 2010 -0800
>>>>> 
>>>>>   Export the Ofs forwarding list to allow XrdPss to be more effecient and not redo
>>>>>   forwarded operations.
>>>>>   Change the XrdPss target to be libXrdPss.so in keeping with other plug-ins.
>>>>> 
>>>>> commit 7a32ac1158538c3fb4eaa47bb6ac2a4be7aaf68c
>>>>> Merge: cb4e338c119eb13d35fdd16d0353349cbe0f6629 68cef315596dc6e32c4c3df1120c6bd22449501d
>>>>> Author: Andrew Hanushevsky <[log in to unmask]>
>>>>> Date:   Sun Nov 28 09:12:36 2010 -0800
>>>>> 
>>>>>   Merge branch 'master' of /afs/slac/www/projects/scalla/repo/xrootd
>>>>> 
>>>>> commit cb4e338c119eb13d35fdd16d0353349cbe0f6629
>>>>> Author: Andrew Hanushevsky <[log in to unmask]>
>>>>> Date:   Sun Nov 28 09:10:57 2010 -0800
>>>>> 
>>>>>   Updated proxy server plug-in. Uses XrdFfs where applicable. Runs I/O in parallel.
>>>>> 
>>>>> -----------------------------------------------------------------------
>>>>> 
>>>>> Summary of changes:
>>>>> src/XrdOfs/XrdOfsConfig.cc |   20 ++-
>>>>> src/XrdPss/GNUmakefile     |   21 ++-
>>>>> src/XrdPss/Makefile.am     |   10 +-
>>>>> src/XrdPss/XrdPss.cc       |  219 +++++++++++++++++++++++----------
>>>>> src/XrdPss/XrdPss.hh       |   49 +++++---
>>>>> src/XrdPss/XrdPssAio.cc    |    4 -
>>>>> src/XrdPss/XrdPssConfig.cc |  295 +++++++++++++++++++++++++------------------
>>>>> 7 files changed, 391 insertions(+), 227 deletions(-)
>>>>> 
>>>>> 
>>>>> hooks/post-receive
>>>>> --
>>>>> xrootd
>>>>> 
>>>> 
>> 
>>