All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <tom_rini@mentor.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: cmake-native trouble making
Date: Wed, 08 Sep 2010 13:20:19 -0700	[thread overview]
Message-ID: <4C87F003.7060106@mentor.com> (raw)
In-Reply-To: <ly7hiwknoq.fsf@ensc-virt.intern.sigma-chemnitz.de>

Enrico Scholz wrote:
> Tom Rini <tom_rini@mentor.com> writes:
> 
>>>> | /usr/bin/ld: cannot find -ltinfo
>>> no, 'DEPENDS = "ncurses-native"'; BUT:
>>>
>>> * it will cause problems on (host)distributions which do not have libtinfo
>>>   (e.g. RHEL5) and have ncurses-native in their ASSUME_PROVIDED
>> Isn't the whole point of ASSUME_PROVIDED that user be aware?
> 
> Would be too arbitrary in this case because it forces people to check by
> trial & error whether ASSUME_PROVIDED are working.  A raw guideline for
> adding an entry to ASSUME_PROVIDED should be, whether a corresponding,
> recent -devel package has been installed on the host.

That seems totally backwards.  The guideline for adding an entry to 
ASSUME_PROVIDED is that you know better than the build system.  No one 
doing commits for native packages should have anything set, or at least 
confirming that having an empty set (aside from bitbake.conf anyhow...) 
works fine too as that's the common case.

> imo, packages should not have so strict requirements on the actual
> provider but check in their configure task what to use (this includes
> autodetection of libtinfo vs. libtermcap vs. libncurses).

Yes, and we're grappling with the fallout of a big switch, which 
includes fixing up (or upgrading) other packages to be happy with a 
given version.

> For cmake, problem above can by solved by an initial configuration
> script having
> 
> | SET (BUILD_CursesDialog OFF CACHE BOOL "" FORCE)
>

Great, lets do that.

> 
>>> * some hours ago I added a libtermcap.so compatibility symlink to
>>>   ncurses(-native) so that 5ea3047995421d99f7f3537cb8f9ae23f3185a9f
>>>   might not be needed anymore
>> Um, is there a reason this wasn't just done as a ln within the recipe?
> 
> It's a mix of personal preference, following Fedora's packaging of this
> file and the hope to use some more of the power of linker scripts (giving
> out a deprecated warning, add AS_NEEDED, linking against multiple files).
> There was no opposition when I wrote about linker scripts neither.
> 
> Perhaps I should have made it inline by 'echo INPUT(-ltinfo) > ...' but
> I did not saw this when I committed the patch.

Well, it wasn't at all clear from the commit message why anything more 
than a symlink was done.  So can you please add a comment and a commit 
message explaining what's going on here?  Thanks!

-- 
Tom Rini
Mentor Graphics Corporation



      reply	other threads:[~2010-09-08 20:21 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-08  8:07 cmake-native trouble making Jaap de Jong
2010-09-08 13:04 ` Enrico Scholz
2010-09-08 18:42   ` Tom Rini
2010-09-08 20:03     ` Enrico Scholz
2010-09-08 20:20       ` Tom Rini [this message]

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=4C87F003.7060106@mentor.com \
    --to=tom_rini@mentor.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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.