[oe] [PATCH meta-webserver] apache2: correct header file location
Randy MacLeod
randy.macleod at windriver.com
Fri Mar 8 03:06:08 UTC 2013
On 13-03-07 03:05 PM, Tumati, Pradeep wrote:
> Randy,
>
> Just wanted to forward you this email since Mark Hatle is on vacation...
Thanks.
>
> See my comments below
>
>> -----Original Message-----
>> From: Paul Eggleton [mailto:paul.eggleton at linux.intel.com]
>> Sent: Thursday, March 07, 2013 7:51 AM
>> To: Hatle, Mark
>> Cc: Tumati, Pradeep; openembedded-devel at lists.openembedded.org
>> Subject: Re: [oe] [PATCH meta-webserver] apache2: correct header file
>> location
>>
>> On Tuesday 19 February 2013 21:09:34 Mark Hatle wrote:
>>> From: Pradeep Tumati <pradeep.tumati at windriver.com>
>>>
>>> Apache header files are being placed in ../usr/include/lib32-apache2
>>> under the sysroot where as lib32-modphp looks for
>>> ../usr/include/apache2 directory. This fix ensures that apache2 header
>>> files are placed in the ../usr/include/apache2/ directory.
>>>
>>> Signed-off-by: Pradeep Tumati <pradeep.tumati at windriver.com>
>>> Signed-off-by: Jeff Polk <jeff.polk at windriver.com>
>>> ---
>>> recipes-httpd/apache2/apache2_2.4.2.bb | 32
>>> ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16
>>> deletions(-)
>>>
>>> diff --git a/recipes-httpd/apache2/apache2_2.4.2.bb
>>> b/recipes-httpd/apache2/apache2_2.4.2.bb index af7840d..af8c4ca 100644
>>> --- a/recipes-httpd/apache2/apache2_2.4.2.bb
>>> +++ b/recipes-httpd/apache2/apache2_2.4.2.bb
>>> @@ -41,13 +41,13 @@ EXTRA_OECONF = "--enable-ssl \
>>> --enable-rewrite \
>>> --with-dbm=sdbm \
>>> --with-berkeley-db=no \
>>> - --localstatedir=/var/${PN} \
>>> + --localstatedir=/var/${BPN} \
>>> --with-gdbm=no \
>>> --with-ndbm=no \
>>> - --includedir=${includedir}/${PN} \
>>> - --datadir=${datadir}/${PN} \
>>> - --sysconfdir=${sysconfdir}/${PN} \
>>> - --libexecdir=${libdir}/${PN}/modules \
>>> + --includedir=${includedir}/${BPN} \
>>> + --datadir=${datadir}/${BPN} \
>>> + --sysconfdir=${sysconfdir}/${BPN} \
>>> + --libexecdir=${libdir}/${BPN}/modules \
>>> ap_cv_void_ptr_lt_long=no \
>>> --enable-mpms-shared \
>>> ac_cv_have_threadsafe_pollset=no"
>>> @@ -68,19 +68,19 @@ do_install_append() {
>>> -e 's,/usr/bin/,${bindir}/,g' \
>>> -e 's,/usr/lib,${libdir}/,g' \
>>> -e 's,/etc/,${sysconfdir}/,g' \
>>> - -e 's,/usr/,${prefix}/,g' > ${D}/${sysconfdir}/init.d/${PN}
>>> - chmod 755 ${D}/${sysconfdir}/init.d/${PN}
>>> + -e 's,/usr/,${prefix}/,g' > ${D}/${sysconfdir}/init.d/${BPN}
>>> + chmod 755 ${D}/${sysconfdir}/init.d/${BPN}
>>> # remove the goofy original files...
>>> - rm -rf ${D}/${sysconfdir}/${PN}/original
>>> + rm -rf ${D}/${sysconfdir}/${BPN}/original
>>> # Expat should be found in the staging area via DEPENDS...
>>> rm -f ${D}/${libdir}/libexpat.*
>>>
>>> - install -d ${D}${sysconfdir}/${PN}/conf.d
>>> - install -d ${D}${sysconfdir}/${PN}/modules.d
>>> + install -d ${D}${sysconfdir}/${BPN}/conf.d
>>> + install -d ${D}${sysconfdir}/${BPN}/modules.d
>>>
>>> # Ensure configuration file pulls in conf.d and modules.d
>>> - printf "\nIncludeOptional ${sysconfdir}/${PN}/conf.d/*.conf" >>
>>> ${D}/${sysconfdir}/${PN}/httpd.conf - printf "\nIncludeOptional
>>> ${sysconfdir}/${PN}/modules.d/*.conf\n\n" >>
>>> ${D}/${sysconfdir}/${PN}/httpd.conf + printf "\nIncludeOptional
>>> ${sysconfdir}/${BPN}/conf.d/*.conf" >>
>>> ${D}/${sysconfdir}/${BPN}/httpd.conf
>>> + printf "\nIncludeOptional
>> ${sysconfdir}/${BPN}/modules.d/*.conf\n\n"
>>> +>>
>>> ${D}/${sysconfdir}/${BPN}/httpd.conf }
>>>
>>> SYSROOT_PREPROCESS_FUNCS += "apache_sysroot_preprocess"
>>> @@ -88,12 +88,12 @@ SYSROOT_PREPROCESS_FUNCS +=
>> "apache_sysroot_preprocess"
>>> apache_sysroot_preprocess () {
>>> install -d ${SYSROOT_DESTDIR}${bindir_crossscripts}/
>>> install -m 755 ${D}${bindir}/apxs
>>> ${SYSROOT_DESTDIR}${bindir_crossscripts}/ - sed -i 's!my
>> $installbuilddir
>> =
>>> .*!my $installbuilddir = "${STAGING_DIR_HOST}/${datadir}/${PN}/build";!'
>>> ${SYSROOT_DESTDIR}${bindir_crossscripts}/apxs + sed -i 's!my
>>> $installbuilddir = .*!my $installbuilddir =
>>> "${STAGING_DIR_HOST}/${datadir}/${BPN}/build";!'
>>> ${SYSROOT_DESTDIR}${bindir_crossscripts}/apxs sed -i 's!my $libtool =
>>> .*!my $libtool = "${STAGING_BINDIR_CROSS}/${TARGET_PREFIX}libtool";!'
>>> ${SYSROOT_DESTDIR}${bindir_crossscripts}/apxs
>>>
>>> - sed -i 's!^APR_CONFIG = .*!APR_CONFIG =
>>> ${STAGING_BINDIR_CROSS}/apr-1-config!'
>>> ${SYSROOT_DESTDIR}${datadir}/${PN}/build/config_vars.mk - sed -i
>>> 's!^APU_CONFIG = .*!APU_CONFIG = ${STAGING_BINDIR_CROSS}/apu-1-
>> config!'
>>> ${SYSROOT_DESTDIR}${datadir}/${PN}/build/config_vars.mk - sed -i
>>> 's!^includedir = .*!includedir = ${STAGING_INCDIR}/apache2!'
>>> ${SYSROOT_DESTDIR}${datadir}/${PN}/build/config_vars.mk + sed -i
>>> 's!^APR_CONFIG = .*!APR_CONFIG = ${STAGING_BINDIR_CROSS}/apr-1-
>> config!'
>>> ${SYSROOT_DESTDIR}${datadir}/${BPN}/build/config_vars.mk + sed -i
>>> 's!^APU_CONFIG = .*!APU_CONFIG = ${STAGING_BINDIR_CROSS}/apu-1-
>> config!'
>>> ${SYSROOT_DESTDIR}${datadir}/${BPN}/build/config_vars.mk + sed -i
>>> 's!^includedir = .*!includedir = ${STAGING_INCDIR}/apache2!'
>>> ${SYSROOT_DESTDIR}${datadir}/${BPN}/build/config_vars.mk }
>>>
>>> #
>>
>> Like the last patch this is not actually against meta-webserver master, and
>> like the last patch I was about to rebase and apply it on top, and again I notice
>> that there are ${PN} references that haven't been changed but appear to
>> need to be - the ones I can see are in the FILES_ values e.g.
>>
>> FILES_${PN}-dbg += "${libdir}/${PN}/modules/.debug"
>>
>> To match up with the paths we're specifying in EXTRA_OECONF this would
>> need to be changed to:
>>
>> FILES_${PN}-dbg += "${libdir}/${BPN}/modules/.debug"
>>
>> Surely this sort of thing should be showing up in testing?
>
>
> This is Paul's response to the apache2 multilib defect that I have
> worked on earlier. Paul points out that a few more ${PN}s have to
> be changed to ${BPN}s and so I was wondering if you can guide us
> how to proceed from here.
We're a bit behind the upstream meta-openembedded/meta-webserver repo:
we have : apache2_2.4.2.bb
upstream has : apache2_2.4.3.bb
Not a big deal and we'll get caught up soon, I expect.
Anyway, we do in fact have this error in the meta-webserver layer's
apache2_2.4.2 recipe but there isn't a warning generated when
the multilib dbg rpm is packaged...Ah, that's because we have a
local bbappend that needs to be sent upstream. Song can you do that
that in the next day or so.
Pradeep, because we're a version behind upstream, it's best if
we adjust our patches before sending upstream.
Hope that helps,
// Randy
>
> Thanks,
> Pradeep
>
>>
>> Cheers,
>> Paul
>>
>> --
>>
>> Paul Eggleton
>> Intel Open Source Technology Centre
--
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350
More information about the Openembedded-devel
mailing list