* [meta-oe][PATCH 0/1] proftpd update
@ 2021-04-19 7:52 Andrej Kozemcak
2021-04-19 7:52 ` [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release Andrej Kozemcak
2021-04-19 21:17 ` [oe] [meta-oe][PATCH 0/1] proftpd update Khem Raj
0 siblings, 2 replies; 6+ messages in thread
From: Andrej Kozemcak @ 2021-04-19 7:52 UTC (permalink / raw)
To: openembedded-devel; +Cc: Andrej Kozemcak
I would like to apply this update also to dunfell branch, if posible.
Because there is lot of bugs which is not a posible fix with the easy
way.
Andrej Kozemcak (1):
proftpd: Update to 1.3.7a release
.../proftpd/files/build_fixup.patch | 69 ++++++++-----------
.../proftpd/files/contrib.patch | 29 ++++----
.../{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} | 12 ++--
3 files changed, 47 insertions(+), 63 deletions(-)
rename meta-networking/recipes-daemons/proftpd/{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} (93%)
--
2.20.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release
2021-04-19 7:52 [meta-oe][PATCH 0/1] proftpd update Andrej Kozemcak
@ 2021-04-19 7:52 ` Andrej Kozemcak
2021-04-19 21:16 ` [oe] " Khem Raj
2021-04-19 21:17 ` [oe] [meta-oe][PATCH 0/1] proftpd update Khem Raj
1 sibling, 1 reply; 6+ messages in thread
From: Andrej Kozemcak @ 2021-04-19 7:52 UTC (permalink / raw)
To: openembedded-devel; +Cc: Andrej Kozemcak
Signed-off-by: Andrej Kozemcak <andrej.kozemcak@siemens.com>
---
.../proftpd/files/build_fixup.patch | 69 ++++++++-----------
.../proftpd/files/contrib.patch | 29 ++++----
.../{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} | 12 ++--
3 files changed, 47 insertions(+), 63 deletions(-)
rename meta-networking/recipes-daemons/proftpd/{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} (93%)
diff --git a/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch b/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
index 9816d4d22..d888dd826 100644
--- a/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
+++ b/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
@@ -9,10 +9,10 @@ redefine PR_RUN_DIR as ${localstatedir}/run
Signed-off-By: Armin Kuster <akuster808@gmail.com>
-Index: proftpd-1.3.6/Make.rules.in
-===================================================================
---- proftpd-1.3.6.orig/Make.rules.in
-+++ proftpd-1.3.6/Make.rules.in
+diff --git a/Make.rules.in b/Make.rules.in
+index a5028ac..ccd7807 100644
+--- a/Make.rules.in
++++ b/Make.rules.in
@@ -30,9 +30,9 @@ INSTALL=@INSTALL@
INSTALL_STRIP=@INSTALL_STRIP@
INSTALL_USER=@install_user@
@@ -26,11 +26,11 @@ Index: proftpd-1.3.6/Make.rules.in
RM=rm -f
SHELL=@CONFIG_SHELL@
-Index: proftpd-1.3.6/Makefile.in
-===================================================================
---- proftpd-1.3.6.orig/Makefile.in
-+++ proftpd-1.3.6/Makefile.in
-@@ -105,7 +105,6 @@ check: proftpd$(EXEEXT)
+diff --git a/Makefile.in b/Makefile.in
+index 9434cbd..13f2e16 100644
+--- a/Makefile.in
++++ b/Makefile.in
+@@ -123,7 +123,6 @@ check: proftpd$(EXEEXT)
$(DESTDIR)$(localedir) $(DESTDIR)$(includedir) $(DESTDIR)$(includedir)/proftpd $(DESTDIR)$(libdir) $(DESTDIR)$(pkgconfigdir) $(DESTDIR)$(libdir)/proftpd $(DESTDIR)$(libexecdir) $(DESTDIR)$(localstatedir) $(DESTDIR)$(sysconfdir) $(DESTDIR)$(bindir) $(DESTDIR)$(sbindir) $(DESTDIR)$(mandir) $(DESTDIR)$(mandir)/man1 $(DESTDIR)$(mandir)/man5 $(DESTDIR)$(mandir)/man8:
@if [ ! -d $@ ]; then \
mkdir -p $@; \
@@ -38,20 +38,20 @@ Index: proftpd-1.3.6/Makefile.in
chmod 0755 $@; \
fi
-@@ -115,7 +114,6 @@ install-proftpd: proftpd $(DESTDIR)$(inc
+@@ -133,7 +132,6 @@ install-proftpd: proftpd$(EXEEXT) $(DESTDIR)$(includedir) $(DESTDIR)$(localstate
rm -f $(DESTDIR)$(sbindir)/in.proftpd ; \
fi
- ln -s proftpd $(DESTDIR)$(sbindir)/in.proftpd
+ ln -s $(top_builddir)/proftpd $(DESTDIR)$(sbindir)/in.proftpd
- -chown -h $(INSTALL_USER):$(INSTALL_GROUP) $(DESTDIR)$(sbindir)/in.proftpd
install-libs: $(DESTDIR)$(libdir)/proftpd
cd lib/ && $(MAKE) install
-@@ -152,11 +150,11 @@ install-utils: $(DESTDIR)$(sbindir) $(DE
- $(INSTALL_SBIN) ftpshut $(DESTDIR)$(sbindir)/ftpshut
- $(INSTALL_BIN) ftptop $(DESTDIR)$(bindir)/ftptop
- $(INSTALL_BIN) ftpwho $(DESTDIR)$(bindir)/ftpwho
-- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 src/prxs $(DESTDIR)$(bindir)/prxs
-+ $(INSTALL) -m 0755 src/prxs $(DESTDIR)$(bindir)/prxs
+@@ -170,11 +168,11 @@ install-utils: $(DESTDIR)$(sbindir) $(DESTDIR)$(bindir)
+ $(INSTALL_SBIN) $(top_builddir)/ftpshut $(DESTDIR)$(sbindir)/ftpshut
+ $(INSTALL_BIN) $(top_builddir)/ftptop $(DESTDIR)$(bindir)/ftptop
+ $(INSTALL_BIN) $(top_builddir)/ftpwho $(DESTDIR)$(bindir)/ftpwho
+- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(top_builddir)/src/prxs $(DESTDIR)$(bindir)/prxs
++ $(INSTALL) -m 0755 $(top_builddir)/src/prxs $(DESTDIR)$(bindir)/prxs
install-conf: $(DESTDIR)$(sysconfdir)
if [ ! -f $(DESTDIR)$(sysconfdir)/proftpd.conf ] ; then \
@@ -60,11 +60,11 @@ Index: proftpd-1.3.6/Makefile.in
$(top_srcdir)/sample-configurations/basic.conf \
$(DESTDIR)$(sysconfdir)/proftpd.conf ; \
fi
-Index: proftpd-1.3.6/configure
-===================================================================
---- proftpd-1.3.6.orig/configure
-+++ proftpd-1.3.6/configure
-@@ -41777,7 +41777,7 @@ _ACEOF
+diff --git a/configure b/configure
+index 0a51670..69fa130 100755
+--- a/configure
++++ b/configure
+@@ -25116,7 +25116,7 @@ _ACEOF
cat >>confdefs.h <<_ACEOF
@@ -73,7 +73,7 @@ Index: proftpd-1.3.6/configure
_ACEOF
-@@ -41787,7 +41787,7 @@ _ACEOF
+@@ -25126,7 +25126,7 @@ _ACEOF
cat >>confdefs.h <<_ACEOF
@@ -82,11 +82,11 @@ Index: proftpd-1.3.6/configure
_ACEOF
-Index: proftpd-1.3.6/configure.in
-===================================================================
---- proftpd-1.3.6.orig/configure.in
-+++ proftpd-1.3.6/configure.in
-@@ -3833,8 +3833,8 @@ locale_dir="`eval echo ${locale_dir}`"
+diff --git a/configure.in b/configure.in
+index 50561de..93cad3f 100644
+--- a/configure.in
++++ b/configure.in
+@@ -4059,8 +4059,8 @@ locale_dir="`eval echo ${locale_dir}`"
AC_DEFINE_UNQUOTED(PR_LOCALE_DIR, "`eval echo "${locale_dir}"`", [Define the locale directory])
AC_DEFINE_UNQUOTED(PR_RUN_DIR, "`eval echo "${localstatedir}"`", [Define the run directory])
@@ -97,16 +97,3 @@ Index: proftpd-1.3.6/configure.in
prefix="$pr_saved_prefix"
exec_prefix="$pr_saved_exec_prefix"
-Index: proftpd-1.3.6/lib/libcap/Makefile
-===================================================================
---- proftpd-1.3.6.orig/lib/libcap/Makefile
-+++ proftpd-1.3.6/lib/libcap/Makefile
-@@ -26,7 +26,7 @@ OBJS=$(addsuffix .o, $(FILES))
- all: $(LIBNAME)
-
- _makenames: _makenames.c cap_names.sed
-- $(CC) $(CFLAGS) $(LDFLAGS) $< -o $@
-+ $(BUILD_CC) $(CFLAGS) $(LDFLAGS) $< -o $@
-
- cap_names.h: _makenames
- ./_makenames > cap_names.h
diff --git a/meta-networking/recipes-daemons/proftpd/files/contrib.patch b/meta-networking/recipes-daemons/proftpd/files/contrib.patch
index 7e2a8e3ce..96c237d38 100644
--- a/meta-networking/recipes-daemons/proftpd/files/contrib.patch
+++ b/meta-networking/recipes-daemons/proftpd/files/contrib.patch
@@ -11,32 +11,29 @@ Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/Makefile.in b/Makefile.in
-index 5b2e683..ee72fe1 100644
+index 9434cbd..8da7d1f 100644
--- a/Makefile.in
+++ b/Makefile.in
-@@ -120,7 +120,7 @@ install-modules: $(DESTDIR)$(libexecdir) $(DESTDIR)$(sysconfdir)
+@@ -163,7 +163,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
+ $(INSTALL_BIN) $(top_builddir)/ftpcount $(DESTDIR)$(bindir)/ftpcount
+ $(INSTALL_BIN) $(top_builddir)/ftpdctl $(DESTDIR)$(bindir)/ftpdctl
+ $(INSTALL_SBIN) $(top_builddir)/ftpscrub $(DESTDIR)$(sbindir)/ftpscrub
diff --git a/contrib/Makefile.in b/contrib/Makefile.in
-index 5bcc038..51d248c 100644
+index d1a31af..65d2abc 100644
--- a/contrib/Makefile.in
+++ b/contrib/Makefile.in
-@@ -18,6 +18,6 @@ Makefile: Makefile.in ../config.status
+@@ -17,6 +17,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
-
+- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpasswd $(DESTDIR)$(bindir)/ftpasswd
+- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpmail $(DESTDIR)$(bindir)/ftpmail
+- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpquota $(DESTDIR)$(bindir)/ftpquota
++ $(INSTALL) -m 0755 $(srcdir)/ftpasswd $(DESTDIR)$(bindir)/ftpasswd
++ $(INSTALL) -m 0755 $(srcdir)/ftpmail $(DESTDIR)$(bindir)/ftpmail
++ $(INSTALL) -m 0755 $(srcdir)/ftpquota $(DESTDIR)$(bindir)/ftpquota
diff --git a/meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb b/meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
similarity index 93%
rename from meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
rename to meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
index 1e4697a63..7bd41546e 100644
--- a/meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
+++ b/meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
@@ -4,17 +4,17 @@ HOMEPAGE = "http://www.proftpd.org"
LICENSE = "GPLv2+"
LIC_FILES_CHKSUM = "file://COPYING;md5=fb0d1484d11915fa88a6a7702f1dc184"
-SRC_URI = "ftp://ftp.proftpd.org/distrib/source/${BPN}-${PV}.tar.gz \
+SRC_URI = "https://github.com/proftpd/proftpd/archive/refs/tags/v${PV}.tar.gz \
file://basic.conf.patch \
file://proftpd-basic.init \
file://default \
- file://close-RequireValidShell-check.patch \
- file://contrib.patch \
+ file://contrib.patch \
file://build_fixup.patch \
+ file://close-RequireValidShell-check.patch \
file://proftpd.service \
"
-SRC_URI[md5sum] = "13270911c42aac842435f18205546a1b"
-SRC_URI[sha256sum] = "91ef74b143495d5ff97c4d4770c6804072a8c8eb1ad1ecc8cc541b40e152ecaf"
+SRC_URI[md5sum] = "4a9b8877b2e9b08d70e71ad56c19e2c9"
+SRC_URI[sha256sum] = "8b7bbf9757988935352d9dec5ebf96b6a1e6b63a6cdac2e93202ac6c42c4cd96"
inherit autotools-brokensep useradd update-rc.d systemd multilib_script
@@ -24,7 +24,7 @@ PACKAGECONFIG ??= "shadow \
PACKAGECONFIG[curses] = "--enable-curses --enable-ncurses, --disable-curses --disable-ncurses, ncurses"
PACKAGECONFIG[openssl] = "--enable-openssl, --disable-openssl, openssl, openssl"
-PACKAGECONFIG[pam] = "--enable-auth-pam, --disable-auth-pam, libpam, libpam"
+PACKAGECONFIG[pam] = "--enable-tests, --enable-auth-pam, --disable-auth-pam, libpam, libpam"
PACKAGECONFIG[ipv6] = "--enable-ipv6, --disable-ipv6"
PACKAGECONFIG[shadow] = "--enable-shadow, --disable-shadow"
PACKAGECONFIG[pcre] = "--enable-pcre, --disable-pcre, libpcre "
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [oe] [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release
2021-04-19 7:52 ` [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release Andrej Kozemcak
@ 2021-04-19 21:16 ` Khem Raj
2021-04-20 10:13 ` Andrej Kozemcak
0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2021-04-19 21:16 UTC (permalink / raw)
To: Andrej Kozemcak, openembedded-devel
Hello Andrej
Thanks for your contribution, there are few comments I have
On 4/19/21 12:52 AM, Andrej Kozemcak wrote:
> Signed-off-by: Andrej Kozemcak <andrej.kozemcak@siemens.com>
> ---
> .../proftpd/files/build_fixup.patch | 69 ++++++++-----------
> .../proftpd/files/contrib.patch | 29 ++++----
> .../{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} | 12 ++--
> 3 files changed, 47 insertions(+), 63 deletions(-)
> rename meta-networking/recipes-daemons/proftpd/{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} (93%)
>
> diff --git a/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch b/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> index 9816d4d22..d888dd826 100644
> --- a/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> +++ b/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> @@ -9,10 +9,10 @@ redefine PR_RUN_DIR as ${localstatedir}/run
> Signed-off-By: Armin Kuster <akuster808@gmail.com>
>
>
> -Index: proftpd-1.3.6/Make.rules.in
> -===================================================================
> ---- proftpd-1.3.6.orig/Make.rules.in
> -+++ proftpd-1.3.6/Make.rules.in
> +diff --git a/Make.rules.in b/Make.rules.in
> +index a5028ac..ccd7807 100644
> +--- a/Make.rules.in
> ++++ b/Make.rules.in
> @@ -30,9 +30,9 @@ INSTALL=@INSTALL@
> INSTALL_STRIP=@INSTALL_STRIP@
> INSTALL_USER=@install_user@
> @@ -26,11 +26,11 @@ Index: proftpd-1.3.6/Make.rules.in
>
> RM=rm -f
> SHELL=@CONFIG_SHELL@
> -Index: proftpd-1.3.6/Makefile.in
> -===================================================================
> ---- proftpd-1.3.6.orig/Makefile.in
> -+++ proftpd-1.3.6/Makefile.in
> -@@ -105,7 +105,6 @@ check: proftpd$(EXEEXT)
> +diff --git a/Makefile.in b/Makefile.in
> +index 9434cbd..13f2e16 100644
> +--- a/Makefile.in
> ++++ b/Makefile.in
> +@@ -123,7 +123,6 @@ check: proftpd$(EXEEXT)
> $(DESTDIR)$(localedir) $(DESTDIR)$(includedir) $(DESTDIR)$(includedir)/proftpd $(DESTDIR)$(libdir) $(DESTDIR)$(pkgconfigdir) $(DESTDIR)$(libdir)/proftpd $(DESTDIR)$(libexecdir) $(DESTDIR)$(localstatedir) $(DESTDIR)$(sysconfdir) $(DESTDIR)$(bindir) $(DESTDIR)$(sbindir) $(DESTDIR)$(mandir) $(DESTDIR)$(mandir)/man1 $(DESTDIR)$(mandir)/man5 $(DESTDIR)$(mandir)/man8:
> @if [ ! -d $@ ]; then \
> mkdir -p $@; \
> @@ -38,20 +38,20 @@ Index: proftpd-1.3.6/Makefile.in
> chmod 0755 $@; \
> fi
>
> -@@ -115,7 +114,6 @@ install-proftpd: proftpd $(DESTDIR)$(inc
> +@@ -133,7 +132,6 @@ install-proftpd: proftpd$(EXEEXT) $(DESTDIR)$(includedir) $(DESTDIR)$(localstate
> rm -f $(DESTDIR)$(sbindir)/in.proftpd ; \
> fi
> - ln -s proftpd $(DESTDIR)$(sbindir)/in.proftpd
> + ln -s $(top_builddir)/proftpd $(DESTDIR)$(sbindir)/in.proftpd
> - -chown -h $(INSTALL_USER):$(INSTALL_GROUP) $(DESTDIR)$(sbindir)/in.proftpd
>
> install-libs: $(DESTDIR)$(libdir)/proftpd
> cd lib/ && $(MAKE) install
> -@@ -152,11 +150,11 @@ install-utils: $(DESTDIR)$(sbindir) $(DE
> - $(INSTALL_SBIN) ftpshut $(DESTDIR)$(sbindir)/ftpshut
> - $(INSTALL_BIN) ftptop $(DESTDIR)$(bindir)/ftptop
> - $(INSTALL_BIN) ftpwho $(DESTDIR)$(bindir)/ftpwho
> -- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 src/prxs $(DESTDIR)$(bindir)/prxs
> -+ $(INSTALL) -m 0755 src/prxs $(DESTDIR)$(bindir)/prxs
> +@@ -170,11 +168,11 @@ install-utils: $(DESTDIR)$(sbindir) $(DESTDIR)$(bindir)
> + $(INSTALL_SBIN) $(top_builddir)/ftpshut $(DESTDIR)$(sbindir)/ftpshut
> + $(INSTALL_BIN) $(top_builddir)/ftptop $(DESTDIR)$(bindir)/ftptop
> + $(INSTALL_BIN) $(top_builddir)/ftpwho $(DESTDIR)$(bindir)/ftpwho
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(top_builddir)/src/prxs $(DESTDIR)$(bindir)/prxs
> ++ $(INSTALL) -m 0755 $(top_builddir)/src/prxs $(DESTDIR)$(bindir)/prxs
>
> install-conf: $(DESTDIR)$(sysconfdir)
> if [ ! -f $(DESTDIR)$(sysconfdir)/proftpd.conf ] ; then \
> @@ -60,11 +60,11 @@ Index: proftpd-1.3.6/Makefile.in
> $(top_srcdir)/sample-configurations/basic.conf \
> $(DESTDIR)$(sysconfdir)/proftpd.conf ; \
> fi
> -Index: proftpd-1.3.6/configure
> -===================================================================
> ---- proftpd-1.3.6.orig/configure
> -+++ proftpd-1.3.6/configure
> -@@ -41777,7 +41777,7 @@ _ACEOF
> +diff --git a/configure b/configure
> +index 0a51670..69fa130 100755
> +--- a/configure
> ++++ b/configure
> +@@ -25116,7 +25116,7 @@ _ACEOF
>
>
> cat >>confdefs.h <<_ACEOF
> @@ -73,7 +73,7 @@ Index: proftpd-1.3.6/configure
> _ACEOF
>
>
> -@@ -41787,7 +41787,7 @@ _ACEOF
> +@@ -25126,7 +25126,7 @@ _ACEOF
>
>
> cat >>confdefs.h <<_ACEOF
> @@ -82,11 +82,11 @@ Index: proftpd-1.3.6/configure
> _ACEOF
>
>
> -Index: proftpd-1.3.6/configure.in
> -===================================================================
> ---- proftpd-1.3.6.orig/configure.in
> -+++ proftpd-1.3.6/configure.in
> -@@ -3833,8 +3833,8 @@ locale_dir="`eval echo ${locale_dir}`"
> +diff --git a/configure.in b/configure.in
> +index 50561de..93cad3f 100644
> +--- a/configure.in
> ++++ b/configure.in
> +@@ -4059,8 +4059,8 @@ locale_dir="`eval echo ${locale_dir}`"
> AC_DEFINE_UNQUOTED(PR_LOCALE_DIR, "`eval echo "${locale_dir}"`", [Define the locale directory])
>
> AC_DEFINE_UNQUOTED(PR_RUN_DIR, "`eval echo "${localstatedir}"`", [Define the run directory])
> @@ -97,16 +97,3 @@ Index: proftpd-1.3.6/configure.in
>
> prefix="$pr_saved_prefix"
> exec_prefix="$pr_saved_exec_prefix"
> -Index: proftpd-1.3.6/lib/libcap/Makefile
> -===================================================================
> ---- proftpd-1.3.6.orig/lib/libcap/Makefile
> -+++ proftpd-1.3.6/lib/libcap/Makefile
> -@@ -26,7 +26,7 @@ OBJS=$(addsuffix .o, $(FILES))
> - all: $(LIBNAME)
> -
> - _makenames: _makenames.c cap_names.sed
> -- $(CC) $(CFLAGS) $(LDFLAGS) $< -o $@
> -+ $(BUILD_CC) $(CFLAGS) $(LDFLAGS) $< -o $@
> -
> - cap_names.h: _makenames
> - ./_makenames > cap_names.h
> diff --git a/meta-networking/recipes-daemons/proftpd/files/contrib.patch b/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> index 7e2a8e3ce..96c237d38 100644
> --- a/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> +++ b/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> @@ -11,32 +11,29 @@ Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Makefile.in b/Makefile.in
> -index 5b2e683..ee72fe1 100644
> +index 9434cbd..8da7d1f 100644
> --- a/Makefile.in
> +++ b/Makefile.in
> -@@ -120,7 +120,7 @@ install-modules: $(DESTDIR)$(libexecdir) $(DESTDIR)$(sysconfdir)
> +@@ -163,7 +163,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
> + $(INSTALL_BIN) $(top_builddir)/ftpcount $(DESTDIR)$(bindir)/ftpcount
> + $(INSTALL_BIN) $(top_builddir)/ftpdctl $(DESTDIR)$(bindir)/ftpdctl
> + $(INSTALL_SBIN) $(top_builddir)/ftpscrub $(DESTDIR)$(sbindir)/ftpscrub
> diff --git a/contrib/Makefile.in b/contrib/Makefile.in
> -index 5bcc038..51d248c 100644
> +index d1a31af..65d2abc 100644
> --- a/contrib/Makefile.in
> +++ b/contrib/Makefile.in
> -@@ -18,6 +18,6 @@ Makefile: Makefile.in ../config.status
> +@@ -17,6 +17,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
> -
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpasswd $(DESTDIR)$(bindir)/ftpasswd
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpmail $(DESTDIR)$(bindir)/ftpmail
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpquota $(DESTDIR)$(bindir)/ftpquota
> ++ $(INSTALL) -m 0755 $(srcdir)/ftpasswd $(DESTDIR)$(bindir)/ftpasswd
> ++ $(INSTALL) -m 0755 $(srcdir)/ftpmail $(DESTDIR)$(bindir)/ftpmail
> ++ $(INSTALL) -m 0755 $(srcdir)/ftpquota $(DESTDIR)$(bindir)/ftpquota
> diff --git a/meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb b/meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
> similarity index 93%
> rename from meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
> rename to meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
> index 1e4697a63..7bd41546e 100644
> --- a/meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
> +++ b/meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
> @@ -4,17 +4,17 @@ HOMEPAGE = "http://www.proftpd.org"
> LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=fb0d1484d11915fa88a6a7702f1dc184"
>
> -SRC_URI = "ftp://ftp.proftpd.org/distrib/source/${BPN}-${PV}.tar.gz \
> +SRC_URI = "https://github.com/proftpd/proftpd/archive/refs/tags/v${PV}.tar.gz \
usually we do not use github archive provided tarballs as they are known
to mutate and checksums change. Perhaps better to use github fetcher if
there is no other place where release tarballs are available.
> file://basic.conf.patch \
> file://proftpd-basic.init \
> file://default \
> - file://close-RequireValidShell-check.patch \
> - file://contrib.patch \
> + file://contrib.patch \
> file://build_fixup.patch \
> + file://close-RequireValidShell-check.patch \
> file://proftpd.service \
There seems to mix of spaces and tabs, so perhaps make them uniform as
well while here, perhaps in a separate patch though.
> "
> -SRC_URI[md5sum] = "13270911c42aac842435f18205546a1b"
> -SRC_URI[sha256sum] = "91ef74b143495d5ff97c4d4770c6804072a8c8eb1ad1ecc8cc541b40e152ecaf"
> +SRC_URI[md5sum] = "4a9b8877b2e9b08d70e71ad56c19e2c9"
> +SRC_URI[sha256sum] = "8b7bbf9757988935352d9dec5ebf96b6a1e6b63a6cdac2e93202ac6c42c4cd96"
>
> inherit autotools-brokensep useradd update-rc.d systemd multilib_script
>
> @@ -24,7 +24,7 @@ PACKAGECONFIG ??= "shadow \
>
> PACKAGECONFIG[curses] = "--enable-curses --enable-ncurses, --disable-curses --disable-ncurses, ncurses"
> PACKAGECONFIG[openssl] = "--enable-openssl, --disable-openssl, openssl, openssl"
> -PACKAGECONFIG[pam] = "--enable-auth-pam, --disable-auth-pam, libpam, libpam"
> +PACKAGECONFIG[pam] = "--enable-tests, --enable-auth-pam, --disable-auth-pam, libpam, libpam"
This seems a bit off to me. There is a syntax problem where the comma
after --enable-tests is not required if you wanted to enable tests when
pam is enabled in packageconfig. Second question is if this is required
to be bundled with this knob or should it be a seprate independent knob.
> PACKAGECONFIG[ipv6] = "--enable-ipv6, --disable-ipv6"
> PACKAGECONFIG[shadow] = "--enable-shadow, --disable-shadow"
> PACKAGECONFIG[pcre] = "--enable-pcre, --disable-pcre, libpcre "
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [oe] [meta-oe][PATCH 0/1] proftpd update
2021-04-19 7:52 [meta-oe][PATCH 0/1] proftpd update Andrej Kozemcak
2021-04-19 7:52 ` [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release Andrej Kozemcak
@ 2021-04-19 21:17 ` Khem Raj
2021-05-06 5:11 ` Andrej Kozemcak
1 sibling, 1 reply; 6+ messages in thread
From: Khem Raj @ 2021-04-19 21:17 UTC (permalink / raw)
To: Andrej Kozemcak, openembedded-devel, Armin Kuster
On 4/19/21 12:52 AM, Andrej Kozemcak wrote:
> I would like to apply this update also to dunfell branch, if posible.
> Because there is lot of bugs which is not a posible fix with the easy
> way.
This is a leaf package so I think it might be ok from dependency point
of view, I would leave this to Armin and other interested community
members to weigh in for dunfell backport.
>
> Andrej Kozemcak (1):
> proftpd: Update to 1.3.7a release
>
> .../proftpd/files/build_fixup.patch | 69 ++++++++-----------
> .../proftpd/files/contrib.patch | 29 ++++----
> .../{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} | 12 ++--
> 3 files changed, 47 insertions(+), 63 deletions(-)
> rename meta-networking/recipes-daemons/proftpd/{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} (93%)
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [oe] [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release
2021-04-19 21:16 ` [oe] " Khem Raj
@ 2021-04-20 10:13 ` Andrej Kozemcak
0 siblings, 0 replies; 6+ messages in thread
From: Andrej Kozemcak @ 2021-04-20 10:13 UTC (permalink / raw)
To: Khem Raj, openembedded-devel
Hi Raj,
Thank you for the review.
I looked at you commend and fix the issue.
I will send the new patch.
>> +PACKAGECONFIG[pam] = "--enable-tests, --enable-auth-pam, --disable-auth-pam, libpam, libpam"
> This seems a bit off to me. There is a syntax problem where the comma after --enable-tests is not required if you wanted to enable tests when pam is enabled in > > packageconfig. Second question is if this is required to be bundled with this knob or should it be a seprate independent knob.
My error, I was playing with testing and I forgot/miss remove that line. (But I was 100% sure I was remove it).
Thank you, for noticed that.
Best Regards,
Andrej
-----Original Message-----
From: Khem Raj <raj.khem@gmail.com>
Sent: Monday, April 19, 2021 11:16 PM
To: Kozemcak, Andrej (ADV D EU SK SI-BP1) <andrej.kozemcak@siemens.com>; openembedded-devel@lists.openembedded.org
Subject: Re: [oe] [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release
Hello Andrej
Thanks for your contribution, there are few comments I have
On 4/19/21 12:52 AM, Andrej Kozemcak wrote:
> Signed-off-by: Andrej Kozemcak <andrej.kozemcak@siemens.com>
> ---
> .../proftpd/files/build_fixup.patch | 69 ++++++++-----------
> .../proftpd/files/contrib.patch | 29 ++++----
> .../{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} | 12 ++--
> 3 files changed, 47 insertions(+), 63 deletions(-)
> rename meta-networking/recipes-daemons/proftpd/{proftpd_1.3.6.bb =>
> proftpd_1.3.7a.bb} (93%)
>
> diff --git
> a/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> b/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> index 9816d4d22..d888dd826 100644
> --- a/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> +++ b/meta-networking/recipes-daemons/proftpd/files/build_fixup.patch
> @@ -9,10 +9,10 @@ redefine PR_RUN_DIR as ${localstatedir}/run
> Signed-off-By: Armin Kuster <akuster808@gmail.com>
>
>
> -Index: proftpd-1.3.6/Make.rules.in
> -===================================================================
> ---- proftpd-1.3.6.orig/Make.rules.in
> -+++ proftpd-1.3.6/Make.rules.in
> +diff --git a/Make.rules.in b/Make.rules.in index a5028ac..ccd7807
> +100644
> +--- a/Make.rules.in
> ++++ b/Make.rules.in
> @@ -30,9 +30,9 @@ INSTALL=@INSTALL@
> INSTALL_STRIP=@INSTALL_STRIP@
> INSTALL_USER=@install_user@
> @@ -26,11 +26,11 @@ Index: proftpd-1.3.6/Make.rules.in
>
> RM=rm -f
> SHELL=@CONFIG_SHELL@
> -Index: proftpd-1.3.6/Makefile.in
> -===================================================================
> ---- proftpd-1.3.6.orig/Makefile.in
> -+++ proftpd-1.3.6/Makefile.in
> -@@ -105,7 +105,6 @@ check: proftpd$(EXEEXT)
> +diff --git a/Makefile.in b/Makefile.in index 9434cbd..13f2e16 100644
> +--- a/Makefile.in
> ++++ b/Makefile.in
> +@@ -123,7 +123,6 @@ check: proftpd$(EXEEXT)
> $(DESTDIR)$(localedir) $(DESTDIR)$(includedir) $(DESTDIR)$(includedir)/proftpd $(DESTDIR)$(libdir) $(DESTDIR)$(pkgconfigdir) $(DESTDIR)$(libdir)/proftpd $(DESTDIR)$(libexecdir) $(DESTDIR)$(localstatedir) $(DESTDIR)$(sysconfdir) $(DESTDIR)$(bindir) $(DESTDIR)$(sbindir) $(DESTDIR)$(mandir) $(DESTDIR)$(mandir)/man1 $(DESTDIR)$(mandir)/man5 $(DESTDIR)$(mandir)/man8:
> @if [ ! -d $@ ]; then \
> mkdir -p $@; \
> @@ -38,20 +38,20 @@ Index: proftpd-1.3.6/Makefile.in
> chmod 0755 $@; \
> fi
>
> -@@ -115,7 +114,6 @@ install-proftpd: proftpd $(DESTDIR)$(inc
> +@@ -133,7 +132,6 @@ install-proftpd: proftpd$(EXEEXT)
> +$(DESTDIR)$(includedir) $(DESTDIR)$(localstate
> rm -f $(DESTDIR)$(sbindir)/in.proftpd ; \
> fi
> - ln -s proftpd $(DESTDIR)$(sbindir)/in.proftpd
> + ln -s $(top_builddir)/proftpd $(DESTDIR)$(sbindir)/in.proftpd
> - -chown -h $(INSTALL_USER):$(INSTALL_GROUP) $(DESTDIR)$(sbindir)/in.proftpd
>
> install-libs: $(DESTDIR)$(libdir)/proftpd
> cd lib/ && $(MAKE) install
> -@@ -152,11 +150,11 @@ install-utils: $(DESTDIR)$(sbindir) $(DE
> - $(INSTALL_SBIN) ftpshut $(DESTDIR)$(sbindir)/ftpshut
> - $(INSTALL_BIN) ftptop $(DESTDIR)$(bindir)/ftptop
> - $(INSTALL_BIN) ftpwho $(DESTDIR)$(bindir)/ftpwho
> -- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 src/prxs $(DESTDIR)$(bindir)/prxs
> -+ $(INSTALL) -m 0755 src/prxs $(DESTDIR)$(bindir)/prxs
> +@@ -170,11 +168,11 @@ install-utils: $(DESTDIR)$(sbindir) $(DESTDIR)$(bindir)
> + $(INSTALL_SBIN) $(top_builddir)/ftpshut $(DESTDIR)$(sbindir)/ftpshut
> + $(INSTALL_BIN) $(top_builddir)/ftptop $(DESTDIR)$(bindir)/ftptop
> + $(INSTALL_BIN) $(top_builddir)/ftpwho $(DESTDIR)$(bindir)/ftpwho
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(top_builddir)/src/prxs $(DESTDIR)$(bindir)/prxs
> ++ $(INSTALL) -m 0755 $(top_builddir)/src/prxs
> ++$(DESTDIR)$(bindir)/prxs
>
> install-conf: $(DESTDIR)$(sysconfdir)
> if [ ! -f $(DESTDIR)$(sysconfdir)/proftpd.conf ] ; then \ @@
> -60,11 +60,11 @@ Index: proftpd-1.3.6/Makefile.in
> $(top_srcdir)/sample-configurations/basic.conf \
> $(DESTDIR)$(sysconfdir)/proftpd.conf ; \
> fi
> -Index: proftpd-1.3.6/configure
> -===================================================================
> ---- proftpd-1.3.6.orig/configure
> -+++ proftpd-1.3.6/configure
> -@@ -41777,7 +41777,7 @@ _ACEOF
> +diff --git a/configure b/configure
> +index 0a51670..69fa130 100755
> +--- a/configure
> ++++ b/configure
> +@@ -25116,7 +25116,7 @@ _ACEOF
>
>
> cat >>confdefs.h <<_ACEOF
> @@ -73,7 +73,7 @@ Index: proftpd-1.3.6/configure
> _ACEOF
>
>
> -@@ -41787,7 +41787,7 @@ _ACEOF
> +@@ -25126,7 +25126,7 @@ _ACEOF
>
>
> cat >>confdefs.h <<_ACEOF
> @@ -82,11 +82,11 @@ Index: proftpd-1.3.6/configure
> _ACEOF
>
>
> -Index: proftpd-1.3.6/configure.in
> -===================================================================
> ---- proftpd-1.3.6.orig/configure.in
> -+++ proftpd-1.3.6/configure.in
> -@@ -3833,8 +3833,8 @@ locale_dir="`eval echo ${locale_dir}`"
> +diff --git a/configure.in b/configure.in index 50561de..93cad3f
> +100644
> +--- a/configure.in
> ++++ b/configure.in
> +@@ -4059,8 +4059,8 @@ locale_dir="`eval echo ${locale_dir}`"
> AC_DEFINE_UNQUOTED(PR_LOCALE_DIR, "`eval echo "${locale_dir}"`",
> [Define the locale directory])
>
> AC_DEFINE_UNQUOTED(PR_RUN_DIR, "`eval echo "${localstatedir}"`",
> [Define the run directory]) @@ -97,16 +97,3 @@ Index:
> proftpd-1.3.6/configure.in
>
> prefix="$pr_saved_prefix"
> exec_prefix="$pr_saved_exec_prefix"
> -Index: proftpd-1.3.6/lib/libcap/Makefile
> -===================================================================
> ---- proftpd-1.3.6.orig/lib/libcap/Makefile
> -+++ proftpd-1.3.6/lib/libcap/Makefile
> -@@ -26,7 +26,7 @@ OBJS=$(addsuffix .o, $(FILES))
> - all: $(LIBNAME)
> -
> - _makenames: _makenames.c cap_names.sed
> -- $(CC) $(CFLAGS) $(LDFLAGS) $< -o $@
> -+ $(BUILD_CC) $(CFLAGS) $(LDFLAGS) $< -o $@
> -
> - cap_names.h: _makenames
> - ./_makenames > cap_names.h
> diff --git
> a/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> b/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> index 7e2a8e3ce..96c237d38 100644
> --- a/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> +++ b/meta-networking/recipes-daemons/proftpd/files/contrib.patch
> @@ -11,32 +11,29 @@ Signed-off-by: Kevin Strasser <kevin.strasser@linux.intel.com>
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Makefile.in b/Makefile.in -index 5b2e683..ee72fe1
> 100644
> +index 9434cbd..8da7d1f 100644
> --- a/Makefile.in
> +++ b/Makefile.in
> -@@ -120,7 +120,7 @@ install-modules: $(DESTDIR)$(libexecdir)
> $(DESTDIR)$(sysconfdir)
> +@@ -163,7 +163,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
> + $(INSTALL_BIN) $(top_builddir)/ftpcount $(DESTDIR)$(bindir)/ftpcount
> + $(INSTALL_BIN) $(top_builddir)/ftpdctl $(DESTDIR)$(bindir)/ftpdctl
> + $(INSTALL_SBIN) $(top_builddir)/ftpscrub
> +$(DESTDIR)$(sbindir)/ftpscrub
> diff --git a/contrib/Makefile.in b/contrib/Makefile.in -index
> 5bcc038..51d248c 100644
> +index d1a31af..65d2abc 100644
> --- a/contrib/Makefile.in
> +++ b/contrib/Makefile.in
> -@@ -18,6 +18,6 @@ Makefile: Makefile.in ../config.status
> +@@ -17,6 +17,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
> -
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpasswd $(DESTDIR)$(bindir)/ftpasswd
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpmail $(DESTDIR)$(bindir)/ftpmail
> +- $(INSTALL) -o $(INSTALL_USER) -g $(INSTALL_GROUP) -m 0755 $(srcdir)/ftpquota $(DESTDIR)$(bindir)/ftpquota
> ++ $(INSTALL) -m 0755 $(srcdir)/ftpasswd $(DESTDIR)$(bindir)/ftpasswd
> ++ $(INSTALL) -m 0755 $(srcdir)/ftpmail $(DESTDIR)$(bindir)/ftpmail
> ++ $(INSTALL) -m 0755 $(srcdir)/ftpquota $(DESTDIR)$(bindir)/ftpquota
> diff --git a/meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
> b/meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
> similarity index 93%
> rename from meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
> rename to meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
> index 1e4697a63..7bd41546e 100644
> --- a/meta-networking/recipes-daemons/proftpd/proftpd_1.3.6.bb
> +++ b/meta-networking/recipes-daemons/proftpd/proftpd_1.3.7a.bb
> @@ -4,17 +4,17 @@ HOMEPAGE = "https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.proftpd.org%2F&data=04%7C01%7Candrej.kozemcak%40siemens.com%7Ce79d5485369e4416d5a308d903785e75%7C38ae3bcd95794fd4addab42e1495d55a%7C1%7C0%7C637544637785377046%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=CeYGBB6XgsHXv3MbCqadtvUFBpFMM5lO9BJGXWzN2Y4%3D&reserved=0"
> LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=fb0d1484d11915fa88a6a7702f1dc184"
>
> -SRC_URI =
> "https://eur01.safelinks.protection.outlook.com/?url=ftp%3A%2F%2Fftp.p
> roftpd.org%2Fdistrib%2Fsource%2F%24&data=04%7C01%7Candrej.kozemcak
> %40siemens.com%7Ce79d5485369e4416d5a308d903785e75%7C38ae3bcd95794fd4ad
> dab42e1495d55a%7C1%7C0%7C637544637785377046%7CUnknown%7CTWFpbGZsb3d8ey
> JWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C100
> 0&sdata=6ROO2ugn0EAD8%2BsVOupDogNREpO7OI2Gf6si7DjrvGk%3D&reser
> ved=0{BPN}-${PV}.tar.gz \
> +SRC_URI =
> +"https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgi
> +thub.com%2Fproftpd%2Fproftpd%2Farchive%2Frefs%2Ftags%2Fv%24&data=
> +04%7C01%7Candrej.kozemcak%40siemens.com%7Ce79d5485369e4416d5a308d9037
> +85e75%7C38ae3bcd95794fd4addab42e1495d55a%7C1%7C0%7C637544637785377046
> +%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6
> +Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=TFaiJj5gD7RKwNUGT1EpYj4ZcE15D
> +5lm2UzKACfZWR8%3D&reserved=0{PV}.tar.gz \
usually we do not use github archive provided tarballs as they are known to mutate and checksums change. Perhaps better to use github fetcher if there is no other place where release tarballs are available.
> file://basic.conf.patch \
> file://proftpd-basic.init \
> file://default \
> - file://close-RequireValidShell-check.patch \
> - file://contrib.patch \
> + file://contrib.patch \
> file://build_fixup.patch \
> + file://close-RequireValidShell-check.patch \
> file://proftpd.service \
There seems to mix of spaces and tabs, so perhaps make them uniform as well while here, perhaps in a separate patch though.
> "
> -SRC_URI[md5sum] = "13270911c42aac842435f18205546a1b"
> -SRC_URI[sha256sum] = "91ef74b143495d5ff97c4d4770c6804072a8c8eb1ad1ecc8cc541b40e152ecaf"
> +SRC_URI[md5sum] = "4a9b8877b2e9b08d70e71ad56c19e2c9"
> +SRC_URI[sha256sum] = "8b7bbf9757988935352d9dec5ebf96b6a1e6b63a6cdac2e93202ac6c42c4cd96"
>
> inherit autotools-brokensep useradd update-rc.d systemd
> multilib_script
>
> @@ -24,7 +24,7 @@ PACKAGECONFIG ??= "shadow \
>
> PACKAGECONFIG[curses] = "--enable-curses --enable-ncurses, --disable-curses --disable-ncurses, ncurses"
> PACKAGECONFIG[openssl] = "--enable-openssl, --disable-openssl, openssl, openssl"
> -PACKAGECONFIG[pam] = "--enable-auth-pam, --disable-auth-pam, libpam, libpam"
> +PACKAGECONFIG[pam] = "--enable-tests, --enable-auth-pam, --disable-auth-pam, libpam, libpam"
This seems a bit off to me. There is a syntax problem where the comma after --enable-tests is not required if you wanted to enable tests when pam is enabled in packageconfig. Second question is if this is required to be bundled with this knob or should it be a seprate independent knob.
> PACKAGECONFIG[ipv6] = "--enable-ipv6, --disable-ipv6"
> PACKAGECONFIG[shadow] = "--enable-shadow, --disable-shadow"
> PACKAGECONFIG[pcre] = "--enable-pcre, --disable-pcre, libpcre "
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [oe] [meta-oe][PATCH 0/1] proftpd update
2021-04-19 21:17 ` [oe] [meta-oe][PATCH 0/1] proftpd update Khem Raj
@ 2021-05-06 5:11 ` Andrej Kozemcak
0 siblings, 0 replies; 6+ messages in thread
From: Andrej Kozemcak @ 2021-05-06 5:11 UTC (permalink / raw)
To: Khem Raj, Armin Kuster; +Cc: openembedded-devel
Hi Khem, Armin,
Is want only inform, is there any new information about apply this patch also to dunfell branch?
Thank you for the information,
Best Regards,
Andrej K.
-----Original Message-----
From: Khem Raj <raj.khem@gmail.com>
Sent: Monday, April 19, 2021 11:18 PM
To: Kozemcak, Andrej (ADV D EU SK SI-BP1) <andrej.kozemcak@siemens.com>; openembedded-devel@lists.openembedded.org; Armin Kuster <akuster808@gmail.comt>
Subject: Re: [oe] [meta-oe][PATCH 0/1] proftpd update
On 4/19/21 12:52 AM, Andrej Kozemcak wrote:
> I would like to apply this update also to dunfell branch, if posible.
> Because there is lot of bugs which is not a posible fix with the easy
> way.
This is a leaf package so I think it might be ok from dependency point of view, I would leave this to Armin and other interested community members to weigh in for dunfell backport.
>
> Andrej Kozemcak (1):
> proftpd: Update to 1.3.7a release
>
> .../proftpd/files/build_fixup.patch | 69 ++++++++-----------
> .../proftpd/files/contrib.patch | 29 ++++----
> .../{proftpd_1.3.6.bb => proftpd_1.3.7a.bb} | 12 ++--
> 3 files changed, 47 insertions(+), 63 deletions(-)
> rename meta-networking/recipes-daemons/proftpd/{proftpd_1.3.6.bb =>
> proftpd_1.3.7a.bb} (93%)
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-05-06 5:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-19 7:52 [meta-oe][PATCH 0/1] proftpd update Andrej Kozemcak
2021-04-19 7:52 ` [meta-oe][PATCH 1/1] proftpd: Update to 1.3.7a release Andrej Kozemcak
2021-04-19 21:16 ` [oe] " Khem Raj
2021-04-20 10:13 ` Andrej Kozemcak
2021-04-19 21:17 ` [oe] [meta-oe][PATCH 0/1] proftpd update Khem Raj
2021-05-06 5:11 ` Andrej Kozemcak
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.