Re: timespec declaration issue on Tru64

2008-12-15 Thread Bruno Haible
Daniel Richard G. wrote:
> I found that adding the following line makes the problem go away:
> 
> begin sig2str.c snippet
> #include 
> 
> #include 
> #include < skadoosh
> #include 
> #include 
> #include 
> #include 
> 
> #include "sig2str.h"
> end sig2str.c snippet

This looked weird enough that I looked into it as well. I found the culprit
to be the  override in gnulib. This file is being included
from the system's  and  and itself includes
 and , leading to some half-included files. I'm
applying this fix. Thanks for the report!

> The only conclusion I draw is that Tru64 is a PITA to deal with :-)

Well, this time, gnulib was at fault.

Bruno


2008-12-15  Bruno Haible  

Fix compilation error on OSF/1 4.0.
* lib/sys_select.in.h: When invoked from OSF/1  or
, simply delegate to the system header.
Reported by Daniel Richard G. .

*** lib/sys_select.in.h.orig2008-12-15 13:39:04.0 +0100
--- lib/sys_select.in.h 2008-12-15 13:36:20.0 +0100
***
*** 15,28 
 along with this program; if not, write to the Free Software Foundation,
 Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.  */
  
- #ifndef _GL_SYS_SELECT_H
- 
- #if @HAVE_SYS_SELECT_H@
- 
  # if __GNUC__ >= 3
  @PRAGMA_SYSTEM_HEADER@
  # endif
  
  /* On many platforms,  assumes prior inclusion of
 .  */
  # include 
--- 15,42 
 along with this program; if not, write to the Free Software Foundation,
 Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.  */
  
  # if __GNUC__ >= 3
  @PRAGMA_SYSTEM_HEADER@
  # endif
  
+ /* On OSF/1,  and  include .
+Simply delegate to the system's header in this case.  */
+ #if @HAVE_SYS_SELECT_H@ && defined __osf__ && (defined _SYS_TYPES_H_ && 
!defined _GL_SYS_SELECT_H_REDIRECT_FROM_SYS_TYPES_H) && defined _OSF_SOURCE
+ 
+ # define _GL_SYS_SELECT_H_REDIRECT_FROM_SYS_TYPES_H
+ # @INCLUDE_NEXT@ @NEXT_SYS_SELECT_H@
+ 
+ #elif @HAVE_SYS_SELECT_H@ && defined __osf__ && (defined _SYS_TIME_H_ && 
!defined _GL_SYS_SELECT_H_REDIRECT_FROM_SYS_TIME_H) && defined _OSF_SOURCE
+ 
+ # define _GL_SYS_SELECT_H_REDIRECT_FROM_SYS_TIME_H
+ # @INCLUDE_NEXT@ @NEXT_SYS_SELECT_H@
+ 
+ #else
+ 
+ #ifndef _GL_SYS_SELECT_H
+ 
+ #if @HAVE_SYS_SELECT_H@
+ 
  /* On many platforms,  assumes prior inclusion of
 .  */
  # include 
***
*** 76,78 
--- 90,93 
  
  #endif /* _GL_SYS_SELECT_H */
  #endif /* _GL_SYS_SELECT_H */
+ #endif /* OSF/1 */


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils


git-diff shows none, even before ./bootstrap

2008-12-15 Thread jidanni
Following the instructions in Reading README-hacking,
$ ./bootstrap 
Error: 'autoconf' not found ...
Well, rather then installing all those boring packages, I just skipped down to
> At this point, there should be no difference between your local copy,
> and the GIT master copy:
>$ git diff
> should output no difference.
Ha ha, it shows no difference already,
$ git-diff|wc
  0   0   0
and you wanted to fool me into installing all those debs just to make
a documentation patch! So README-hacking had better say better what
the goal of those steps I skipped are. I mean, taking those steps
could only make my perfect 0 0 0 less perfect. P.S. I used git-clone
--depth 1 as you never know how big it will be without looking in the
docs first, which you have to download first:
http://article.gmane.org/gmane.comp.version-control.git/103218


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils


say in ChangeLog-2008 why dead

2008-12-15 Thread jidanni
In HACKING:
 No more ChangeLog files
 ===
 Do not modify any of the ChangeLog files in coreutils.  Starting in
 2008, the policy changed.
OK, but you better put a note about that at the top of ChangeLog-2008,
else it looks like somebody died.


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils


better figure out a paperless copyright assignment method

2008-12-15 Thread jidanni
 Copyright assignment
 
 ...of an actual sheet of paper...

Sorry, I'm on a non-USA Pacific island mountaintop with no paper
business... anyway, got pictures of RMS and me on my website, so he'll
have to vouch for me.


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils


Re: git-diff shows none, even before ./bootstrap

2008-12-15 Thread Jim Meyering
jida...@jidanni.org wrote:
> Following the instructions in Reading README-hacking,
> $ ./bootstrap
> Error: 'autoconf' not found ...
> Well, rather then installing all those boring packages, I just skipped down to
>> At this point, there should be no difference between your local copy,
>> and the GIT master copy:
>>$ git diff
>> should output no difference.
> Ha ha, it shows no difference already,
> $ git-diff|wc
>   0   0   0
> and you wanted to fool me into installing all those debs just to make
> a documentation patch! So README-hacking had better say better what

;-)
Almost worked, eh?

> the goal of those steps I skipped are. I mean, taking those steps
> could only make my perfect 0 0 0 less perfect. P.S. I used git-clone
> --depth 1 as you never know how big it will be without looking in the

All those pesky makefile rules...
Running "make check" is useful even for documentation changes.
There are some makefile rules that will check for
common policy/syntax-related issues even in the *.texi files.


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils


[PATCH] du: -H now does what POSIX requires

2008-12-15 Thread Jim Meyering
Long ago, before POSIX took du's -H option, GNU du used it as a synonym
for --si (and analogous to --human's -h).  Once POSIX usurped the
short-named -H, GNU du deprecated, it and now it's been long enough that
we can change the meaning to align with POSIX.

>From d2892d80218e78ec8b5e962b0a6d0ce02f2abc91 Mon Sep 17 00:00:00 2001
From: Jim Meyering 
Date: Tue, 9 Dec 2008 08:22:21 +0100
Subject: [PATCH] du: -H now does what POSIX requires

* src/du.c (usage): Update --help output.
(main): Move -H-handling code from the --si block to
the one for --dereference-args (-D).
* doc/coreutils.texi (du invocation): Update description.
* NEWS (Changes in behavior): Mention this.
---
 NEWS   |3 +++
 doc/coreutils.texi |5 +
 src/du.c   |   13 -
 3 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/NEWS b/NEWS
index ca87b23..717af04 100644
--- a/NEWS
+++ b/NEWS
@@ -32,6 +32,9 @@ GNU coreutils NEWS-*- 
outline -*-
   cp and mv: the --reply={yes,no,query} option has been removed.
   Using it has elicited a warning for the last three years.

+  du: -H (initially equivalent to --si) is now equivalent to
+  --dereference-args, and thus works as POSIX requires
+
   ls -l now marks SELinux-only files with the less obtrusive '.',
   rather than '+'.  A file with any other combination of MAC and ACL
   is still marked with a '+'.
diff --git a/doc/coreutils.texi b/doc/coreutils.texi
index 983a694..8bbafe9 100644
--- a/doc/coreutils.texi
+++ b/doc/coreutils.texi
@@ -10016,10 +10016,7 @@ du invocation

 @item -H
 @opindex -H
-Currently, @option{-H} is the same as @option{--si},
-except that @option{-H} evokes a warning.
-This option will be changed to be equivalent to
-...@option{--dereference-args} (@option{-D}).
+Equivalent to @option{--dereference-args} (@option{-D}).

 @item -k
 @opindex -k
diff --git a/src/du.c b/src/du.c
index 6e4d28b..860e8fe 100644
--- a/src/du.c
+++ b/src/du.c
@@ -294,8 +294,7 @@ Mandatory arguments to long options are mandatory for short 
options too.\n\
   --files0-from=F   summarize disk usage of the NUL-terminated file\n\
   names specified in file F;\n\
   If F is - then read names from standard input\n\
-  -Hlike --si, but also evokes a warning; will soon\n\
-  change to be equivalent to --dereference-args (-D)\n\
+  -Hequivalent to --dereference-args (-D)\n\
   -h, --human-readable  print sizes in human readable format (e.g., 1K 234M 
2G)\n\
   --si  like -h, but use powers of 1000 not 1024\n\
 "), stdout);
@@ -729,11 +728,6 @@ main (int argc, char **argv)
  output_block_size = 1;
  break;

-   case 'H':  /* FIXME: remove warning and move this "case 'H'" to
- precede --dereference-args in late 2006.  */
- error (0, 0, _("WARNING: use --si, not -H; the meaning of the -H\
- option will soon\nchange to be the same as that of --dereference-args (-D)"));
- /* fall through */
case HUMAN_SI_OPTION:
  human_output_opts = human_autoscale | human_SI;
  output_block_size = 1;
@@ -762,7 +756,7 @@ main (int argc, char **argv)
  }
  break;

-   case MEGABYTES_LONG_OPTION:
+   case MEGABYTES_LONG_OPTION: /* FIXME: remove in 2009 */
  error (0, 0,
 _("the --megabytes option is deprecated; use -m instead"));
  /* fall through */
@@ -792,7 +786,8 @@ main (int argc, char **argv)
  }
  break;

-   case 'D': /* This will eventually be 'H' (-H), too.  */
+   case 'H':  /* NOTE: before 2008-12, -H was equivalent to --si.  */
+   case 'D':
  symlink_deref_bits = FTS_COMFOLLOW | FTS_PHYSICAL;
  break;

--
1.6.1.rc2.312.g2e96


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils