On 11/23/15 8:20 AM, Mark Hatle wrote:
> On 11/23/15 7:41 AM, Robert Yang wrote:
>> We can install 64 bit pkg to 32 bit pkg, so the previous checking is
>> incorrect, it should not only check mips64 or mips64el, the "mips"
>> should work, and 64bit ELF can also use mips as TUNE_ARCH, and there
>> are other mips varieties, so only check whether TUNE_ARCH starts with
>> mips or not.
>>
>> Signed-off-by: Robert Yang <[email protected]>
>> ---
>> meta/lib/oe/package_manager.py | 9 ++++-----
>> 1 file changed, 4 insertions(+), 5 deletions(-)
>>
>> diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
>> index 964fddc..5e7bb66 100644
>> --- a/meta/lib/oe/package_manager.py
>> +++ b/meta/lib/oe/package_manager.py
>> @@ -954,12 +954,11 @@ class RpmPM(PackageManager):
>> bb.fatal("Invalid RPM_PREFER_ELF_ARCH: %s, it should be one
>> of:\n"
>> "\t1: ELF32 wins\n"
>> "\t2: ELF64 wins\n"
>> - "\t4: ELF64 N32 wins (mips64 or mips64el only)" %
>> + "\t4: ELF64 N32 wins (mips only)" %
>
> N32 is only available on a MIPS64 platform. It requires MIPS64 instructions,
> which is why it is documented as such above.
>
>> prefer_color)
>> - if prefer_color == "4" and self.d.getVar("TUNE_ARCH", True) not
>> in \
>> - ['mips64', 'mips64el']:
>> - bb.fatal("RPM_PREFER_ELF_ARCH = \"4\" is for mips64 or
>> mips64el "
>> - "only.")
>> + if prefer_color == "4" and not \
>> + self.d.getVar("TUNE_ARCH", True).startswith('mips'):
>> + bb.fatal("RPM_PREFER_ELF_ARCH = \"4\" is for mips only.")
>
> The bug is that it's looking for TUNE_ARCH, it should be MACHINE_ARCH in this
> case. Since the overall MACHINE Is what defined if it can be used or not.
I just realized, it's not MACHINE_ARCH either. It's the canonical arch that is
derived from the MACHINE_ARCH -- which may not be available at this point.
So doing the sanity based on the TUNE_ARCH starting with mips might be the best
compromise. But the comment should remain that it's only applicable on MIPS64.
--Mark
> --Mark
>
>> self._invoke_smart('config --set
>> rpm-extra-macros._prefer_color=%s'
>> % prefer_color)
>>
>>
>
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core
