Compile error in rgw/rgw_xml.h in 0.46

2012-05-09 Thread Guido Winkelmann
Compiling Ceph 0.46 fails at rgw/rgw_dencoder.cc with the following errors:

In file included from rgw/rgw_dencoder.cc:7:
rgw/rgw_acl_s3.h:9:19: error: expat.h: No such file or directory
In file included from rgw/rgw_acl_s3.h:11,
 from rgw/rgw_dencoder.cc:7:
rgw/rgw_xml.h:59: error: 'XML_Parser' does not name a type
make[3]: *** [ceph_dencoder-rgw_dencoder.o] Error 1
make[3]: Leaving directory `/root/ceph-0.46/src'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/root/ceph-0.46/src'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/root/ceph-0.46/src'
make: *** [all-recursive] Error 1

This is on CentOS 6.2.

I managed to get it to compile by installing expat-devel first. Maybe the 
configure script should check for the existence of the expat header files?

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


Re: Compile error in rgw/rgw_xml.h in 0.46

2012-05-10 Thread Tommi Virtanen
On Wed, May 9, 2012 at 6:46 AM, Guido Winkelmann
 wrote:
> Compiling Ceph 0.46 fails at rgw/rgw_dencoder.cc with the following errors:
>
> In file included from rgw/rgw_dencoder.cc:7:
> rgw/rgw_acl_s3.h:9:19: error: expat.h: No such file or directory
> In file included from rgw/rgw_acl_s3.h:11,
>                 from rgw/rgw_dencoder.cc:7:
> rgw/rgw_xml.h:59: error: 'XML_Parser' does not name a type
> make[3]: *** [ceph_dencoder-rgw_dencoder.o] Error 1
> make[3]: Leaving directory `/root/ceph-0.46/src'
> make[2]: *** [all-recursive] Error 1
> make[2]: Leaving directory `/root/ceph-0.46/src'
> make[1]: *** [all] Error 2
> make[1]: Leaving directory `/root/ceph-0.46/src'
> make: *** [all-recursive] Error 1
>
> This is on CentOS 6.2.
>
> I managed to get it to compile by installing expat-devel first. Maybe the
> configure script should check for the existence of the expat header files?

What commands did you run to get to that point?

configure.ac does have an explicit check that tries to use XML_Parse
from expat. It's a little bit intricate due to the nesting, but a
quick read says it should work just right. Getting the command you ran
and full output from your configure run would be helpful.

https://github.com/ceph/ceph/blob/master/configure.ac#L183
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Compile error in rgw/rgw_xml.h in 0.46

2012-05-10 Thread Yehuda Sadeh
Oops, missed posting it to the list (seeing Tommi's comment).

On Wed, May 9, 2012 at 11:04 AM, Yehuda Sadeh  wrote:
>
> On Wed, May 9, 2012 at 6:46 AM, Guido Winkelmann
>  wrote:
> > Compiling Ceph 0.46 fails at rgw/rgw_dencoder.cc with the following
> > errors:
> >
> > In file included from rgw/rgw_dencoder.cc:7:
> > rgw/rgw_acl_s3.h:9:19: error: expat.h: No such file or directory
> > In file included from rgw/rgw_acl_s3.h:11,
> >                 from rgw/rgw_dencoder.cc:7:
> > rgw/rgw_xml.h:59: error: 'XML_Parser' does not name a type
> > make[3]: *** [ceph_dencoder-rgw_dencoder.o] Error 1
> > make[3]: Leaving directory `/root/ceph-0.46/src'
> > make[2]: *** [all-recursive] Error 1
> > make[2]: Leaving directory `/root/ceph-0.46/src'
> > make[1]: *** [all] Error 2
> > make[1]: Leaving directory `/root/ceph-0.46/src'
> > make: *** [all-recursive] Error 1
> >
> > This is on CentOS 6.2.
> >
> > I managed to get it to compile by installing expat-devel first. Maybe
> > the
> > configure script should check for the existence of the expat header
> > files?
> >
> Actually, the config script checks for that, but only if rgw is
> enabled. We need to figure out what would be the best solution. I
> don't think expat should be a dependency when rgw is not enabled, so
> we should need to figure how to remove this dependency. It might be
> that the easiest solution would be to not compile all the rgw stuff
> into the dencoder when rgw is not enabled (which makes sense anyway).
> I opened bug #2390 to track this issue.
>
> Thanks,
> Yehuda
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html