Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yegor Yefremov <yegor_sub1@visionsystems.de>
To: buildroot@busybox.net
Subject: [Buildroot] Adding libev to buildroot
Date: Mon, 08 Aug 2011 16:55:25 +0200	[thread overview]
Message-ID: <4E3FF8DD.6070507@visionsystems.de> (raw)
In-Reply-To: <4E3FE254.7030004@roth-itk.de>

Hello Bernhard,

> I did a patch according to Linus' description.
> I hope this works and describes clearly the changes I did.
> 
> Just today I was recompiling a new root filesystem and libev works just like a charm!
> Keep your good work going, thank you!

Thank you for the patch. Some remarks below:

1. the patches should be sent "inline", so it is easier to comment on them
2. your' Signed-off-by is still missing (see other patches on the list)
3. don't use :=, use = instead
4. this is autotools based package, so following template should be used http://buildroot.uclibc.org/buildroot.html#autotools-tutorial

I've reworked your patch. So please execute "make clean" or remove all files installed by installation of libev and test it with reworked patch.

Yegor

---
 package/Config.in       |    1 +
 package/libev/Config.in |    6 ++++++
 package/libev/libev.mk  |   11 +++++++++++
 3 files changed, 18 insertions(+)

Index: b/package/Config.in
===================================================================
--- a/package/Config.in
+++ b/package/Config.in
@@ -341,6 +341,7 @@
 source "package/libdaemon/Config.in"
 source "package/libelf/Config.in"
 source "package/libevent/Config.in"
+source "package/libev/Config.in"
 source "package/libffi/Config.in"
 source "package/libglib2/Config.in"
 source "package/libnspr/Config.in"
Index: b/package/libev/Config.in
===================================================================
--- /dev/null
+++ b/package/libev/Config.in
@@ -0,0 +1,6 @@
+config BR2_PACKAGE_LIBEV
+	bool "libev"
+	help
+	  Userspace library for handling asynchronous notifications
+
+	  http://software.schmorp.de/pkg/libev.html
Index: b/package/libev/libev.mk
===================================================================
--- /dev/null
+++ b/package/libev/libev.mk
@@ -0,0 +1,11 @@
+#############################################################
+#
+# libev
+#
+#############################################################
+LIBEV_VERSION = 4.04
+LIBEV_SOURCE = libev-$(LIBEV_VERSION).tar.gz
+LIBEV_SITE = http://dist.schmorp.de/libev/
+LIBEV_INSTALL_STAGING = YES
+
+$(eval $(call AUTOTARGETS,package,libev))

  reply	other threads:[~2011-08-08 14:55 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-07 11:10 [Buildroot] Adding libev to buildroot Bernhard Roth
2011-08-07 21:04 ` Yegor Yefremov
     [not found]   ` <4E3F79CF.40608@roth-itk.de>
     [not found]     ` <CAGm1_ktqDGmoRtbWuN8uDM3qiL00K+kS-VbyVcDV3p9sqav9Jg@mail.gmail.com>
2011-08-08 13:19       ` Bernhard Roth
2011-08-08 14:55         ` Yegor Yefremov [this message]
2011-09-10 15:13           ` Peter Korsgaard
2011-09-10 21:27             ` Yegor Yefremov

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=4E3FF8DD.6070507@visionsystems.de \
    --to=yegor_sub1@visionsystems.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox