All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vyacheslav Yurkov <uvv.mail@gmail.com>
To: Khem Raj <raj.khem@gmail.com>
Cc: openembeded-devel <openembedded-devel@lists.openembedded.org>
Subject: Re: [meta-oe][PATCH] libeigen: allow empty packages
Date: Tue, 2 Oct 2018 19:52:23 +0200	[thread overview]
Message-ID: <575d6aa9-91d2-b4e6-2f0c-0b7a1edfad39@gmail.com> (raw)
In-Reply-To: <CAMKF1srWp+SANX2zBC_H4tRZS_Jk7+2ZMXBgA4z==+h7KyjTsg@mail.gmail.com>

Hi Khem,
It's a build time failure. libeigen is header-only library.
All headers go into dev package, therefore the main package is empty. 
Dependency on the dev package doesn't work, AFAIK, so you have to allow 
empty package for build to succeed. Otherwise I get an error that empty 
packages are not allowed. I reproduced it when I built ceres-solver 
recipe (although I don't use the latest poky). Perhaps I'm missing 
something and it has been solved in the latest poky?

Vyacheslav

On 02/10/2018 17:14, Khem Raj wrote:
> Hi Vyacheslav
>
> On Tue, Oct 2, 2018 at 1:33 AM Vyacheslav Yurkov <uvv.mail@gmail.com> wrote:
>> When libeigen is used as a dependency (e.g. in ceres library),
>> the build is broken because empty packages are not constructed be default
>>
> can you explain a bit more on whats going on? it is a runtime failure
> or build time failure.
>
>> Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
>> ---
>>   meta-oe/recipes-support/libeigen/libeigen_3.3.4.bb | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/meta-oe/recipes-support/libeigen/libeigen_3.3.4.bb b/meta-oe/recipes-support/libeigen/libeigen_3.3.4.bb
>> index 31a42a9d6..a59320655 100644
>> --- a/meta-oe/recipes-support/libeigen/libeigen_3.3.4.bb
>> +++ b/meta-oe/recipes-support/libeigen/libeigen_3.3.4.bb
>> @@ -18,3 +18,4 @@ FILES_${PN}-dev = "${includedir} ${datadir}/eigen3/cmake ${datadir}/cmake/Module
>>   # ${PN} is empty so we need to tweak -dev and -dbg package dependencies
>>   RDEPENDS_${PN}-dev = ""
>>   RRECOMMENDS_${PN}-dbg = "${PN}-dev (= ${EXTENDPKGV})"
>> +ALLOW_EMPTY_${PN} = "1"
>> --
>> 2.19.0
>>
>> --
>> _______________________________________________



  reply	other threads:[~2018-10-02 17:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-02  8:32 [meta-oe][PATCH] libeigen: allow empty packages Vyacheslav Yurkov
2018-10-02 15:14 ` Khem Raj
2018-10-02 17:52   ` Vyacheslav Yurkov [this message]
2018-10-02 18:43     ` 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=575d6aa9-91d2-b4e6-2f0c-0b7a1edfad39@gmail.com \
    --to=uvv.mail@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=raj.khem@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.