[Buildroot] [PATCH 1/1] package/micropython: drop patch

Chris Packham judge.packham at gmail.com
Fri Oct 18 07:26:22 UTC 2019


Hi Fabrice,

On Fri, Oct 18, 2019 at 5:52 AM Fabrice Fontaine
<fontaine.fabrice at gmail.com> wrote:
>
> Drop patch and set GIT_DIR as suggested by upstream during review of an
> upstreamable solution, see
> https://github.com/micropython/micropython/pull/5002
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>

Thanks for taking the time to look at this. I take it this triggers
the get_version_info_from_docs_conf fallback, which Is what I was
going with in the original patch.

Reviewed-by: Chris Packham <judge.packham at gmail.com>

> ---
>  package/micropython/0001-fix-version.patch | 22 ----------------------
>  package/micropython/micropython.mk         | 11 ++++++++---
>  2 files changed, 8 insertions(+), 25 deletions(-)
>  delete mode 100644 package/micropython/0001-fix-version.patch
>
> diff --git a/package/micropython/0001-fix-version.patch b/package/micropython/0001-fix-version.patch
> deleted file mode 100644
> index 1bf2992169..0000000000
> --- a/package/micropython/0001-fix-version.patch
> +++ /dev/null
> @@ -1,22 +0,0 @@
> -Always use the fallback behaviour for determining the version number
> -
> -When built from within buildroot the micropython build process detects
> -that it is inside a git repository and ends up using the buildroot
> -version number instead of the correct micropython version number.
> -
> -Signed-off-by: Chris Packham <judge.packham at gmail.com>
> ----
> -diff -Naur micropython-v1.4.5-old/py/makeversionhdr.py micropython-v1.4.5/py/makeversionhdr.py
> ---- micropython-v1.4.5-old/py/makeversionhdr.py        2015-08-12 00:42:30.000000000 +1200
> -+++ micropython-v1.4.5/py/makeversionhdr.py    2015-09-16 10:21:06.698962836 +1200
> -@@ -70,9 +70,7 @@
> -
> - def make_version_header(filename):
> -     # Get version info using git, with fallback to docs/conf.py
> --    info = get_version_info_from_git()
> --    if info is None:
> --        info = get_version_info_from_docs_conf()
> -+    info = get_version_info_from_docs_conf()
> -
> -     git_tag, git_hash, ver = info
> -
> diff --git a/package/micropython/micropython.mk b/package/micropython/micropython.mk
> index d636d0e869..5824b4ab18 100644
> --- a/package/micropython/micropython.mk
> +++ b/package/micropython/micropython.mk
> @@ -10,6 +10,11 @@ MICROPYTHON_LICENSE = MIT
>  MICROPYTHON_LICENSE_FILES = LICENSE
>  MICROPYTHON_DEPENDENCIES = host-pkgconf libffi
>
> +# Set GIT_DIR so package won't use buildroot's version number
> +MICROPYTHON_MAKE_ENV = \
> +       $(TARGET_MAKE_ENV) \
> +       GIT_DIR=.
> +
>  # Use fallback implementation for exception handling on architectures that don't
>  # have explicit support.
>  ifeq ($(BR2_i386)$(BR2_x86_64)$(BR2_arm)$(BR2_armeb),)
> @@ -22,15 +27,15 @@ MICROPYTHON_MAKE_OPTS = MICROPY_PY_BTREE=0
>  MICROPYTHON_MAKE_OPTS += MICROPY_PY_USSL=0
>
>  define MICROPYTHON_BUILD_CMDS
> -       $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/mpy-cross
> -       $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/ports/unix \
> +       $(MICROPYTHON_MAKE_ENV) $(MAKE) -C $(@D)/mpy-cross
> +       $(MICROPYTHON_MAKE_ENV) $(MAKE) -C $(@D)/ports/unix \
>                 $(MICROPYTHON_MAKE_OPTS) \
>                 CROSS_COMPILE=$(TARGET_CROSS) \
>                 CFLAGS_EXTRA=$(MICROPYTHON_CFLAGS)
>  endef
>
>  define MICROPYTHON_INSTALL_TARGET_CMDS
> -       $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/ports/unix \
> +       $(MICROPYTHON_MAKE_ENV) $(MAKE) -C $(@D)/ports/unix \
>                 $(MICROPYTHON_MAKE_OPTS) \
>                 CROSS_COMPILE=$(TARGET_CROSS) \
>                 CFLAGS_EXTRA=$(MICROPYTHON_CFLAGS) \
> --
> 2.23.0
>



More information about the buildroot mailing list