All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@uclibc.org>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] Add package fribidi
Date: Tue, 05 Jul 2011 11:56:59 +0200	[thread overview]
Message-ID: <87aactvyms.fsf@macbook.be.48ers.dk> (raw)
In-Reply-To: <CA+pfegEo9-Q9y8TQxwnbhASem=a1uU5=YnuvY=MbZ3rO1-ycPw@mail.gmail.com> (Murat Demirten's message of "Tue, 5 Jul 2011 12:42:05 +0300")

>>>>> "Murat" == Murat Demirten <mdemirten@yh.com.tr> writes:

 Murat> Hi Peter,

 Murat> There is no information about toolchain or localization
 Murat> requirements. But I just look at the source code and it seems
 Murat> that? this library can be compiled in all the cases, there is no
 Murat> special requirements.

OK, good - I didn't actually check, just expected so because of the
unicode handling.

 Murat> We need a custom install step because we don't want to see other
 Murat> than the library itself on the target.

 Murat> But in some cases, maybe fribidi and fribidi-config binary
 Murat> required by someone else on target. So, keeping these binaries
 Murat> in target usr/bin and giving the possibility to remove with
 Murat> standard postbuild script mechanism will be fine (Is there any
 Murat> hints to not install header files on the target?).

 Murat> What do you think about? Is there any policy for the packages
 Murat> which provides useful libraries and some binaries rarely need on
 Murat> target? Do we have to put all the binaries in target too?
 Murat> Sometimes these types of binaries not prepared for running on
 Murat> the target (like navit->maptool). Putting this binaries only to
 Murat> staging area keeps the target clean but I see that, there must
 Murat> be a policy, we can't put custom binary install logic for every
 Murat> single package.

Buildroot already cleans up header files and so on if not wanted on a
global level (see target-finalize).

For optional binaries and similar, it is typically handled by a sub
option in the package you can enable if you want them.

-- 
Bye, Peter Korsgaard

      reply	other threads:[~2011-07-05  9:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-13  9:40 [Buildroot] [PATCH] Add package fribidi Murat Demirten
2011-07-05  9:03 ` Peter Korsgaard
2011-07-05  9:42   ` Murat Demirten
2011-07-05  9:56     ` Peter Korsgaard [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=87aactvyms.fsf@macbook.be.48ers.dk \
    --to=jacmet@uclibc.org \
    --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.