All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] Correct tclConfig.sh in the sysroots
@ 2016-05-31  9:51 Peter Kjellerstedt
  2016-05-31  9:51 ` [PATCH 1/1] tcl: Only set BINCONFIG_GLOB for target builds Peter Kjellerstedt
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Kjellerstedt @ 2016-05-31  9:51 UTC (permalink / raw)
  To: openembedded-core

Restore the contents of tclConfig.sh in the sysroots so that
postgresql can build again.

//Peter

The following changes since commit fcc2c3c4b3ca08528722442c90acd27e89291405:

  yocto-bsps: Update to 4.1 to include musb fixes (2016-05-30 15:58:16 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib pkj/tcl
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=pkj/tcl

Peter Kjellerstedt (1):
  tcl: Only set BINCONFIG_GLOB for target builds

 meta/recipes-devtools/tcltk/tcl_8.6.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.1.0



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/1] tcl: Only set BINCONFIG_GLOB for target builds
  2016-05-31  9:51 [PATCH 0/1] Correct tclConfig.sh in the sysroots Peter Kjellerstedt
@ 2016-05-31  9:51 ` Peter Kjellerstedt
  2016-05-31 21:32   ` Andreas Müller
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Kjellerstedt @ 2016-05-31  9:51 UTC (permalink / raw)
  To: openembedded-core

The recent change of how ${bindir_crossscripts} is installed to the
sysroot had the unforeseen effect that tclConfig.sh in the sysroot
contained invalid paths, which caused postgresql to fail to build.
The change in the contents of tclConfig.sh was due to that it was
previously installed to the sysroot after binconfig had done its work,
and was thus unaffected by it.

This change makes sure the contents of tclConfig.sh is the same as
before, both in the sysroots and on target. This will make postgresql
build again.

This also changes the BINCONFIG_GLOB to only match tclConfig.sh.
Before it would also match itclConfig.sh, tclooConfig.sh and
tdbcConfig.sh, which were consequently installed to the sysroots.
However, that seems to have been accidentally introduced with the use
of binconfig (based on what is installed in do_install()).

Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
---
 meta/recipes-devtools/tcltk/tcl_8.6.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/tcltk/tcl_8.6.4.bb b/meta/recipes-devtools/tcltk/tcl_8.6.4.bb
index 61be81d..3a3fd83 100644
--- a/meta/recipes-devtools/tcltk/tcl_8.6.4.bb
+++ b/meta/recipes-devtools/tcltk/tcl_8.6.4.bb
@@ -93,7 +93,7 @@ do_install_ptest() {
 }
 
 # Fix some paths that might be used by Tcl extensions
-BINCONFIG_GLOB = "*Config.sh"
+BINCONFIG_GLOB_class-target = "tclConfig.sh"
 
 # Fix the path in sstate
 SSTATE_SCAN_FILES += "*Config.sh"
-- 
2.1.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] tcl: Only set BINCONFIG_GLOB for target builds
  2016-05-31  9:51 ` [PATCH 1/1] tcl: Only set BINCONFIG_GLOB for target builds Peter Kjellerstedt
@ 2016-05-31 21:32   ` Andreas Müller
  0 siblings, 0 replies; 3+ messages in thread
From: Andreas Müller @ 2016-05-31 21:32 UTC (permalink / raw)
  To: Peter Kjellerstedt; +Cc: Patches and discussions about the oe-core layer

On Tue, May 31, 2016 at 11:51 AM, Peter Kjellerstedt
<peter.kjellerstedt@axis.com> wrote:
> The recent change of how ${bindir_crossscripts} is installed to the
> sysroot had the unforeseen effect that tclConfig.sh in the sysroot
> contained invalid paths, which caused postgresql to fail to build.
> The change in the contents of tclConfig.sh was due to that it was
> previously installed to the sysroot after binconfig had done its work,
> and was thus unaffected by it.
>
> This change makes sure the contents of tclConfig.sh is the same as
> before, both in the sysroots and on target. This will make postgresql
> build again.
>
> This also changes the BINCONFIG_GLOB to only match tclConfig.sh.
> Before it would also match itclConfig.sh, tclooConfig.sh and
> tdbcConfig.sh, which were consequently installed to the sysroots.
> However, that seems to have been accidentally introduced with the use
> of binconfig (based on what is installed in do_install()).
>
> Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> ---
>  meta/recipes-devtools/tcltk/tcl_8.6.4.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/tcltk/tcl_8.6.4.bb b/meta/recipes-devtools/tcltk/tcl_8.6.4.bb
> index 61be81d..3a3fd83 100644
> --- a/meta/recipes-devtools/tcltk/tcl_8.6.4.bb
> +++ b/meta/recipes-devtools/tcltk/tcl_8.6.4.bb
> @@ -93,7 +93,7 @@ do_install_ptest() {
>  }
>
>  # Fix some paths that might be used by Tcl extensions
> -BINCONFIG_GLOB = "*Config.sh"
> +BINCONFIG_GLOB_class-target = "tclConfig.sh"
>
>  # Fix the path in sstate
>  SSTATE_SCAN_FILES += "*Config.sh"
> --
> 2.1.0
>
> --
For me tclConfig.sh still contains absolute paths and postgresql still
fails. I did ccleansstate for tcl and postgresql, applied this patch
and tried to build postgresql.

Andreas


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-05-31 21:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-31  9:51 [PATCH 0/1] Correct tclConfig.sh in the sysroots Peter Kjellerstedt
2016-05-31  9:51 ` [PATCH 1/1] tcl: Only set BINCONFIG_GLOB for target builds Peter Kjellerstedt
2016-05-31 21:32   ` Andreas Müller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.