Package: coreutils
Version: 8.5-1
Severity: minor
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu maverick ubuntu-patch

Hi Michael,

The latest coreutils package FTBFS in Ubuntu on armel, powerpc, and sparc
due to a reproducible failure in the gnulib testsuite (test-linkat).  We
haven't sorted out why the build failure was limited to these three
architectures, or why it didn't also affect Debian, but it's resolved by the
attached patch which corrects a missing header include that was noticed
while debugging.  With the prototypes in place, the package again passes
the test suite on all architectures.  So while the package isn't currently
failing to build in Debian, it might well do so in the future; and fixing
the missing prototypes is obviously correct, so I would suggest applying the
patch in Debian (and upstream) as well.

The accompanying changelog entry for this patch is:

  [ John Rigby ]
  * debian/patches/99_stat_prototype_for_linkat.dpatch: Add missing header
    include for stat() prototype.  LP: #591968.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                    http://www.debian.org/
slanga...@ubuntu.com                                     vor...@debian.org
=== modified file 'debian/patches/00list'
--- debian/patches/00list	2010-04-27 20:32:54 +0000
+++ debian/patches/00list	2010-06-11 01:38:10 +0000
@@ -4,3 +4,4 @@
 99_disable_tests.misc.ls-time
 99_kfbsd_build_tests
 99_kfbsd_fstat_patch
+99_stat_prototype_for_linkat

=== added file 'debian/patches/99_stat_prototype_for_linkat.dpatch'
--- debian/patches/99_stat_prototype_for_linkat.dpatch	1970-01-01 00:00:00 +0000
+++ debian/patches/99_stat_prototype_for_linkat.dpatch	2010-06-11 06:20:05 +0000
@@ -0,0 +1,18 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+# Author: John Rigby <john.ri...@canonical.com>
+# Bug-Ubuntu: https://bugs.launchpad.net/bugs/591968
+# Description: Add missing header include for stat() prototype
+
+...@dpatch@
+=== modified file 'gnulib-tests/test-linkat.c'
+--- coreutils.orig/gnulib-tests/test-linkat.c	2010-04-27 20:32:54 +0000
++++ coreutils/gnulib-tests/test-linkat.c	2010-06-10 16:11:38 +0000
+@@ -31,6 +31,7 @@
+ #include <stdio.h>
+ #include <stdlib.h>
+ #include <string.h>
++#include <sys/stat.h>
+ 
+ #include "areadlink.h"
+ #include "filenamecat.h"
+

Reply via email to