Summary: | meson.eclass changes made "virtx meson_src_test" not die anymore | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Mart Raudsepp <leio> |
Component: | Eclasses | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | floppym, sam |
Priority: | Normal | Keywords: | PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://github.com/gentoo/gentoo/pull/36284 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Mart Raudsepp
2024-04-16 19:51:04 UTC
A reduced test case would be helpful here; I would prefer not to install epiphany just to reproduce the failure. Real culprit is here: https://gitweb.gentoo.org/repo/gentoo.git/commit/eclass/meson.eclass?id=dbeb507f468dcc75095d54b59c165dcc70ef558d The popd calls will reset $? to 0. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=afe0521495aeed0a22a3656e7bad193fba39805e commit afe0521495aeed0a22a3656e7bad193fba39805e Author: Mike Gilbert <floppym@gentoo.org> AuthorDate: 2024-04-16 20:50:33 +0000 Commit: Mike Gilbert <floppym@gentoo.org> CommitDate: 2024-04-19 23:10:45 +0000 meson.eclass: preserve exit status in phase funcs When the functions are called with nonfatal, we need to ensure 'popd' does not clobber the exit status of the called command. Update meson_src_configure as well just for consistency. Closes: https://bugs.gentoo.org/930119 Signed-off-by: Mike Gilbert <floppym@gentoo.org> eclass/meson.eclass | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) |