All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Khem Raj" <raj.khem@gmail.com>
To: "Andreas Müller" <schnitzeltony@gmail.com>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: [oe] [PATCH 1/5] fltk/CMake: Do not export executable 'fluid'
Date: Thu, 19 Aug 2021 07:12:33 -0700	[thread overview]
Message-ID: <CAMKF1srH-YoTsww_ZjNgY11U6CwG+cycvAMWmU8EZvbNAs+E4g@mail.gmail.com> (raw)
In-Reply-To: <20210819110248.3505349-1-schnitzeltony@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3662 bytes --]

On Thu, Aug 19, 2021 at 4:02 AM Andreas Müller <schnitzeltony@gmail.com>
wrote:

> It does not work for us: The executable 'fluid' is not installed in
> receipe-sysroot. So fltk cmake config contains an invalid reference.
> Consumers of fltk using cmake will fail during configure for this invalid
> reference with something like:
>
> | CMake Error at
> <...>/recipe-sysroot/usr/share/fltk/FLTK-Targets.cmake:129 (message):
> |   The imported target "fluid" references the file
> |
> |      "<...>/recipe-sysroot/usr/bin/fluid"
> |
> |   but this file does not exist.


Do we get it from native sysroot then ? Or is it not required at all for
cross compiles

>
>
> Signed-off-by: Andreas Müller <schnitzeltony@gmail.com>
> ---
>  meta-oe/recipes-support/fltk/fltk.bb          |  2 +
>  ...Lists.txt-Do-not-export-fluid-target.patch | 42 +++++++++++++++++++
>  2 files changed, 44 insertions(+)
>  create mode 100644
> meta-oe/recipes-support/fltk/fltk/0003-fluid-CMakeLists.txt-Do-not-export-fluid-target.patch
>
> diff --git a/meta-oe/recipes-support/fltk/fltk.bb
> b/meta-oe/recipes-support/fltk/fltk.bb
> index ca2b517d7..3c9ee5ee3 100644
> --- a/meta-oe/recipes-support/fltk/fltk.bb
> +++ b/meta-oe/recipes-support/fltk/fltk.bb
> @@ -5,6 +5,8 @@ DEPENDS = "alsa-lib zlib jpeg libpng libxext libxft"
>  inherit features_check binconfig lib_package gtk-icon-cache mime mime-xdg
>  REQUIRED_DISTRO_FEATURES = "x11"
>
> +SRC_URI +=
> "file://0003-fluid-CMakeLists.txt-Do-not-export-fluid-target.patch"
> +
>  EXTRA_OECMAKE = " \
>      -DOPTION_BUILD_SHARED_LIBS=ON \
>      -DOPTION_USE_THREADS=ON \
> diff --git
> a/meta-oe/recipes-support/fltk/fltk/0003-fluid-CMakeLists.txt-Do-not-export-fluid-target.patch
> b/meta-oe/recipes-support/fltk/fltk/0003-fluid-CMakeLists.txt-Do-not-export-fluid-target.patch
> new file mode 100644
> index 000000000..72bb48a32
> --- /dev/null
> +++
> b/meta-oe/recipes-support/fltk/fltk/0003-fluid-CMakeLists.txt-Do-not-export-fluid-target.patch
> @@ -0,0 +1,42 @@
> +From b6985abcb03fb09685da4044963e8f085035b208 Mon Sep 17 00:00:00 2001
> +From: =?UTF-8?q?Andreas=20M=C3=BCller?= <schnitzeltony@gmail.com>
> +Date: Thu, 19 Aug 2021 08:06:45 +0200
> +Subject: [PATCH] fluid/CMakeLists.txt: Do not export fluid target
> +MIME-Version: 1.0
> +Content-Type: text/plain; charset=UTF-8
> +Content-Transfer-Encoding: 8bit
> +
> +It does not work for us: The executable 'fluid' is not installed in
> +receipe-sysroot. So fltk cmake config contains an invalid reference.
> +Consumers of fltk using cmake will fail during configure for this invalid
> +reference with something like:
> +
> +| CMake Error at
> <...>/recipe-sysroot/usr/share/fltk/FLTK-Targets.cmake:129 (message):
> +|   The imported target "fluid" references the file
> +|
> +|      "<...>/recipe-sysroot/usr/bin/fluid"
> +|
> +|   but this file does not exist.
> +
> +Upstream-Status: Inappropriate [OE specific]
> +
> +Signed-off-by: Andreas Müller <schnitzeltony@gmail.com>
> +---
> + fluid/CMakeLists.txt | 1 -
> + 1 file changed, 1 deletion(-)
> +
> +diff --git a/fluid/CMakeLists.txt b/fluid/CMakeLists.txt
> +index 472c84c..f971121 100644
> +--- a/fluid/CMakeLists.txt
> ++++ b/fluid/CMakeLists.txt
> +@@ -86,7 +86,6 @@ if (APPLE AND (NOT OPTION_APPLE_X11))
> +   install (TARGETS fluid DESTINATION ${FLTK_BINDIR})
> + else()
> +   install (TARGETS fluid
> +-    EXPORT FLTK-Targets
> +     RUNTIME DESTINATION ${FLTK_BINDIR}
> +     LIBRARY DESTINATION ${FLTK_LIBDIR}
> +     ARCHIVE DESTINATION ${FLTK_LIBDIR}
> +--
> +2.31.1
> +
> --
> 2.31.1
>
>
> 
>
>

[-- Attachment #2: Type: text/html, Size: 4924 bytes --]

  parent reply	other threads:[~2021-08-19 14:12 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-19 11:02 [PATCH 1/5] fltk/CMake: Do not export executable 'fluid' Andreas Müller
2021-08-19 11:02 ` [PATCH 2/5] fltk: upgrade 1.3.6 -> 1.3.7 Andreas Müller
2021-08-19 11:02 ` [PATCH 3/5] network-manager-applet: upgrade 1.22.0 -> 1.24.0 Andreas Müller
2021-08-19 11:02 ` [PATCH 4/5] networkmanager: upgrade 1.32.4 -> 1.32.8 Andreas Müller
2021-08-19 11:02 ` [PATCH 5/5] udisks2: upgrade 2.9.2 -> 2.9.3 Andreas Müller
2021-08-19 14:12 ` Khem Raj [this message]
2021-08-19 17:28   ` [oe] [PATCH 1/5] fltk/CMake: Do not export executable 'fluid' Andreas Müller
2021-08-19 17:36     ` Khem Raj

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAMKF1srH-YoTsww_ZjNgY11U6CwG+cycvAMWmU8EZvbNAs+E4g@mail.gmail.com \
    --to=raj.khem@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=schnitzeltony@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.