On Wed, 2012-03-21 at 22:45 +0800, Robert Yang wrote:
> Hi Richard,
>
> Please see my comments inline ...
>
> On 03/21/2012 08:36 PM, Richard Purdie wrote:
> > On Wed, 2012-03-21 at 16:38 +0800, Robert Yang wrote:
> >> Remove the do_configure_prepend which is used for removing
> >> ${S}/m4/lib-link.m4 to compile under libtool 2.2.2, but our libtool has
> >> been upgraded to 2.4.2, and:
> >>
> >> for the 3 recipes:
> >> gst-plugins-bad_0.10.22.bb
> >> gst-plugins-base_0.10.35.bb
> >> gst-plugins-good_0.10.30.bb
> >>
> >> They don't have m4/lib-link.m4 any more.
> >>
> >> For the gst-plugins-ugly_0.10.18.bb, though it has m4/lib-link.m4, our
> >> libtool version is now 2.4.2, so we can remove the do_configure_prepend
> >> from all of them.
> >>
> >> [YOCTO #2148]
> >>
> >> Signed-off-by: Robert Yang<liezhi.y...@windriver.com>
> >> ---
> >> .../gstreamer/gst-plugins-bad_0.10.22.bb | 7 +------
> >> .../gstreamer/gst-plugins-base_0.10.35.bb | 7 +------
> >> .../gstreamer/gst-plugins-good_0.10.30.bb | 7 +------
> >> .../gstreamer/gst-plugins-ugly_0.10.18.bb | 7 +------
> >> 4 files changed, 4 insertions(+), 24 deletions(-)
> >
> > Not so easy I'm afraid. We need to ensure our libtool is being used.
> > Even if the version matches, our libtool is patched to ensure the
> > sysroot and rpaths work correctly. I'm therefore a little nervous about
> > removing these at this point.
> >
>
> Yes, I agree with you, though I checked the log.do_configure and
> log.do_compile,
> it used our libtool. The safe way maybe just remove the
> do_configure_prepend from gst-plugins-bad, gst-plugins-base and
> gst-plugins-good since they don't have the ${S}/m4/lib-link.m4 any more.
>
> For gst-plugins-ugly, we can remove when it would not have
> ${S}/m4/lib-link.m4.
>
> I noticed this because there was error when build gst-plugins-base/bad/good:
>
> rm: cannot remove `/path/to/m4/lib-link.m4': No such file or directory.
>
> Here is an updated patch and pull request which only removes
> do_configure_prepend for gst-plugins-base/bad/good:
> (I'm not sure whether send a V2 is preferred or not for such case).
>
> git://git.pokylinux.org/poky-contrib robert/gst_v2
> http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/gst_v2
>
> Subject: [PATCH 1/1] gstreamer 0.10.x: remove do_configure_prepend
>
> Remove the do_configure_prepend which is used for removing
> ${S}/m4/lib-link.m4, but the 3 recipes don't have m4/lib-link.m4 any
> more:
> gst-plugins-bad_0.10.22.bb
> gst-plugins-base_0.10.35.bb
> gst-plugins-good_0.10.30.bb
For what its worth, I just looked at the tarballs for these and the .m4
files are still there in the two I checked. This patch is therefore not
right in its current form.
Cheers,
Richard
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
