[yocto] [meta-baryon][PATCH 2/2] proftpd: update package to 1.3.4b

[Paul Eggleton]

Hi Kevin,

On Monday 10 September 2012 14:50:10 Kevin Strasser wrote:
> The contrib directry now contains its own Makefile which is
> used during installation. It was required to pass DESTDIR through
> when it gets called from the base Makefile.
> 
> Signed-off-by: Kevin Strasser <kevin.strasser at linux.intel.com>
> ---
>  recipes-extended/proftpd/files/contrib.patch       |   38
> ++++++++++++++++++++ .../{proftpd_1.3.3c.bb => proftpd_1.3.4b.bb}       |  
>  7 ++--
>  2 files changed, 42 insertions(+), 3 deletions(-)
>  create mode 100644 recipes-extended/proftpd/files/contrib.patch
>  rename recipes-extended/proftpd/{proftpd_1.3.3c.bb => proftpd_1.3.4b.bb}
> (82%)
> 
> diff --git a/recipes-extended/proftpd/files/contrib.patch
> b/recipes-extended/proftpd/files/contrib.patch new file mode 100644
> index 0000000..76b9a5c
> --- /dev/null
> +++ b/recipes-extended/proftpd/files/contrib.patch
> @@ -0,0 +1,38 @@
> +Upstream-Status: Inappropriate [configuration]
> +
> +Signed-off-by: Kevin Strasser <kevin.strasser at linux.intel.com>
> +---
> + Makefile.in         |    2 +-
> + contrib/Makefile.in |    6 +++---
> + 2 files changed, 4 insertions(+), 4 deletions(-)
> +
> +diff --git a/Makefile.in b/Makefile.in
> +index 5b2e683..ee72fe1 100644
> +--- a/Makefile.in
> ++++ b/Makefile.in
> +@@ -120,7 +120,7 @@ install-modules: $(DESTDIR)$(libexecdir)
> $(DESTDIR)$(sysconfdir) + 	test -z "$(SHARED_MODULE_OBJS)" -a -z
> "$(SHARED_MODULE_DIRS)" -a -z "$(STATIC_MODULE_DIRS)" || (cd modules/ &&
> $(MAKE) install) +
> + install-utils: $(DESTDIR)$(sbindir) $(DESTDIR)$(bindir)
> +-	cd contrib/ && $(MAKE) install-utils
> ++	cd contrib/ && $(MAKE) DESTDIR=${DESTDIR} install-utils
> + 	$(INSTALL_BIN)  ftpcount $(DESTDIR)$(bindir)/ftpcount
> + 	$(INSTALL_BIN)  ftpdctl  $(DESTDIR)$(bindir)/ftpdctl
> + 	$(INSTALL_SBIN) ftpscrub $(DESTDIR)$(sbindir)/ftpscrub
> +diff --git a/contrib/Makefile.in b/contrib/Makefile.in
> +index 5bcc038..51d248c 100644
> +--- a/contrib/Makefile.in
> ++++ b/contrib/Makefile.in
> +@@ -18,6 +18,6 @@ Makefile: Makefile.in ../config.status
> + 	cd ../ && ./config.status
> +
> + install-utils:
> +-	$(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 ftpasswd
> $(DESTDIR)$(bindir)/ftpasswd +-	$(INSTALL) -o $(INSTALL_USER) -g
> $(INSTALL_GROUP) -m 0755 ftpmail $(DESTDIR)$(bindir)/ftpmail +-	$(INSTALL)
> -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 ftpquota
> $(DESTDIR)$(bindir)/ftpquota ++	$(INSTALL) -m 0755 ftpasswd
> $(DESTDIR)$(bindir)/ftpasswd
> ++	$(INSTALL) -m 0755 ftpmail $(DESTDIR)$(bindir)/ftpmail
> ++	$(INSTALL) -m 0755 ftpquota $(DESTDIR)$(bindir)/ftpquota
> +--
> +1.7.9.5
> +
> diff --git a/recipes-extended/proftpd/proftpd_1.3.3c.bb
> b/recipes-extended/proftpd/proftpd_1.3.4b.bb similarity index 82%
> rename from recipes-extended/proftpd/proftpd_1.3.3c.bb
> rename to recipes-extended/proftpd/proftpd_1.3.4b.bb
> index 8de80fc..b7e201b 100644
> --- a/recipes-extended/proftpd/proftpd_1.3.3c.bb
> +++ b/recipes-extended/proftpd/proftpd_1.3.4b.bb
> @@ -2,17 +2,18 @@ DESCRIPTION = "Secure ftp daemon"
>  SECTION = "console/network"
> 
>  LICENSE = "GPLv2+"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=fdedcde17b1ffd967d86c20fe0ac158a"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=fb0d1484d11915fa88a6a7702f1dc184"

A few points:

1) Re contrib.patch - surely it would be valid to pass DESTDIR through for 
everyone? If so we should be able to consider sending this patch upstream, in 
which case Upstream-Status should be Pending.

2) When LIC_FILES_CHKSUM changes for any reason we need to mention clearly why 
in the commit message (even if it is a mundane reason such as the copyright 
year changing).

3) Minor - s/directry/directory in the commit message.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre