Util-Linux package development
 help / color / mirror / Atom feed
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

      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