This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Fix hang in ext/POSIX/t/sysconf.t on GNU/Hurd
authorPino Toscano <pino@debian.org>
Mon, 31 Oct 2011 21:37:04 +0000 (21:37 +0000)
committerNicholas Clark <nick@ccl4.org>
Fri, 4 Nov 2011 13:15:13 +0000 (14:15 +0100)
while compiling perl 5.14.2 on GNU/Hurd, I ran into what it seems a
undefined POSIX behaviour in ext/POSIX/t/sysconf.t.

      my $fd = POSIX::open($fifo, O_RDWR)
      or skip("could not open $fifo ($!)", 3 * @path_consts_fifo);

according to the POSIX open()[1] about O_RDWR,
  The result is undefined if this flag is applied to a FIFO.
.... which is actually our case.
Apparently Linux and *FreeBSD (and maybe also OSes) accept this
behaviour, but on GNU/Hurd this causes the open() call to block
undefinitely. Given there's nothing done with the FIFO if not querying
{,f}pathconf() values, the proposed solution I attached is to change
the opening mode to "O_RDONLY | O_NONBLOCK".

[1] http://pubs.opengroup.org/onlinepubs/9699919799/functions/open.html

ext/POSIX/t/sysconf.t

index 8008996..65625a8 100644 (file)
@@ -125,7 +125,7 @@ SKIP: {
        or skip("could not create fifo $fifo ($!)", 2 * 3 * @path_consts_fifo);
 
   SKIP: {
-      my $fd = POSIX::open($fifo, O_RDWR)
+      my $fd = POSIX::open($fifo, O_RDONLY | O_NONBLOCK)
          or skip("could not open $fifo ($!)", 3 * @path_consts_fifo);
 
       for my $constant (@path_consts_fifo) {