* [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
@ 2013-02-26 12:36 Ross Burton
2013-02-26 14:32 ` Otavio Salvador
2013-02-26 22:37 ` Martin Jansa
0 siblings, 2 replies; 12+ messages in thread
From: Ross Burton @ 2013-02-26 12:36 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb | 2 ++
1 file changed, 2 insertions(+)
diff --git a/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb b/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
index 8ca2f6a..ee3e20c 100644
--- a/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
+++ b/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
@@ -19,6 +19,8 @@ SRC_URI[sha256sum] = "4e1cf5468d488aaf320ddfd439e03cbda0b213550fd734026d23138e4b
SECTION = "x11/libs"
DEPENDS = "intltool-native xkbcomp-native glib-2.0 virtual/gettext"
+PACKAGE_ARCH = "all"
+
PR = "r0"
EXTRA_OECONF = "--with-xkb-rules-symlink=xorg --disable-runtime-deps"
--
1.7.10.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 12:36 [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH Ross Burton
@ 2013-02-26 14:32 ` Otavio Salvador
2013-02-26 17:42 ` Burton, Ross
2013-02-26 22:37 ` Martin Jansa
1 sibling, 1 reply; 12+ messages in thread
From: Otavio Salvador @ 2013-02-26 14:32 UTC (permalink / raw)
To: Ross Burton; +Cc: Patches and discussions about the oe-core layer
On Tue, Feb 26, 2013 at 9:36 AM, Ross Burton <ross.burton@intel.com> wrote:
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
> meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb b/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
> index 8ca2f6a..ee3e20c 100644
> --- a/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
> +++ b/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
> @@ -19,6 +19,8 @@ SRC_URI[sha256sum] = "4e1cf5468d488aaf320ddfd439e03cbda0b213550fd734026d23138e4b
> SECTION = "x11/libs"
> DEPENDS = "intltool-native xkbcomp-native glib-2.0 virtual/gettext"
>
> +PACKAGE_ARCH = "all"
> +
> PR = "r0"
>
> EXTRA_OECONF = "--with-xkb-rules-symlink=xorg --disable-runtime-deps"
Please inherit allarch
--
Otavio Salvador O.S. Systems
E-mail: otavio@ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 14:32 ` Otavio Salvador
@ 2013-02-26 17:42 ` Burton, Ross
2013-02-26 17:47 ` Otavio Salvador
0 siblings, 1 reply; 12+ messages in thread
From: Burton, Ross @ 2013-02-26 17:42 UTC (permalink / raw)
To: Otavio Salvador; +Cc: Patches and discussions about the oe-core layer
On 26 February 2013 14:32, Otavio Salvador <otavio@ossystems.com.br> wrote:
> Please inherit allarch
That doesn't play nicely with autotools:
| checking build system type... x86_64-pc-linux-gnu
| checking host system type... Invalid configuration
`allarch-poky-linux': machine `allarch-poky' not recognized
| configure: error: /bin/bash ./config.sub allarch-poky-linux failed
Considering a lot of allarch is designed to remove dependencies that
this package uses and thus will need to be restored, directly setting
PACKAGE_ARCH is the cleaner solution.
Ross
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 17:42 ` Burton, Ross
@ 2013-02-26 17:47 ` Otavio Salvador
2013-02-26 17:55 ` Burton, Ross
0 siblings, 1 reply; 12+ messages in thread
From: Otavio Salvador @ 2013-02-26 17:47 UTC (permalink / raw)
To: Burton, Ross; +Cc: Patches and discussions about the oe-core layer
On Tue, Feb 26, 2013 at 2:42 PM, Burton, Ross <ross.burton@intel.com> wrote:
> On 26 February 2013 14:32, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> Please inherit allarch
>
> That doesn't play nicely with autotools:
>
> | checking build system type... x86_64-pc-linux-gnu
> | checking host system type... Invalid configuration
> `allarch-poky-linux': machine `allarch-poky' not recognized
> | configure: error: /bin/bash ./config.sub allarch-poky-linux failed
>
> Considering a lot of allarch is designed to remove dependencies that
> this package uses and thus will need to be restored, directly setting
> PACKAGE_ARCH is the cleaner solution.
If it changes the build depending in target and host, in the end it is
not architecture independant code.
--
Otavio Salvador O.S. Systems
E-mail: otavio@ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 17:47 ` Otavio Salvador
@ 2013-02-26 17:55 ` Burton, Ross
2013-02-26 18:09 ` Otavio Salvador
2013-02-26 19:43 ` Richard Purdie
0 siblings, 2 replies; 12+ messages in thread
From: Burton, Ross @ 2013-02-26 17:55 UTC (permalink / raw)
To: Otavio Salvador; +Cc: Patches and discussions about the oe-core layer
On 26 February 2013 17:47, Otavio Salvador <otavio@ossystems.com.br> wrote:
> If it changes the build depending in target and host, in the end it is
> not architecture independant code.
I think this is just autotools being nosy and failing when it doesn't
know what "alllinux" is. The files don't change as they are literally
just installed.
Ross
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 17:55 ` Burton, Ross
@ 2013-02-26 18:09 ` Otavio Salvador
2013-02-26 19:43 ` Richard Purdie
1 sibling, 0 replies; 12+ messages in thread
From: Otavio Salvador @ 2013-02-26 18:09 UTC (permalink / raw)
To: Burton, Ross; +Cc: Patches and discussions about the oe-core layer
On Tue, Feb 26, 2013 at 2:55 PM, Burton, Ross <ross.burton@intel.com> wrote:
> On 26 February 2013 17:47, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> If it changes the build depending in target and host, in the end it is
>> not architecture independant code.
>
> I think this is just autotools being nosy and failing when it doesn't
> know what "alllinux" is. The files don't change as they are literally
> just installed.
allarch does more than PACKAGE_ARCH so I think it'd be good to use it.
Maybe it'd not be hard to disable this call in configure.ac? I don't
have a strong opinion about it but using PACKAGE_ARCH only seems like
a workaround for me.
--
Otavio Salvador O.S. Systems
E-mail: otavio@ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 17:55 ` Burton, Ross
2013-02-26 18:09 ` Otavio Salvador
@ 2013-02-26 19:43 ` Richard Purdie
2013-02-26 20:44 ` Burton, Ross
1 sibling, 1 reply; 12+ messages in thread
From: Richard Purdie @ 2013-02-26 19:43 UTC (permalink / raw)
To: Burton, Ross; +Cc: Patches, Otavio Salvador, about the oe-core layer
On Tue, 2013-02-26 at 17:55 +0000, Burton, Ross wrote:
> On 26 February 2013 17:47, Otavio Salvador <otavio@ossystems.com.br> wrote:
> > If it changes the build depending in target and host, in the end it is
> > not architecture independant code.
>
> I think this is just autotools being nosy and failing when it doesn't
> know what "alllinux" is. The files don't change as they are literally
> just installed.
Check exactly what is in configure.ac. You should be able to use allarch
and if you can't, it means the configure script is referencing the
compiler, even if it shouldn't be using it. Its designed such that these
accesses fail verbosely. It should be a simple fix if the recipe really
is just configuration.
Using the allarch class is infinitely preferable as it gets the sstate
bits right.
Cheers,
Richard
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 19:43 ` Richard Purdie
@ 2013-02-26 20:44 ` Burton, Ross
2013-02-26 20:52 ` Otavio Salvador
0 siblings, 1 reply; 12+ messages in thread
From: Burton, Ross @ 2013-02-26 20:44 UTC (permalink / raw)
To: Richard Purdie
Cc: Otavio Salvador, Patches and discussions about the oe-core layer
On 26 February 2013 19:43, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> Check exactly what is in configure.ac. You should be able to use allarch
> and if you can't, it means the configure script is referencing the
> compiler, even if it shouldn't be using it. Its designed such that these
> accesses fail verbosely. It should be a simple fix if the recipe really
> is just configuration.
| configure:2707: checking host system type
| configure:2716: error: /bin/bash ./config.sub allarch-poky-linux failed
The configure.ac ends up running AC_CANONICAL_HOST because it uses
different manpage sections for Solaris and everything else.
I think they should be using AC_CANONICAL_SYSTEM, but that calls
AC_CANONICAL_HOST anyway so that wouldn't achieve anything.
So, extend my patch, or intrusively patch out the documentation build
code in xkeyboard-config.
Ross
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 20:44 ` Burton, Ross
@ 2013-02-26 20:52 ` Otavio Salvador
2013-02-27 14:47 ` Burton, Ross
0 siblings, 1 reply; 12+ messages in thread
From: Otavio Salvador @ 2013-02-26 20:52 UTC (permalink / raw)
To: Burton, Ross; +Cc: Patches and discussions about the oe-core layer
On Tue, Feb 26, 2013 at 5:44 PM, Burton, Ross <ross.burton@intel.com> wrote:
> On 26 February 2013 19:43, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
>> Check exactly what is in configure.ac. You should be able to use allarch
>> and if you can't, it means the configure script is referencing the
>> compiler, even if it shouldn't be using it. Its designed such that these
>> accesses fail verbosely. It should be a simple fix if the recipe really
>> is just configuration.
>
> | configure:2707: checking host system type
> | configure:2716: error: /bin/bash ./config.sub allarch-poky-linux failed
>
> The configure.ac ends up running AC_CANONICAL_HOST because it uses
> different manpage sections for Solaris and everything else.
>
> I think they should be using AC_CANONICAL_SYSTEM, but that calls
> AC_CANONICAL_HOST anyway so that wouldn't achieve anything.
>
> So, extend my patch, or intrusively patch out the documentation build
> code in xkeyboard-config.
I'd say to drop the AC_CANONICAL_HOST call and make it fake Linux as a
fixed value.
--
Otavio Salvador O.S. Systems
E-mail: otavio@ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 20:52 ` Otavio Salvador
@ 2013-02-27 14:47 ` Burton, Ross
2013-03-01 0:10 ` Richard Purdie
0 siblings, 1 reply; 12+ messages in thread
From: Burton, Ross @ 2013-02-27 14:47 UTC (permalink / raw)
To: Otavio Salvador; +Cc: Patches and discussions about the oe-core layer
On 26 February 2013 20:52, Otavio Salvador <otavio@ossystems.com.br> wrote:
> I'd say to drop the AC_CANONICAL_HOST call and make it fake Linux as a
> fixed value.
Did that in xorg-macros as that's where the definition is, but
intltool's configure fragment links a program to set DATADIRNAME
(although that appears to be deprecated), so it's trying to find a C
compiler now.
How do I *properly* set allarch so sstate "works" but still allow a
compiler to run so it can identify distribution-specific values (that
have nothing to do with the architecture)?
Ross
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-27 14:47 ` Burton, Ross
@ 2013-03-01 0:10 ` Richard Purdie
0 siblings, 0 replies; 12+ messages in thread
From: Richard Purdie @ 2013-03-01 0:10 UTC (permalink / raw)
To: Burton, Ross; +Cc: Patches, Otavio Salvador, about the oe-core layer
On Wed, 2013-02-27 at 14:47 +0000, Burton, Ross wrote:
> On 26 February 2013 20:52, Otavio Salvador <otavio@ossystems.com.br> wrote:
> > I'd say to drop the AC_CANONICAL_HOST call and make it fake Linux as a
> > fixed value.
>
> Did that in xorg-macros as that's where the definition is, but
> intltool's configure fragment links a program to set DATADIRNAME
> (although that appears to be deprecated), so it's trying to find a C
> compiler now.
>
> How do I *properly* set allarch so sstate "works" but still allow a
> compiler to run so it can identify distribution-specific values (that
> have nothing to do with the architecture)?
Inherit the class, then set the compiler options to point to the native
compiler? I'm very nervous about doing that mind and would prefer to
"fix" the macros if we can.
Cheers,
Richard
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH
2013-02-26 12:36 [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH Ross Burton
2013-02-26 14:32 ` Otavio Salvador
@ 2013-02-26 22:37 ` Martin Jansa
1 sibling, 0 replies; 12+ messages in thread
From: Martin Jansa @ 2013-02-26 22:37 UTC (permalink / raw)
To: Ross Burton; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 1170 bytes --]
On Tue, Feb 26, 2013 at 12:36:56PM +0000, Ross Burton wrote:
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
> meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb b/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
> index 8ca2f6a..ee3e20c 100644
> --- a/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
> +++ b/meta/recipes-graphics/xorg-lib/xkeyboard-config_2.8.bb
> @@ -19,6 +19,8 @@ SRC_URI[sha256sum] = "4e1cf5468d488aaf320ddfd439e03cbda0b213550fd734026d23138e4b
> SECTION = "x11/libs"
> DEPENDS = "intltool-native xkbcomp-native glib-2.0 virtual/gettext"
>
> +PACKAGE_ARCH = "all"
Why not inherit allarch?
> +
> PR = "r0"
>
> EXTRA_OECONF = "--with-xkb-rules-symlink=xorg --disable-runtime-deps"
> --
> 1.7.10.4
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-03-01 0:27 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-26 12:36 [PATCH] xkeyboard-config: this is architecture-indepedent, so set PACKAGE_ARCH Ross Burton
2013-02-26 14:32 ` Otavio Salvador
2013-02-26 17:42 ` Burton, Ross
2013-02-26 17:47 ` Otavio Salvador
2013-02-26 17:55 ` Burton, Ross
2013-02-26 18:09 ` Otavio Salvador
2013-02-26 19:43 ` Richard Purdie
2013-02-26 20:44 ` Burton, Ross
2013-02-26 20:52 ` Otavio Salvador
2013-02-27 14:47 ` Burton, Ross
2013-03-01 0:10 ` Richard Purdie
2013-02-26 22:37 ` Martin Jansa
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox