From: Ruediger Meier <sweet_f_a@gmx.de>
To: Andreas Henriksson <andreas@fatal.se>
Cc: util-linux@vger.kernel.org
Subject: Re: [PATCH] build: uuidd requires linux now, uses signalfd
Date: Mon, 14 Mar 2016 12:38:58 +0100 [thread overview]
Message-ID: <201603141238.58351.sweet_f_a@gmx.de> (raw)
In-Reply-To: <20160314064255.GA19644@fatal.se>
On Monday 14 March 2016, Andreas Henriksson wrote:
> Hello Ruediger Meier.
>
> On Sun, Mar 13, 2016 at 09:45:44PM +0100, Ruediger Meier wrote:
> > On Sunday 13 March 2016, Andreas Henriksson wrote:
> > > Specify that uuidd can only be built on Linux now since
> >
> > Did you managed to build ipcs and ipcrm and do they work
> > correctly? Which system have you tried?
>
> No I have not. This was simply a reaction to the immediate problems
> found in the build daemon logs after uploading 2.28-rc1 to
> Debian Experimental, see
> https://buildd.debian.org/status/package.php?p=util-linux&suite=exper
>imental
>
Nice build logs!
BTW since you are running the tests with --verbose you could also force
--parallel=1 to not mix output of certain tests.
> It's possible additional fixes are needed. I don't run non-Linux
> myself.
Hurd has no WEXITED:
login-utils/sulogin.c:1073:35: error: 'WEXITED' undeclared
ret = waitid(P_ALL, 0, &status, WEXITED);
Maybe to be fixed like this (ifdef'ed)
https://lists.debian.org/debian-hurd/2013/07/msg00046.html
And I see my old question answered regarding 565964a9.
fdisk/bsd test fails on alpha because of
libfdisk/src/bsd.c:
#ifdef __alpha__
{BSD_FS_EXT2, "ext2"},
#else
{BSD_FS_MSDOS, "MS-DOS"},
#endif
Could be simply skipped for now like this:
diff --git a/tests/ts/fdisk/bsd b/tests/ts/fdisk/bsd
index cd7a6c6..386402a 100755
--- a/tests/ts/fdisk/bsd
+++ b/tests/ts/fdisk/bsd
@@ -137,4 +137,5 @@ echo -e "${FDISK_CMD_BSD_LIST_TYPES}" | \
fdisk_bsd_offset_and_byte_order_clean
ts_fdisk_clean ${TEST_IMAGE_NAME}
+grep "3 Version 7 8 ext2 c ISO-9660 10 AdvFS" $TS_OUTPUT && ts_skip "alpha undefined"
ts_finalize
cu,
Rudi
prev parent reply other threads:[~2016-03-14 11:39 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-13 17:56 [PATCH] build: uuidd requires linux now, uses signalfd Andreas Henriksson
2016-03-13 20:45 ` Ruediger Meier
2016-03-14 6:42 ` Andreas Henriksson
2016-03-14 11:38 ` Ruediger Meier [this message]
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=201603141238.58351.sweet_f_a@gmx.de \
--to=sweet_f_a@gmx.de \
--cc=andreas@fatal.se \
--cc=util-linux@vger.kernel.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