Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 349673 - sys-devel/crossdev: unknown C library causes bogus LPKG install
Summary: sys-devel/crossdev: unknown C library causes bogus LPKG install
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Embedded Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-25 12:21 UTC by Dennis Schridde
Modified: 2011-08-19 03:41 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to fix definition of LPKG (crossdev-always-define-lpkg.patch,772 bytes, patch)
2010-12-27 08:44 UTC, Dennis Schridde
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Schridde 2010-12-25 12:21:05 UTC
# crossdev arm-none-symbianelf
-> Creates an entry for "cross-arm-none-symbianelf/[latest]" in /etc/portage/package.keywords, probably because it misdetects libc:
 * binutils:              binutils-[latest]
 * gcc:                   gcc-[latest]
 * headers:               linux-headers-[latest]
 * libc:                  -[latest]

Instead it should generate an error message earlier, or just leave out the package it does not know about. (But in the latter case it should also not install linux-headers...)

Reproducible: Always
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2010-12-25 16:23:10 UTC
That would be a quadruplet, right?

$ crossdev 
Usage: crossdev [options] --target TARGET
[...]
Target (-t) takes a tuple ARCH-VENDOR-OS-LIBC; see 'crossdev -t help'
$ crossdev -t help
Supported Architectures:
   - alpha                                     - arm / armeb
   - hppa (parisc)                             - ia64
   - i386 / i486 / i586 / i686 (x86)           - m68k
   - mips / mipsel / mips64 / mips64el
   - powerpc (ppc) / powerpc64 (ppc64)
   - sparc / sparc64                           - s390 / s390x
   - sh / sh[1-5] / sh64                       - x86_64 (amd64)
Supported C Libraries:
   - glibc (gnu)
   - klibc       [prob wont work]
   - newlib      [bare metal/no operating system]
   - uclibc      [not all arches are ported]
Special Targets:
   - avr      http://www.nongnu.org/avr-libc/
   - bfin     http://blackfin.uclinux.org/
   - h8300    http://h8300-hms.sourceforge.net/
   - mingw32  http://www.mingw.org/
   - msp430   http://mspgcc.sourceforge.net/
   - nios2    http://www.altera.com/products/ip/processors/nios2/ni2-index.html
   - xc16x    http://www.infineon.com/
   - ee / iop / dvp (ps2) [Playstation 2 targets]
   - ppu / spu (cell) [Cell/Playstation 3 targets]
Softfloat toolchains:
   Include 'softfloat' in the 'vendor' field
   e.g. armeb-softfloat-linux-uclibc  powerpc-booya_softfloat-linux-gnu
$ [blink]
Comment 2 Dennis Schridde 2010-12-25 17:21:08 UTC
In any case there is no error message shown.
Comment 3 SpanKY gentoo-dev 2010-12-26 22:21:24 UTC
i'm not going to encode logic for every possible tuple.  the "validness" changes based on version of tools.
Comment 4 Dennis Schridde 2010-12-27 08:22:44 UTC
In my case crossdev was unable to figure out the correct software to install, which resulted in junk being placed into package.keywords. crossdev should be able to detect this case.
Comment 5 Dennis Schridde 2010-12-27 08:44:53 UTC
Created attachment 258155 [details, diff]
Patch to fix definition of LPKG

Attached patch makes sure that LPKG is defined, at least to "[none]".

The line ": ${LPKG=[none]}" had good intentions, but should have been using ":=" instead of "=". I put the "[none]" to the other default definitions to make it more obvious.
Comment 7 Dennis Schridde 2011-08-18 20:45:17 UTC
Thanks for committing my patch!

Tiny critique: You got my name wrong in the signed-off-by: s/Schroeder/Schridde/. But it is correct in the author field, so n/m.
Comment 8 SpanKY gentoo-dev 2011-08-19 03:41:05 UTC
i dont recall hand editing things, so not sure what happened with the copy & paste.  oh well, cant go back now :P.