[hurd,commited,2/3] hurd: Fix building io/tst-fchownat.c

Message ID 20180128172308.7647-3-samuel.thibault@ens-lyon.org
State New
Headers show
Series
  • hurd: fix building tests
Related show

Commit Message

Samuel Thibault Jan. 28, 2018, 5:23 p.m.
* io/tst-fchownat.c [!defined _POSIX_CHOWN_RESTRICTED]: Act like when
	_POSIX_CHOWN_RESTRICTED is defined to 0.
---
 ChangeLog         | 2 ++
 io/tst-fchownat.c | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

-- 
2.15.1

Comments

Andreas Schwab Jan. 28, 2018, 5:35 p.m. | #1
On Jan 28 2018, Samuel Thibault <samuel.thibault@ens-lyon.org> wrote:

> 	* io/tst-fchownat.c [!defined _POSIX_CHOWN_RESTRICTED]: Act like when

> 	_POSIX_CHOWN_RESTRICTED is defined to 0.


_POSIX_CHOWN_RESTRICTED must not be undefined (and undefined is _not_
the same as 0).  Fix the hurd header instead.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."
Samuel Thibault Jan. 28, 2018, 5:37 p.m. | #2
Andreas Schwab, on dim. 28 janv. 2018 18:35:37 +0100, wrote:
> On Jan 28 2018, Samuel Thibault <samuel.thibault@ens-lyon.org> wrote:

> 

> > 	* io/tst-fchownat.c [!defined _POSIX_CHOWN_RESTRICTED]: Act like when

> > 	_POSIX_CHOWN_RESTRICTED is defined to 0.

> 

> _POSIX_CHOWN_RESTRICTED must not be undefined (and undefined is _not_

> the same as 0).  Fix the hurd header instead.


So the #ifdef in posix/fpathconf.c should be dropped?

Samuel
Samuel Thibault Jan. 28, 2018, 6:15 p.m. | #3
Samuel Thibault, on dim. 28 janv. 2018 18:37:17 +0100, wrote:
> Andreas Schwab, on dim. 28 janv. 2018 18:35:37 +0100, wrote:

> > On Jan 28 2018, Samuel Thibault <samuel.thibault@ens-lyon.org> wrote:

> > 

> > > 	* io/tst-fchownat.c [!defined _POSIX_CHOWN_RESTRICTED]: Act like when

> > > 	_POSIX_CHOWN_RESTRICTED is defined to 0.

> > 

> > _POSIX_CHOWN_RESTRICTED must not be undefined (and undefined is _not_

> > the same as 0).  Fix the hurd header instead.

> 

> So the #ifdef in posix/fpathconf.c should be dropped?


I have reverted it.

Samuel

Patch

diff --git a/ChangeLog b/ChangeLog
index 23a56d69cd..23058435c4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -56,6 +56,8 @@ 
 	* libio/tst-memstream3.c (FWRITE): Rename to _FWRITE.
 	(do_test_bz20181): Rename accordingly.
 	* libio/tst-wmemstream3.c (FWRITE): Rename accordingly.
+	* io/tst-fchownat.c [!defined _POSIX_CHOWN_RESTRICTED]: Act like when
+	_POSIX_CHOWN_RESTRICTED is defined to 0.
 
 2008-12-18  Thomas Schwinge  <tschwinge@gnu.org>
 
diff --git a/io/tst-fchownat.c b/io/tst-fchownat.c
index e8adf6229f..8318842249 100644
--- a/io/tst-fchownat.c
+++ b/io/tst-fchownat.c
@@ -20,7 +20,7 @@  static int dir_fd;
 static void
 prepare (void)
 {
-#if _POSIX_CHOWN_RESTRICTED == 0
+#if !defined _POSIX_CHOWN_RESTRICTED || _POSIX_CHOWN_RESTRICTED == 0
   if (pathconf (test_dir, _PC_CHOWN_RESTRICTED) != 0)
 #endif
     {