Patchwork New package liblog4c-localtime

login
register
mail settings
Submitter Alexander Varnin
Date Oct. 8, 2012, 6:56 a.m.
Message ID <1349679414-2967-1-git-send-email-fenixk19@mail.ru>
Download mbox | patch
Permalink /patch/189920/
State Superseded
Headers show

Comments

Alexander Varnin - Oct. 8, 2012, 6:56 a.m.
---
 package/Config.in                                |    1 +
 package/liblog4c-localtime/Config.in             |    9 +++++++++
 package/liblog4c-localtime/liblog4c-localtime.mk |   17 +++++++++++++++++
 3 files changed, 27 insertions(+), 0 deletions(-)
 create mode 100644 package/liblog4c-localtime/Config.in
 create mode 100644 package/liblog4c-localtime/liblog4c-localtime.mk
Arnout Vandecappelle - Oct. 9, 2012, 8:29 p.m.
[Alexander, your mail server refuses mail from me, so I'm sending my
comments only to the list; I already sent them last Friday, but
obviously you didn't see them.]

  [List: sorry for the double posting...]

On 04/10/12 11:08, Alexander Varnin wrote:

  Add a Signed-off-by line for yourself.  This is a short way for you to
assert that you are entitled to contribute the patch under buildroot's
GPL license.  See  http://kerneltrap.org/files/Jeremy/DCO.txt for more
details.

> ---
>   package/Config.in                                |    1 +
>   package/liblog4c-localtime/Config.in             |    9 +++++++++
>   package/liblog4c-localtime/liblog4c-localtime.mk |   11 +++++++++++
>   3 files changed, 21 insertions(+), 0 deletions(-)
>   create mode 100644 package/liblog4c-localtime/Config.in
>   create mode 100644 package/liblog4c-localtime/liblog4c-localtime.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 8a7c9c4..25752e5 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -387,6 +387,7 @@ source "package/lttng-libust/Config.in"
>   source "package/orc/Config.in"
>   source "package/poco/Config.in"
>   source "package/startup-notification/Config.in"
> +source "package/liblog4c-localtime/Config.in"
>   endmenu
>
>   menu "Text and terminal handling"
> diff --git a/package/liblog4c-localtime/Config.in b/package/liblog4c-localtime/Config.in
> new file mode 100644
> index 0000000..58797be
> --- /dev/null
> +++ b/package/liblog4c-localtime/Config.in
> @@ -0,0 +1,9 @@
> +config BR2_PACKAGE_LIBLOG4C
> +    bool "liblog4c-localtime"
> +    help
> +      Log4c is a library of C for flexible logging to files, syslog
> +      and other destinations.
> +      This version is with localtime patch, to make lib show times in
> +      local timezone.
> +
> +      https://github.com/rcmadruga/log4c-localtime

  Add the upstream URL as well: http://log4c.sourceforge.net/

> diff --git a/package/liblog4c-localtime/liblog4c-localtime.mk b/package/liblog4c-localtime/liblog4c-localtime.mk
> new file mode 100644
> index 0000000..f0599c3
> --- /dev/null
> +++ b/package/liblog4c-localtime/liblog4c-localtime.mk
> @@ -0,0 +1,11 @@
> + #############################################################
> + #
> + # liblog4c
> + #
> + #############################################################
> + LIBLOG4C_VERSION = 1.2.1
> + LIBLOG4C_SITE = git://github.com/rcmadruga/log4c-localtime.git

  There's also a tarball:
LIBLOG4C_LOCALTIME_VERSION = v1.0
LIBLOG4C_LOCALTIME_SITE    = https://github.com/rcmadruga/log4c-localtime/tarball/v$(LIBLOG4C_LOCALTIME_VERSION)

> + LIBLOG4C_INSTALL_STAGING = YES
> +
> + $(eval $(call AUTOTARGETS))
> +

  Spurious newline at the end.


  It looks like it requires expat, so you need "select BR2_PACKAGE_EXPAT"
to Config.in, and "LIBLOG4C_LOCALTIME_DEPENDENCIES = expat" in .mk.  You'll
probably also need --disable-expattest for cross-compilation.


  Regards,
  Arnout
Alexander Varnin - Oct. 11, 2012, 9:17 a.m.
10.10.2012 00:29, Arnout Vandecappelle пишет:
> + $(eval $(call AUTOTARGETS)) 
Will this line work on newer versions of buildroot?
Arnout Vandecappelle - Oct. 11, 2012, 9:20 a.m.
On 11/10/12 11:17, Alexander Varnin wrote:
> 10.10.2012 00:29, Arnout Vandecappelle пишет:
>> + $(eval $(call AUTOTARGETS))
> Will this line work on newer versions of buildroot?

  No it won't, but Thomas already gave that comment so I didn't repeat it.

  Regards,
  Arnout

Patch

diff --git a/package/Config.in b/package/Config.in
index 8a7c9c4..25752e5 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -387,6 +387,7 @@  source "package/lttng-libust/Config.in"
 source "package/orc/Config.in"
 source "package/poco/Config.in"
 source "package/startup-notification/Config.in"
+source "package/liblog4c-localtime/Config.in"
 endmenu
 
 menu "Text and terminal handling"
diff --git a/package/liblog4c-localtime/Config.in b/package/liblog4c-localtime/Config.in
new file mode 100644
index 0000000..68feab5
--- /dev/null
+++ b/package/liblog4c-localtime/Config.in
@@ -0,0 +1,9 @@ 
+config BR2_PACKAGE_LIBLOG4C_LOCALTIME
+	bool "liblog4c-localtime"
+	help
+	  Log4c is a library of C for flexible logging to files, syslog 
+	  and other destinations.
+	  This version is with localtime patch, to make lib show times in
+	  local timezone.
+
+	  https://github.com/rcmadruga/log4c-localtime
diff --git a/package/liblog4c-localtime/liblog4c-localtime.mk b/package/liblog4c-localtime/liblog4c-localtime.mk
new file mode 100644
index 0000000..9936759
--- /dev/null
+++ b/package/liblog4c-localtime/liblog4c-localtime.mk
@@ -0,0 +1,17 @@ 
+ #############################################################
+ #
+ # liblog4c
+ #
+ #############################################################
+ LIBLOG4C_LOCALTIME_VERSION = 6b92bb84dad6f3f1a7cb97035e45f87bce5fa158
+ LIBLOG4C_LOCALTIME_SITE = git://github.com/rcmadruga/log4c-localtime.git
+ LIBLOG4C_LOCALTIME_INSTALL_STAGING = YES 
+
+ define LIBLOG4C_LOCALTIME_HOOK
+   chmod +x $(@D)/configure
+ endef
+
+ LIBLOG4C_LOCALTIME_PRE_CONFIGURE_HOOKS += LIBLOG4C_LOCALTIME_HOOK
+
+ $(eval $(autotools-package))
+