From: Johnnynator <Johnnynator@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] meson: update to 0.58.1.
Date: Thu, 17 Jun 2021 09:13:47 +0200 [thread overview]
Message-ID: <20210617071347._X6BtRFQTP8XRyNyQyqljYt7tnst_-_Juu7RxG_9mLY@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-31529@inbox.vuxu.org>
[-- Attachment #1: Type: text/plain, Size: 1571 bytes --]
There is an updated pull request by Johnnynator against master on the void-packages repository
https://github.com/Johnnynator/void-packages meson
https://github.com/void-linux/void-packages/pull/31529
meson: update to 0.58.1.
<!-- Mark items with [x] where applicable -->
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/31529.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-meson-31529.patch --]
[-- Type: text/x-diff, Size: 7422 bytes --]
From b41c6651a42d91b8ee8a20408ca07e71b5b7c9b7 Mon Sep 17 00:00:00 2001
From: John <me@johnnynator.dev>
Date: Wed, 16 Jun 2021 20:21:29 +0200
Subject: [PATCH] meson: update to 0.58.1.
---
.../dont-use-native-pkgconfig-for-gir.patch | 20 ++---
srcpkgs/meson/patches/fix-unittest.patch | 77 -------------------
srcpkgs/meson/template | 4 +-
3 files changed, 12 insertions(+), 89 deletions(-)
delete mode 100644 srcpkgs/meson/patches/fix-unittest.patch
diff --git a/srcpkgs/meson/patches/dont-use-native-pkgconfig-for-gir.patch b/srcpkgs/meson/patches/dont-use-native-pkgconfig-for-gir.patch
index 27253a71bc7f..6203e82095d0 100644
--- a/srcpkgs/meson/patches/dont-use-native-pkgconfig-for-gir.patch
+++ b/srcpkgs/meson/patches/dont-use-native-pkgconfig-for-gir.patch
@@ -1,11 +1,11 @@
---- mesonbuild/modules/gnome.py 2020-06-29 19:00:15.000000000 +0200
-+++ - 2020-07-13 12:48:58.087313471 +0200
-@@ -403,7 +403,7 @@
+--- mesonbuild/modules/gnome.py 2021-06-07 17:35:31.000000000 +0000
++++ - 2021-06-17 07:12:07.442352693 +0000
+@@ -87,7 +87,7 @@
+ once=True)
+
+ def _get_native_dep(self, state, depname, required=True):
+- kwargs = {'native': True, 'required': required}
++ kwargs = {'native': False, 'required': required}
+ holder = self.interpreter.func_dependency(state.current_node, [depname], kwargs)
+ return holder.held_object
- def _get_gir_dep(self, state):
- if not self.gir_dep:
-- kwargs = {'native': True, 'required': True}
-+ kwargs = {'native': False, 'required': True}
- holder = self.interpreter.func_dependency(state.current_node, ['gobject-introspection-1.0'], kwargs)
- self.gir_dep = holder.held_object
- giscanner = state.environment.lookup_binary_entry(MachineChoice.HOST, 'g-ir-scanner')
diff --git a/srcpkgs/meson/patches/fix-unittest.patch b/srcpkgs/meson/patches/fix-unittest.patch
deleted file mode 100644
index 746de056ed72..000000000000
--- a/srcpkgs/meson/patches/fix-unittest.patch
+++ /dev/null
@@ -1,77 +0,0 @@
-commit 777cbe9c0faa018bafccf3e7adb8dad162276e35
-Author: Érico Rolim <erico.erc@gmail.com>
-Date: Mon Feb 1 00:26:49 2021 -0300
-
- tests/common: fix " 37 has function" test on musl systems.
-
- This commit fixes the test that asserts on whether the lchmod() function
- should have been detected as available by Meson. It does so by assuming
- that on Linux systems not using glibc, the function will be available.
-
- - fix comment about lchmod on Linux: musl has implemented the function
- correctly since 2013, so the assumption in the test wasn't correct.
- Furthermore, musl doesn't use glibc's stub mechanism.
- - fix include to receive __GLIBC__ definition: including almost any
- header in glibc will end up defining __GLIBC__, since most headers
- include <features.h>. The <gnu/libc-version.h> header was probably
- chosen because of its name, but its actual purpose is defining functions
- for checking glibc version at runtime (instead of what the binary was
- built with), so it isn't necessary to use it. Since it is a completely
- non standard header, including it makes the test suite fail on musl due
- to not finding the header.
-
-diff --git test cases/common/37 has function/meson.build test cases/common/37 has function/meson.build
-index 26f13d6fa..a59480c2d 100644
---- test cases/common/39 has function/meson.build
-+++ test cases/common/39 has function/meson.build
-@@ -44,11 +44,13 @@ foreach cc : compilers
- error('Found non-existent function "hfkerhisadf".')
- endif
-
-- # With glibc on Linux lchmod is a stub that will always return an error,
-- # we want to detect that and declare that the function is not available.
-- # We can't check for the C library used here of course, but if it's not
-- # implemented in glibc it's probably not implemented in any other 'slimmer'
-- # C library variants either, so the check should be safe either way hopefully.
-+ # With glibc (before 2.32, see below) on Linux, lchmod is a stub that will
-+ # always return an error, we want to detect that and declare that the
-+ # function is not available.
-+ # We can't check for the C library used here of course, but the main
-+ # alternative Linux C library (musl) doesn't use glibc's stub mechanism;
-+ # also, it has implemented lchmod since 2013, so it should be safe to check
-+ # that lchmod is available on Linux when not using glibc.
- if host_system == 'linux' or host_system == 'darwin'
- assert (cc.has_function('poll', prefix : '#include <poll.h>',
- args : unit_test_args),
-@@ -57,15 +59,24 @@ foreach cc : compilers
- has_lchmod = cc.has_function('lchmod', prefix : lchmod_prefix, args : unit_test_args)
-
- if host_system == 'linux'
-- glibc_major = cc.get_define('__GLIBC__', prefix: '#include <gnu/libc-version.h>', args: unit_test_args)
-- glibc_minor = cc.get_define('__GLIBC_MINOR__', prefix: '#include <gnu/libc-version.h>', args: unit_test_args)
-- glibc_vers = '@0@.@1@'.format(glibc_major, glibc_minor)
-- message('GLIBC vetsion:', glibc_vers)
-+ # __GLIBC__ macro can be retrieved by including almost any C library header
-+ glibc_major = cc.get_define('__GLIBC__', prefix: '#include <unistd.h>', args: unit_test_args)
-+ # __GLIBC__ will only be set for glibc
-+ if glibc_major != ''
-+ glibc_print = 'hi "@0@" hi'.format(glibc_major)
-+ message(glibc_print)
-+ glibc_minor = cc.get_define('__GLIBC_MINOR__', prefix: '#include <unistd.h>', args: unit_test_args)
-+ glibc_vers = '@0@.@1@'.format(glibc_major, glibc_minor)
-+ message('GLIBC version:', glibc_vers)
-
-- # lchmod was implemented in glibc 2.32 (https://sourceware.org/pipermail/libc-announce/2020/000029.html)
-- if glibc_vers.version_compare('<2.32')
-- assert (not has_lchmod, '"lchmod" check should have failed')
-+ # lchmod was implemented in glibc 2.32 (https://sourceware.org/pipermail/libc-announce/2020/000029.html)
-+ if glibc_vers.version_compare('<2.32')
-+ assert (not has_lchmod, '"lchmod" check should have failed')
-+ else
-+ assert (has_lchmod, '"lchmod" check should have succeeded')
-+ endif
- else
-+ # Other C libraries for Linux should have lchmod
- assert (has_lchmod, '"lchmod" check should have succeeded')
- endif
- else
diff --git a/srcpkgs/meson/template b/srcpkgs/meson/template
index 64a12e6d7a98..a059488dabf8 100644
--- a/srcpkgs/meson/template
+++ b/srcpkgs/meson/template
@@ -1,6 +1,6 @@
# Template file for 'meson'
pkgname=meson
-version=0.56.2
+version=0.58.1
revision=1
build_style=python3-module
hostmakedepends="python3-devel python3-setuptools"
@@ -14,7 +14,7 @@ license="Apache-2.0"
homepage="https://mesonbuild.com"
changelog="https://github.com/mesonbuild/meson/blob/master/docs/markdown/Release-notes-for-${version%.*}.0.md"
distfiles="https://github.com/mesonbuild/meson/releases/download/$version/$pkgname-$version.tar.gz"
-checksum=3cb8bdb91383f7f8da642f916e4c44066a29262caa499341e2880f010edb87f4
+checksum=3144a3da662fcf79f1e5602fa929f2821cba4eba28c2c923fe0a7d3e3db04d5d
# XXX: sanitizers aren't available on musl
if [ "$XBPS_TARGET_LIBC" = glibc ]; then
next prev parent reply other threads:[~2021-06-17 7:13 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-16 19:29 [PR PATCH] " Johnnynator
2021-06-17 1:36 ` ericonr
2021-06-17 1:37 ` ericonr
2021-06-17 7:00 ` Johnnynator
2021-06-17 7:13 ` Johnnynator [this message]
2021-06-17 7:33 ` [PR PATCH] [Merged]: " Johnnynator
2021-07-17 22:21 [PR PATCH] " paper42
2021-07-18 21:34 ` [PR PATCH] [Updated] " paper42
2021-07-20 9:29 ` paper42
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210617071347._X6BtRFQTP8XRyNyQyqljYt7tnst_-_Juu7RxG_9mLY@z \
--to=johnnynator@users.noreply.github.com \
--cc=ml@inbox.vuxu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).