All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/4] package/cmake: revert the bump to 3.7
Date: Sun, 26 Feb 2017 09:59:38 +0100	[thread overview]
Message-ID: <20170226085938.GA4043@free.fr> (raw)
In-Reply-To: <1488098694.10882.1.camel@embedded.rocks>

J?rg, All,

On 2017-02-26 09:44 +0100, J?rg Krause spake thusly:
> On Sat, 2017-02-25 at 19:51 +0100, Yann E. MORIN wrote:
> > Hello All!
> > 
> > cmake 3.7 causes serious regressions in some cmake-based packages,
> > related to how RPATH is handled.
> 
> For an explanation what's going wrong please have a look at [1].
> [1] http://lists.busybox.net/pipermail/buildroot/2017-February/183579.html

Are you sure this is related? The code you point to in this mail should
not be executed in cross-compilation:

    if(NOT CMAKE_CROSSCOMPILING AND EXISTS "/etc/debian_version")
      set_property(GLOBAL PROPERTY FIND_LIBRARY_USE_LIB32_PATHS FALSE)
      set_property(GLOBAL PROPERTY FIND_LIBRARY_USE_LIB64_PATHS FALSE)
    endif()

So, if this code is indeed the culprit, then it eans that
CMAKE_CROSSCOMPILING is not set, and that in turn is probably the real
issue.

(But I only had one coffee since I woke up, so I may have missed
something...)

Regards,
Yann E. MORIN.

> > See for example:
> > 
> > ? - domoticz : http://autobuild.buildroot.org/results/fd0/fd0ba54c7ab
> > f973691b39a0ca1bb4e07d749593a/
> > ? - freerdp??: http://autobuild.buildroot.org/results/5d4/5d429d0e288
> > 754a541ee5d8be515454c5fccd28b/
> > ? - libcec???: http://autobuild.buildroot.org/results/3f3/3f3593bab77
> > 34dd274faf5b5690895e9424cbb89/
> > 
> > (and many others)
> 
> For example bctoolbox. Although the build errors are fixed, CMake does
> not find the optionally mbedTLS package if zlib is available as it
> links against the host zlib when using `check_symbol_exists()`.
> 
> > All causes the link to be attemped against host libraries, which is
> > definitely not appropriate...
> > 
> > Properly fixing this so close to the release is problematic; we'd
> > risk
> > having to hunt down packages one by one.
> 
> A fix which worked for me was to remove the line `set(CMAKE_SYSTEM_NAME
> Linux)` from the toolchainfile to be able to set the flag
> `FIND_LIBRARY_USE_LIB32_PATHS` property to `FALSE`. Of course, we also
> have to set the necessary flags which are set depending on
> `CMAKE_SYSTEM_NAME`. However, reverting CMake is probably much cleaner.
> 
> [1] http://lists.busybox.net/pipermail/buildroot/2017-February/183579.h
> tml
> [2] https://git.buildroot.net/buildroot/tree/support/misc/toolchainfile
> .cmake.in#n13

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2017-02-26  8:59 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-25 18:51 [Buildroot] [PATCH 0/4] package/cmake: revert the bump to 3.7 Yann E. MORIN
2017-02-25 18:51 ` [Buildroot] [PATCH 1/4] Revert "cmake: bump version to 3.7.2" Yann E. MORIN
2017-02-25 18:51 ` [Buildroot] [PATCH 2/4] Revert "cmake: adjust dependencies added for libuv" Yann E. MORIN
2017-02-25 18:51 ` [Buildroot] [PATCH 3/4] Revert "cmake: bump version to 3.7.1" Yann E. MORIN
2017-02-25 18:51 ` [Buildroot] [PATCH 4/4] Revert "cmake: bump version to 3.7.0" Yann E. MORIN
2017-02-25 20:44 ` [Buildroot] [PATCH 0/4] package/cmake: revert the bump to 3.7 Baruch Siach
2017-02-25 22:10   ` Yann E. MORIN
2017-02-26  4:23     ` Baruch Siach
2017-02-26  8:44 ` Jörg Krause
2017-02-26  8:59   ` Yann E. MORIN [this message]
2017-02-26  9:12     ` Jörg Krause
2017-02-26 16:32 ` Ben Boeckel
2017-02-26 16:45   ` Yann E. MORIN
2017-02-26 20:29     ` Ben Boeckel
2017-02-26 21:34     ` Jörg Krause
2017-02-26 21:41 ` Peter Korsgaard
2017-02-26 22:40   ` Yann E. MORIN

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=20170226085938.GA4043@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /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.