Re: [PATCH 37/58] gnu-config-native: add dependency on perl-native

[Saul Wold <sgw@linux.intel.com>]

On 05/10/2011 07:20 AM, Cui, Dexuan wrote:
> Richard Purdie wrote:
>> On Tue, 2011-05-10 at 22:02 +0800, Cui, Dexuan wrote:
>>>> or we hardcode to /usr/bin/perl. Since the perl-native binary won't
>>>> be in the path for any of these, autodetecting and letting it
>>>> hardcode is probably fine. It has the advantage that if there are
>>>> any binary modules ever involved, they'll match the version of perl
>>>> they were built for regardless of whether perl-native is a
>>>> dependency or not. If someone adds a perl-native dependency to
>>>> autoconf-native, it will also still select the "correct" perl.
>>>>
>>>> Whilst doing this we need to keep bug #968 in mind and ensure that
>>>> if perl-native is used, all of perl-native is in the sysroot. That
>>>> bug is where the perl binary was installed, the library was not and
>>>> an error occurred. If it is in its own directory which is only
>>>> added to the PATH for users with the correct dependency, we avoid
>>>> this.
>>>>
>>>> As for bug #941, the bottom line is that however autoconf-native
>>>> selects its perl version, the strings encoded in gnu-config should
>>>> really match those in the other autoconf-native scripts.
>>> I saw a commit that was once suspended was pushed into poky
>>> master yesterday:
>>> http://git.pokylinux.org/cgit/cgit.cgi/poky/commit/?id=605141a93443df042634b2219a8628a9004be023
>>> Actually it does fix(or at least workaround) bug #941 and bug #968.
>>>
>>> Looks there are much work to do if we install perl-native to its own
>>> sysroot. At present we can use the above commit as a workaround.
>>
>> Its merged as a workaround but I still think we need to clean this up
>> properly and we need to continue working on it.
> I actually meant the priority could be lowered since we have a workaround. :-)
> Surely, I'll continue to investigate it.
>
Just to be clear, just because we have a work around does not mean we 
can lower the priority, this is still a valid bug that needs a proper 
patch in a timely manner.

941 is still a major bug and requires a fix before the milestone 1 
release stabilization period (May 20), which is 10 days from now.

Sau!


> Thanks,
> -- Dexuan
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>