[OE-core] [PATCH 2/4] ninja: upgrade 1.9.0 -> 1.10.0

Tue Feb 25 19:56:58 UTC 2020

On Tue, Feb 25, 2020 at 11:47 AM Alexander Kanavin
<alex.kanavin at gmail.com> wrote:
>
> These are the commits produced monthly by a tool called Automated Upgrade Helper and sent directly to recipe maintainers as listed in maintainers.inc. When maintainers receive the patches, they're supposed to reset the author to themselves, but sometimes it doesn't happen.
>
> Search for 'auh' in oe-core list for monthly summaries of AUH work.

Thanks. The maintainers sometimes forgetting to reset the author was
the part I was missing. Maybe the patch checker could be updated to
check for that?

> Alex
>
> On Tue, 25 Feb 2020 at 20:43, Andre McCurdy <armccurdy at gmail.com> wrote:
>>
>> On Fri, Feb 21, 2020 at 2:46 PM Khem Raj <raj.khem at gmail.com> wrote:
>> >
>> > From: Upgrade Helper <auh at auh.yoctoproject.org>
>>
>> Why does the author sometimes get set to "Upgrade Helper"? It doesn't
>> seem to happen very often (4 times in the past 6 years in oe-core).
>> What's special about those 4 cases?
>>
>> I think it would be clearer if the author was set to whoever ran the tool.
>>
>> > Signed-off-by: Khem Raj <raj.khem at gmail.com>
>> > ---
>> >  .../ninja/ninja/fix-musl.patch                | 46 -------------------
>> >  .../ninja/{ninja_1.9.0.bb => ninja_1.10.0.bb} |  6 +--
>> >  2 files changed, 2 insertions(+), 50 deletions(-)
>> >  delete mode 100644 meta/recipes-devtools/ninja/ninja/fix-musl.patch
>> >  rename meta/recipes-devtools/ninja/{ninja_1.9.0.bb => ninja_1.10.0.bb} (87%)
>> >
>> > diff --git a/meta/recipes-devtools/ninja/ninja/fix-musl.patch b/meta/recipes-devtools/ninja/ninja/fix-musl.patch
>> > deleted file mode 100644
>> > index 7fd779110d..0000000000
>> > --- a/meta/recipes-devtools/ninja/ninja/fix-musl.patch
>> > +++ /dev/null
>> > @@ -1,46 +0,0 @@
>> > -From e92a915183f8bff268f74c8cd00d986301a2bbf8 Mon Sep 17 00:00:00 2001
>> > -From: makepost <makepost at firemail.cc>
>> > -Date: Mon, 24 Dec 2018 03:13:16 +0200
>> > -Subject: [PATCH] Use st_mtim if st_mtime is macro, fix #1510
>> > -
>> > -In POSIX.1-2008, sys_stat has a st_mtim member and a st_mtime backward
>> > -compatibility macro. Should help avoid hardcoding platform detection.
>> > -
>> > -Signed-off-by: Oleksandr Kravchuk <open.source at oleksandr-kravchuk.com>
>> > -
>> > -Upstream-Status: Pending
>> > ----
>> > - src/disk_interface.cc | 14 ++++----------
>> > - 1 file changed, 4 insertions(+), 10 deletions(-)
>> > -
>> > -diff --git a/src/disk_interface.cc b/src/disk_interface.cc
>> > -index d4c2fb0..dc297c4 100644
>> > ---- a/src/disk_interface.cc
>> > -+++ b/src/disk_interface.cc
>> > -@@ -202,19 +202,13 @@ TimeStamp RealDiskInterface::Stat(const string& path, string* err) const {
>> > -   // that it doesn't exist.
>> > -   if (st.st_mtime == 0)
>> > -     return 1;
>> > --#if defined(__APPLE__) && !defined(_POSIX_C_SOURCE)
>> > -+#if defined(_AIX)
>> > -+  return (int64_t)st.st_mtime * 1000000000LL + st.st_mtime_n;
>> > -+#elif defined(__APPLE__)
>> > -   return ((int64_t)st.st_mtimespec.tv_sec * 1000000000LL +
>> > -           st.st_mtimespec.tv_nsec);
>> > --#elif (_POSIX_C_SOURCE >= 200809L || _XOPEN_SOURCE >= 700 || defined(_BSD_SOURCE) || defined(_SVID_SOURCE) || \
>> > --       defined(__BIONIC__) || (defined (__SVR4) && defined (__sun)) || defined(__FreeBSD__))
>> > --  // For glibc, see "Timestamp files" in the Notes of http://www.kernel.org/doc/man-pages/online/pages/man2/stat.2.html
>> > --  // newlib, uClibc and musl follow the kernel (or Cygwin) headers and define the right macro values above.
>> > --  // For bsd, see https://github.com/freebsd/freebsd/blob/master/sys/sys/stat.h and similar
>> > --  // For bionic, C and POSIX API is always enabled.
>> > --  // For solaris, see https://docs.oracle.com/cd/E88353_01/html/E37841/stat-2.html.
>> > -+#elif defined(st_mtime) // A macro, so we're likely on modern POSIX.
>> > -   return (int64_t)st.st_mtim.tv_sec * 1000000000LL + st.st_mtim.tv_nsec;
>> > --#elif defined(_AIX)
>> > --  return (int64_t)st.st_mtime * 1000000000LL + st.st_mtime_n;
>> > - #else
>> > -   return (int64_t)st.st_mtime * 1000000000LL + st.st_mtimensec;
>> > - #endif
>> > ---
>> > -2.17.1
>> > -
>> > diff --git a/meta/recipes-devtools/ninja/ninja_1.9.0.bb b/meta/recipes-devtools/ninja/ninja_1.10.0.bb
>> > similarity index 87%
>> > rename from meta/recipes-devtools/ninja/ninja_1.9.0.bb
>> > rename to meta/recipes-devtools/ninja/ninja_1.10.0.bb
>> > index f1236e8ac8..bdc6365052 100644
>> > --- a/meta/recipes-devtools/ninja/ninja_1.9.0.bb
>> > +++ b/meta/recipes-devtools/ninja/ninja_1.10.0.bb
>> > @@ -5,11 +5,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=a81586a64ad4e476c791cda7e2f2c52e"
>> >
>> >  DEPENDS = "re2c-native ninja-native"
>> >
>> > -SRCREV = "b25c08bda4949192c69cea4cee057887341a2ffc"
>> > +SRCREV = "ed7f67040b370189d989adbd60ff8ea29957231f"
>> >
>> > -SRC_URI = "git://github.com/ninja-build/ninja.git;branch=release \
>> > -           file://fix-musl.patch \
>> > -"
>> > +SRC_URI = "git://github.com/ninja-build/ninja.git;branch=release"
>> >  UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>.*)"
>> >
>> >  S = "${WORKDIR}/git"
>> > --
>> > 2.25.1
>> >
>> > --
>> > _______________________________________________
>> > Openembedded-core mailing list
>> > Openembedded-core at lists.openembedded.org
>> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core at lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core