Re: [OE-core] [PATCH v3 5/9] libgpg-error: Add ARC support
On Fri, 2018-09-21 at 17:26 +0300, Alexey Brodkin wrote: > Signed-off-by: Alexey Brodkin > Cc: Ross Burton > Cc: Werner Koch > --- > > Changes v2 -> v3: > * Use proper [recently upstreamed] fix for ARC Glibc toolchain > * Fix compilation for x86_64 due to renamed header > > Changes v1 -> v2: > > * Added upstream status Its still breaking on the autobuilder: https://autobuilder.yoctoproject.org/typhoon/#/builders/21/builds/72/st eps/7/logs/step1b I think its for i586 which needs fixing similarly to x86_64. I did pull the simpler less risky pieces in for 2.6 but this experience is making me nervous about the remaining patches which should perhaps wait until 2.7. Cheers, Richard ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc
Re: [PATCH v2] u-boot: Add mkenvimage tool
On Thu, 2018-11-15 at 15:00 +, Burton, Ross wrote: > On Thu, 15 Nov 2018 at 14:50, Marek Vasut wrote: > > I have to admit, the recipe name becomes a misnomer then. But > > renaming > > this recipe would be hard, since it's used all over the place. Any > > ideas? > > Call it u-boot-tools, potentially merge the other recipe tool, and > add > RPROVIDES for compatibility? You mean PROVIDES since we're talking PN namespace, not PACKAGES? Cheers, Richard ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc
Re: [PATCH v3] u-boot: Add mkenvimage tool
On Mon, 2018-11-19 at 13:04 +0300, Alexey Brodkin wrote: > This utility is used for creation of images containing > usable in run-time U-Boot environment. > > As of today this utility is added per-board like here [1] > for Intel Edison board. > > [1] > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb > > Given there're quite some U-Boot tools that we may want to add later > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > still for compatibility we provide "u-boot-mkimage" with help > of PROVIDES as well as proposed "u-boot-mkenvimage". > > Signed-off-by: Alexey Brodkin > Cc: Richard Purdie > Cc: Otavio Salvador > Cc: Martin Jansa > Cc: Ross Burton > Cc: Marek Vasut > --- I know others are talking about splitting the packaging and I'm ok with deferring that to a separate patch. This patch did break the build in oe-selftest as you renamed a recipe and didn't update conf/distro/include/maintainers.inc. Could you send a new version with that tweak please? (grep https://autobuilder.yoctoproject.org/typhoon/#/builders/28/builds/242/steps/7/logs/step2d for the recipe name to see the test that failed) Cheers, Richard ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc
Re: [PATCH v4] u-boot: Add mkenvimage tool
On Tue, 2018-11-20 at 23:21 +0300, Alexey Brodkin wrote:
> This utility is used for creation of images containing
> usable in run-time U-Boot environment.
>
> As of today this utility is added per-board like here [1]
> for Intel Edison board.
>
> [1]
> http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb
>
> Given there're quite some U-Boot tools that we may want to add later
> this recipe name switch from "u-boot-mkimage" to generic "u-boot-
> tools"
> still for compatibility we provide "u-boot-mkimage" with help
> of PROVIDES as well as proposed "u-boot-mkenvimage".
>
> Signed-off-by: Alexey Brodkin
> Cc: Richard Purdie
> Cc: Otavio Salvador
> Cc: Martin Jansa
> Cc: Ross Burton
> Cc: Marek Vasut
> ---
>
> I may "git grep u-boot-mkimage" in more places:
> 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot =
> "u-boot-mkimage-native"
> 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s u-
> boot-mkimage-native dtc-native" % depends
> 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s u-
> boot-mkimage-native" % depends
> 4. meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn-
> u-boot-mkimage = "Ubuntu=uboot-mkimage Debian=uboot-mkimage"
>
> But that's not clear for me if it's required to swap "u-boot-mkimage"
> to "u-boot-tools" see:
> 1. "u-boot-mkimage" is still a valid recipe as we export ot via
> PROVIDES
> (or I'm mixing package name with recipe name which very well
> might be
> the case)
> 2. I don't fully understand a mening of "DISTRO_PN_ALIAS".
> If that's a name of a package from some big Linux distro or how
> target
> utility is named in that distro. For example in Debian Sid
> there's
> a package named "u-boot-tools" which includes "mkimage",
> "mkenvimage",
> "fw_printenv", "fw_setenv" and many more while in
> "distro_alias.inc"
> we still have:
> >8
> DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage
> Debian=uboot-mkimage"
> >8
Sorry, the tests in meta-oe have exposed another issue:
https://8n1.org/14048/3f99
which means the PROVIDES isn't quite right. The reason is there is
magic in the recipe which extends it to the native and nativesdk cases
but its not quite working.
I think the easiest way to address this may be
PROVIDES = "${MLPREFIX}u-boot-mkimage ${MLPREFIX}u-boot-mkenvimage"
PROVIDES_class-native = "u-boot-mkimage-native u-boot-mkenvimage-
native"
since in the nativesdk case, MLPREFIX expands to nativesdk-. The second
class-native line may not be needed. You could check with:
bitbake u-boot-tools -e | grep ^PROVIDES=
and see if it computes the value without manually overriding it.
Cheers,
Richard
___
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
Re: [PATCH v4] u-boot: Add mkenvimage tool
On Tue, 2018-11-20 at 23:21 +0300, Alexey Brodkin wrote: > This utility is used for creation of images containing > usable in run-time U-Boot environment. > > As of today this utility is added per-board like here [1] > for Intel Edison board. > > [1] > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb > > Given there're quite some U-Boot tools that we may want to add later > this recipe name switch from "u-boot-mkimage" to generic "u-boot- > tools" > still for compatibility we provide "u-boot-mkimage" with help > of PROVIDES as well as proposed "u-boot-mkenvimage". > > Signed-off-by: Alexey Brodkin > Cc: Richard Purdie > Cc: Otavio Salvador > Cc: Martin Jansa > Cc: Ross Burton > Cc: Marek Vasut > --- > > I may "git grep u-boot-mkimage" in more places: > 1. meta/classes/image_types.bbclass:311:CONVERSION_DEPENDS_u-boot = > "u-boot-mkimage-native" > 2. meta/classes/kernel-fitimage.bbclass:7:depends = "%s u- > boot-mkimage-native dtc-native" % depends > 3. meta/classes/kernel-uimage.bbclass:6:depends = "%s u- > boot-mkimage-native" % depends > 4. meta/conf/distro/include/distro_alias.inc:357:DISTRO_PN_ALIAS_pn- > u-boot-mkimage = "Ubuntu=uboot-mkimage Debian=uboot-mkimage" > > But that's not clear for me if it's required to swap "u-boot-mkimage" > to "u-boot-tools" see: > 1. "u-boot-mkimage" is still a valid recipe as we export ot via > PROVIDES > (or I'm mixing package name with recipe name which very well > might be > the case) > 2. I don't fully understand a mening of "DISTRO_PN_ALIAS". > If that's a name of a package from some big Linux distro or how > target > utility is named in that distro. For example in Debian Sid > there's > a package named "u-boot-tools" which includes "mkimage", > "mkenvimage", > "fw_printenv", "fw_setenv" and many more while in > "distro_alias.inc" > we still have: > >8 > DISTRO_PN_ALIAS_pn-u-boot-mkimage = "Ubuntu=uboot-mkimage > Debian=uboot-mkimage" > >8 I meant to say, for cleanliness, all the references you mentioned should also be converted in this patch. Most will work with PROVIDES mapping but we should improve them. Cheers, Richard ___ linux-snps-arc mailing list linux-snps-arc@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-snps-arc
Re: [PATCH v6] u-boot: Add mkenvimage tool
On Thu, 2018-11-22 at 13:18 +, Alexey Brodkin wrote:
> Hi, Richard, Otavio,
>
> On Thu, 2018-11-22 at 11:28 +0300, Alexey Brodkin wrote:
> > This utility is used for creation of images containing
> > usable in run-time U-Boot environment.
> >
> > As of today this utility is added per-board like here [1]
> > for Intel Edison board.
> >
> > [1]
> > http://git.yoctoproject.org/cgit/cgit.cgi/meta-intel-edison/tree/meta-intel-edison-bsp/recipes-bsp/u-boot/u-boot-tools_2014.04.bb
> >
> > Given there're quite some U-Boot tools that we may want to add
> > later
> > this recipe name switch from "u-boot-mkimage" to generic "u-boot-
> > tools"
> > still for compatibility we provide "u-boot-mkimage" with help
> > of PROVIDES as well as proposed "u-boot-mkenvimage".
> >
> > Signed-off-by: Alexey Brodkin
> > Cc: Richard Purdie
> > Cc: Otavio Salvador
> > Cc: Martin Jansa
> > Cc: Ross Burton
> > Cc: Marek Vasut
> > ---
> >
> > Changes v5 -> v6:
> > * Really enable separate pakaging of mkimage and mkenvimage
> >
> > Changes v4 -> v5:
> > * Added backward compatibility with "u-boot-mkimage" recipe
> > * Improved handling of nativesdk
> > * Update all references from "u-boot-mkimage" to "u-boot-tools"
> >
> > Changes v3 -> v4:
> > * Updated maintainer info to make Yocto autobuilder happy again
> >
> > Changes v2 -> v3:
> > * Recipe renamed to "u-boot-tools" to better match its contents
> >(especially in the future as we add more tools there)
> > * Existing "u-boot-mkimage" and newly introduced "u-boot-
> > mkenvimage"
> >made avaialble via PROVIDES
> >
> > Changes v1 -> v2:
> > * Got rid of a separate recipe and instead added mkenvimage
> >to u-boot mkimage recipe
>
> So apparently something is still broken :(
> ->8---
> NOTE: Resolving any missing task queue dependencies
> ERROR: Nothing RPROVIDES 'u-boot-tools-native-mkimage-native' (but
> virtual:native:.../openembedded-core/meta/recipes-bsp/u-boot/u-boot-
> tools_2018.07.bb RDEPENDS on or otherwise requires it)
>
> NOTE: Runtime target 'u-boot-tools-native-mkimage-native' is
> unbuildable, removing...
> Missing or unbuildable dependency chain was: ['u-boot-tools-native-
> mkimage-native']
>
> ERROR: Required build target 'core-image-base' has no buildable
> providers.
> Missing or unbuildable dependency chain was: ['core-image-base',
> 'virtual/kernel', 'u-boot-tools-native', 'u-boot-tools-native-
> mkimage-native']
> ->8---
>
> Indeed "u-boot-tools-native-mkimage-native" looks quite odd.
>
> Any hints on how that could be addressed?
Just guessing but maybe
RDEPENDS_${PN}_class-native = ""
?
Cheers,
Richard
___
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
