From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Korsgaard Date: Wed, 09 Apr 2014 00:13:46 +0200 Subject: [Buildroot] [PATCH 4/6] tzdump: new host package In-Reply-To: (Yann E. MORIN's message of "Mon, 7 Apr 2014 21:58:06 +0200") References: Message-ID: <87d2gra2qt.fsf@dell.be.48ers.dk> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net >>>>> "Yann" == Yann E MORIN writes: > From: Alexandre Belloni > tzdump takes timezone info in the installed tzdata format, and > outputs timezone info parseable by uClibc. > Signed-off-by: Alexandre Belloni > [yann.morin.1998 at free.fr: split zic header-install to its own cset] > Signed-off-by: "Yann E. MORIN" > --- > package/tzdump/tzdump.mk | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 package/tzdump/tzdump.mk > diff --git a/package/tzdump/tzdump.mk b/package/tzdump/tzdump.mk > new file mode 100644 > index 0000000..471a719 > --- /dev/null > +++ b/package/tzdump/tzdump.mk > @@ -0,0 +1,24 @@ > +################################################################################ > +# > +# tzdump > +# > +################################################################################ > + > +TZDUMP_VERSION = 65a10105564801094b18c3fcacf4dde4c44e4ab8 > +TZDUMP_SITE = $(call github,alexandrebelloni,tzdump,$(TZDUMP_VERSION)) > +TZDUMP_DEPENDENCIES = host-zic It doesn't look like this is needed. > + > +define HOST_TZDUMP_BUILD_CMDS > + (cd $(@D) ;\ > + $(HOSTCC) $(HOST_CFLAGS) $(HOST_LDFLAGS) -o tzdump tzdump.c ; \ > + ) We don't really need the subshell here. > +endef > + > +define HOST_TZDUMP_INSTALL_CMDS > + mkdir -p $(HOST_DIR)/usr/sbin > + install -D -m 755 $(@D)/tzdump $(HOST_DIR)/usr/sbin/tzdump install -D already creates the destination directory, so mkdir can be dropped. We normally use the INSTALL variable instead of explicit 'install'. Committed with these fixes, thanks. -- Bye, Peter Korsgaard