Hi Paul,
On Mon, Oct 28, 2013 at 11:27:01AM +0000, Paul Eggleton wrote:
> Hi Maxin,
>
> On Monday 21 October 2013 17:36:42 maxin.j...@enea.com wrote:
> > From: "Maxin B. John" <maxin.j...@enea.com>
> >
> > update to latest version 2013.60
> > Update 0006-dropbear-configuration-file.patch for 2013.60
> >
> > Signed-off-by: Maxin B. John <maxin.j...@enea.com>
> > ---
> > .../0001-urandom-xauth-changes-to-options.h.patch | 0
> > .../0002-static_build_fix.patch | 0
> > .../0003-configure.patch | 0
> > .../0004-fix-2kb-keys.patch | 0
> > .../0005-dropbear-enable-pam.patch | 0
> > .../0006-dropbear-configuration-file.patch | 22
> > +++++++------------- .../0007-dropbear-fix-for-x32-abi.patch |
> > 0
> > meta/recipes-core/dropbear/dropbear.inc | 4 ++--
> > meta/recipes-core/dropbear/dropbear_2013.58.bb | 6 ------
> > meta/recipes-core/dropbear/dropbear_2013.60.bb | 6 ++++++
> > 10 files changed, 16 insertions(+), 22 deletions(-)
> > rename meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0001-urandom-xauth-changes-to-options.h.patch (100%)
> > rename meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0002-static_build_fix.patch (100%) rename
> > meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0003-configure.patch (100%) rename
> > meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0004-fix-2kb-keys.patch (100%) rename
> > meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0005-dropbear-enable-pam.patch (100%) rename
> > meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0006-dropbear-configuration-file.patch (56%) rename
> > meta/recipes-core/dropbear/{dropbear-2013.58 =>
> > dropbear-2013.60}/0007-dropbear-fix-for-x32-abi.patch (100%) delete mode
> > 100644 meta/recipes-core/dropbear/dropbear_2013.58.bb create mode 100644
> > meta/recipes-core/dropbear/dropbear_2013.60.bb
> >
> > diff --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0001-urandom-xauth-changes-to
> > -options.h.patch
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0001-urandom-xauth-changes-to
> > -options.h.patch similarity index 100%
> > rename from
> > meta/recipes-core/dropbear/dropbear-2013.58/0001-urandom-xauth-changes-to-o
> > ptions.h.patch rename to
> > meta/recipes-core/dropbear/dropbear-2013.60/0001-urandom-xauth-changes-to-o
> > ptions.h.patch diff --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0002-static_build_fix.patch
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0002-static_build_fix.patch
> > similarity index 100%
> > rename from
> > meta/recipes-core/dropbear/dropbear-2013.58/0002-static_build_fix.patch
> > rename to
> > meta/recipes-core/dropbear/dropbear-2013.60/0002-static_build_fix.patch
> > diff --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0003-configure.patch
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0003-configure.patch
> > similarity index 100%
> > rename from meta/recipes-core/dropbear/dropbear-2013.58/0003-configure.patch
> > rename to meta/recipes-core/dropbear/dropbear-2013.60/0003-configure.patch
> > diff --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0004-fix-2kb-keys.patch
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0004-fix-2kb-keys.patch
> > similarity index 100%
> > rename from
> > meta/recipes-core/dropbear/dropbear-2013.58/0004-fix-2kb-keys.patch rename
> > to meta/recipes-core/dropbear/dropbear-2013.60/0004-fix-2kb-keys.patch diff
> > --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0005-dropbear-enable-pam.patc
> > h
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0005-dropbear-enable-pam.patc
> > h similarity index 100%
> > rename from
> > meta/recipes-core/dropbear/dropbear-2013.58/0005-dropbear-enable-pam.patch
> > rename to
> > meta/recipes-core/dropbear/dropbear-2013.60/0005-dropbear-enable-pam.patch
> > diff --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0006-dropbear-configuration-f
> > ile.patch
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0006-dropbear-configuration-f
> > ile.patch similarity index 56%
> > rename from
> > meta/recipes-core/dropbear/dropbear-2013.58/0006-dropbear-configuration-fil
> > e.patch rename to
> > meta/recipes-core/dropbear/dropbear-2013.60/0006-dropbear-configuration-fil
> > e.patch index 7e38663..fa4c8d0 100644
> > ---
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0006-dropbear-configuration-f
> > ile.patch +++
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0006-dropbear-configuration-f
> > ile.patch @@ -1,28 +1,22 @@
> > Subject: [PATCH 6/6] dropbear configuration file
> >
> > dropbear: Change the path ("/etc/pam.d/sshd" as default) to find a pam
> > configuration file \ -to "/etc/pam.d/dropbear for dropbear when enabling
> > pam supporting" +to "/etc/pam.d/dropbear for dropbear when enabling pam
> > supporting"
> >
> > -Upstream-Status: Inappropriate [configuration]
> > +Upstream-Status: Inappropriate [configuration]
> >
> > +Signed-off-by: Maxin B. John <maxin.j...@enea.com>
> > Signed-off-by: Xiaofeng Yan <xiaofeng....@windriver.com>
> > ---
> > - svr-authpam.c | 2 +-
> > - 1 file changed, 1 insertion(+), 1 deletion(-)
> > -
> > -diff --git a/svr-authpam.c b/svr-authpam.c
> > -index e84f076..e28be7d 100644
> > ---- a/svr-authpam.c
> > -+++ b/svr-authpam.c
> > -@@ -195,7 +195,7 @@ void svr_auth_pam() {
> > +diff -Naur dropbear-2013.60-orig/svr-authpam.c
> > dropbear-2013.60/svr-authpam.c +---
> > dropbear-2013.60-orig/svr-authpam.c 2013-10-16 16:34:53.000000000 +0200
> > ++++ dropbear-2013.60/svr-authpam.c 2013-10-21 17:04:04.969416055 +0200 +@@
> > -211,7 +211,7 @@
> > userData.passwd = password;
> >
> > /* Init pam */
> > - if ((rc = pam_start("sshd", NULL, &pamConv, &pamHandlep)) !=
> > PAM_SUCCESS)
> > { + if ((rc = pam_start("dropbear", NULL, &pamConv, &pamHandlep)) !=
> > PAM_SUCCESS) { - dropbear_log(LOG_WARNING, "pam_start() failed,
> > rc=%d,
> > %s\n",
> > + dropbear_log(LOG_WARNING, "pam_start() failed, rc=%d, %s",
> > rc, pam_strerror(pamHandlep, rc));
> > goto cleanup;
> > ---
> > -1.7.11.7
> > -
> > diff --git
> > a/meta/recipes-core/dropbear/dropbear-2013.58/0007-dropbear-fix-for-x32-abi
> > .patch
> > b/meta/recipes-core/dropbear/dropbear-2013.60/0007-dropbear-fix-for-x32-abi
> > .patch similarity index 100%
> > rename from
> > meta/recipes-core/dropbear/dropbear-2013.58/0007-dropbear-fix-for-x32-abi.p
> > atch rename to
> > meta/recipes-core/dropbear/dropbear-2013.60/0007-dropbear-fix-for-x32-abi.p
> > atch diff --git a/meta/recipes-core/dropbear/dropbear.inc
> > b/meta/recipes-core/dropbear/dropbear.inc index 9498bf8..f71f190 100644
> > --- a/meta/recipes-core/dropbear/dropbear.inc
> > +++ b/meta/recipes-core/dropbear/dropbear.inc
> > @@ -2,12 +2,12 @@ DESCRIPTION = "Dropbear is a lightweight SSH and SCP
> > implementation" HOMEPAGE = "http://matt.ucc.asn.au/dropbear/dropbear.html";
> > SECTION = "console/network"
> >
> > -INC_PR = "r1"
> > +INC_PR = "r0"
> >
> > # some files are from other projects and have others license terms:
> > # public domain, OpenSSH 3.5p1, OpenSSH3.6.1p2, PuTTY
> > LICENSE = "MIT"
> > -LIC_FILES_CHKSUM = "file://LICENSE;md5=3a5b0c2f0d0c49dfde9558ae2036683c"
> > +LIC_FILES_CHKSUM = "file://LICENSE;md5=de74413ba7e05b7a37c4722e75f436cd"
> >
> > DEPENDS = "zlib"
> > RPROVIDES_${PN} = "ssh sshd"
> > diff --git a/meta/recipes-core/dropbear/dropbear_2013.58.bb
> > b/meta/recipes-core/dropbear/dropbear_2013.58.bb deleted file mode 100644
> > index 99a24a7..0000000
> > --- a/meta/recipes-core/dropbear/dropbear_2013.58.bb
> > +++ /dev/null
> > @@ -1,6 +0,0 @@
> > -require dropbear.inc
> > -
> > -SRC_URI[md5sum] = "8d3579bbbfcea48404a3219643076450"
> > -SRC_URI[sha256sum] =
> > "d866189b1d14e15353aeb4feb616f6132a63ea7c25ca8aa12b63a3a397822e55" -
> > -PR = "${INC_PR}.0"
> > diff --git a/meta/recipes-core/dropbear/dropbear_2013.60.bb
> > b/meta/recipes-core/dropbear/dropbear_2013.60.bb new file mode 100644
> > index 0000000..f95c3b5
> > --- /dev/null
> > +++ b/meta/recipes-core/dropbear/dropbear_2013.60.bb
> > @@ -0,0 +1,6 @@
> > +require dropbear.inc
> > +
> > +SRC_URI[md5sum] = "fffa2fec0c887201bed3a3cc6df7fa35"
> > +SRC_URI[sha256sum] =
> > "4c0c52453b4dd96ef79e5968ed3676de008ee0cf133632acae12bbad588681bb" +
> > +PR = "${INC_PR}.0"
>
> LIC_FILES_CHKSUM has been changed by this patch and yet no comment was made
> in
> the commit message mentioning why. In this instance it was simply the
> copyright year changing and in any case it seems the patch has already been
> merged; however in future could you please mention exactly what prompted the
> LIC_FILES_CHKSUM change in the commit message?
Sorry for not including that in the comment. I have checked the
license(s) and found that it was same as before. Since there were no
changes in the license, I just updated it.
>From now onwards, I will take care of it.
> Thanks,
> Paul
Best Regards,
Maxin
> --
>
> Paul Eggleton
> Intel Open Source Technology Centre
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
