Re: cvs access to gnulib?

2012-01-17 Thread Ben Pfaff
k...@freefriends.org (Karl Berry) writes: > Alternatively, is it possible to tell git "I only want to clone one > subdirectory" (namely build-aux)? All the incantations I tried failed. If you just want an up-to-date copy of the build-aux directory, then the following works for me: git a

Re: cvs access to gnulib?

2012-01-17 Thread Bruno Haible
Hi Karl, > Was the cvs access to git intentionally shut down? > (Sorry if I missed the announcement.) > As of some time ago -- looks like it might have been six months, call me > late! -- I am seeing: > > cvs [update aborted]: connect to [pserver.git.sv.gnu.org]:2401 failed: > Connection

cvs access to gnulib?

2012-01-17 Thread Karl Berry
Was the cvs access to git intentionally shut down? (Sorry if I missed the announcement.) As of some time ago -- looks like it might have been six months, call me late! -- I am seeing: cvs [update aborted]: connect to [pserver.git.sv.gnu.org]:2401 failed: Connection timed out That is, :pse

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Bruno Haible
Paul Eggert wrote: > The "cat INSTALL" argument is not that persuasive The other argument is that when viewing a file with "vi" and scrolling through it, the cursor interrupts the reading flow when it is placed on the last line and that line is not a blank one. But of course, these arguments are

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Paul Eggert
On 01/17/12 10:44, Jim Meyering wrote: > Exempting three more is no problem. It's no big deal but I'd rather omit the trailing empty line. The "cat INSTALL" argument is not that persuasive, as the sort of person who types "cat INSTALL" or "more INSTALL" is already used to files that don't end in

[PATCH] changelog: don't cluster multiple entries under the same "date line" (was: Re: [Feature request] gitlog-to-changelog: don't cluster multiple ...)

2012-01-17 Thread Stefano Lattarini
On 01/17/2012 10:59 PM, Joel E. Denny wrote: > On Tue, 17 Jan 2012, Jim Meyering wrote: > >>> Below is a patch that adds a --no-cluster option, which I believe does >>> what Stefano wants. I want it too. OK to push? >> >> Hi Joel, >> >> Thanks. That looks fine, but please adjust the preceding c

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Karl Berry
However, I am optimistic that I will be able to make matching changes upstream. Sorry, but for myself, I think it is a terrible waste of time to be thinking about this for fdl*.texi. The blank lines don't hurt anything and you're not supposed to be modifying those files, so can't we just

Re: [Feature request] gitlog-to-changelog: don't cluster multiple ChangeLog entries under the same "date line"

2012-01-17 Thread Joel E. Denny
On Tue, 17 Jan 2012, Jim Meyering wrote: > > Below is a patch that adds a --no-cluster option, which I believe does > > what Stefano wants. I want it too. OK to push? > > Hi Joel, > > Thanks. That looks fine, but please adjust the preceding comment > to keep in sync with the new behavior. Th

Re: [Feature request] gitlog-to-changelog: don't cluster multiple ChangeLog entries under the same "date line"

2012-01-17 Thread Joel E. Denny
Hi Jim, On Mon, 26 Dec 2011, Jim Meyering wrote: > I think the result is an improvement, but it is still not as readable as I > would like, or maybe it's just not as readable as I am used to ("git log" > output). Why? ... Because when some clumped entries are adjacent to a > multi-paragraph one

Re: [Feature request] gitlog-to-changelog: don't cluster multiple ChangeLog entries under the same "date line"

2012-01-17 Thread Jim Meyering
Joel E. Denny wrote: > On Mon, 26 Dec 2011, Jim Meyering wrote: >> I think the result is an improvement, but it is still not as readable as I >> would like, or maybe it's just not as readable as I am used to ("git log" >> output). Why? ... Because when some clumped entries are adjacent to a >> mu

Re: canonicalize_file_name does not support MS-Windows style file names

2012-01-17 Thread Eli Zaretskii
> From: Bruno Haible > Bcc: br...@haible.de > Date: Tue, 17 Jan 2012 20:07:11 +0100 > > Thanks, I'll look into integrating it. Thanks, looking forward to it. > Do you also have a set of Windows-syntax filenames available on which you > tested it? So that we can extend the unit test. I don't ha

Re: canonicalize_file_name does not support MS-Windows style file names

2012-01-17 Thread Bruno Haible
Hi Eli, > With the patch below, it does. Thanks, I'll look into integrating it. Do you also have a set of Windows-syntax filenames available on which you tested it? So that we can extend the unit test. Btw, this is your first major contribution to Gnulib. Would you mind starting the copyright a

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Bruno Haible
Jim Meyering wrote: > >> * doc/INSTALL: Likewise. > >> * doc/INSTALL.ISO: Likewise. > >> * doc/INSTALL.UTF-8: Likewise. > Exempting three more is no problem. Thanks! Bruno

Re: libvirt 0.9.9 doesn't build on OpenBSD 5.0-current (i386)

2012-01-17 Thread Jiri B
On Tue, Jan 17, 2012 at 07:16:34PM +0100, Bruno Haible wrote: > > Log and tarball of requested files in attachment. > > Thanks, but the allocator.i file is unusable. It contains two commands > instead of a preprocessed output. I need the result of the command > > gcc -std=gnu99 -DHAVE_CONFIG_H -I

Re: sc_prohibit_*_without_use

2012-01-17 Thread Jim Meyering
Bruno Haible wrote: > Would it be possible to generalize the sc_prohibit_*_without_use rules > to more header files easily? > > # Don't include this header unless you use one of its functions. > sc_prohibit_error_without_use: > @h='error.h' \ > re='\ *\('\ > $(_sc_header_w

sc_prohibit_*_without_use

2012-01-17 Thread Bruno Haible
Hi Jim, Would it be possible to generalize the sc_prohibit_*_without_use rules to more header files easily? # Don't include this header unless you use one of its functions. sc_prohibit_error_without_use: @h='error.h' \ re='\http://git.savannah.gnu.org/gitweb/?p=libunistring.git;a=

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Jim Meyering
Bruno Haible wrote: > Hi Jim, > >> * doc/regexprops-generic.texi: Add a newline at EOF. There was none. > > Good point. It was more of a problem with 'cvs' than with 'git'. But I > like this effort anyway. > >> * doc/Copyright/assign.translation.manual: Remove empty lines at EOF. >> * doc/Copyrig

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Bruno Haible
Hi Jim, > * doc/regexprops-generic.texi: Add a newline at EOF. There was none. Good point. It was more of a problem with 'cvs' than with 'git'. But I like this effort anyway. > * doc/Copyright/assign.translation.manual: Remove empty lines at EOF. > * doc/Copyright/request-assign.future: Likewis

canonicalize_file_name does not support MS-Windows style file names

2012-01-17 Thread Eli Zaretskii
With the patch below, it does. HTH --- lib/canonicalize-lgpl.c~0 2011-10-22 16:19:34.0 +0200 +++ lib/canonicalize-lgpl.c 2012-01-17 13:10:52.608428500 +0200 @@ -51,6 +51,7 @@ # define __realpath realpath # include "pathmax.h" # include "malloca.h" +# include "dosname.h" # if HAV

Re: libvirt 0.9.9 doesn't build on OpenBSD 5.0-current (i386)

2012-01-17 Thread Bruno Haible
Hi Jiri, > Log and tarball of requested files in attachment. Thanks, but the allocator.i file is unusable. It contains two commands instead of a preprocessed output. I need the result of the command gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../.. -I/usr/include -I/usr/local/include -g -O2 -E allocat

[PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Jim Meyering
From: Jim Meyering Apply syntax rules at home as well as abroad. Most changes were induced by running this: make srcdir=. _build-aux=build-aux -f top/maint.mk \ sc_prohibit_empty_lines_at_EOF | grep -v modules/ \ | xargs perl -pi -0777 -e 's/\n\n+$/\n/' * cfg.mk (local-checks-to-skip):

[PATCH 4/8] maint: enable sc_prohibit_hash_pjw_without_use

2012-01-17 Thread Jim Meyering
From: Jim Meyering * cfg.mk: Enable sc_prohibit_hash_pjw_without_use. * top/maint.mk (sc_prohibit_hash_pjw_without_use): Adjust regexp to match any use of \, i.e., not necessarily with a following " (". --- ChangeLog|6 ++ cfg.mk |1 - top/maint.mk |2 +- 3 files change

[PATCH 3/8] maint: enable double-word-prohibiting rule

2012-01-17 Thread Jim Meyering
From: Jim Meyering * cfg.mk (local-checks-to-skip): Enable sc_prohibit_doubled_word. Exempt three files. --- ChangeLog |5 + cfg.mk|3 ++- 2 files changed, 7 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index fa86239..2873ddb 100644 --- a/ChangeLog +++ b/Chan

[PATCH 1/8] maint: add framework to run syntax-check rules against gnulib sources

2012-01-17 Thread Jim Meyering
From: Jim Meyering * cfg.mk: New file, to disable all currently-failing tests. We'll enable them one by one, as they are made to pass. * Makefile (sc_maint): New rule. --- ChangeLog |5 + Makefile |7 +++ cfg.mk| 56

run syntax-check rules on gnulib's own sources

2012-01-17 Thread Jim Meyering
Here's a framework by which "make sc_check" will run maint.mk's syntax-check rules on gnulib's own sources. The first patch adds the rule and a new file, cfg.mk which disables all of the currently-failing tests (many!) The next few patches enable tests one by one, removing the test name from the sk

[PATCH 7/8] maint: enable sc_prohibit_openat_without_use

2012-01-17 Thread Jim Meyering
From: Jim Meyering * cfg.mk: Enable sc_prohibit_openat_without_use. Exempt lib/selinux-at.c. --- ChangeLog |4 cfg.mk|2 +- 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index a85f74e..ec42f95 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5

[PATCH 5/8] maint: enable sc_prohibit_intprops_without_use

2012-01-17 Thread Jim Meyering
From: Jim Meyering * cfg.mk: Enable sc_prohibit_intprops_without_use * tests/test-nanosleep.c: Don't include "intprops.h". Not needed. * lib/ftoastr.c: Likewise. --- ChangeLog |4 cfg.mk |1 - lib/ftoastr.c |1 - tests/test-nanosleep.c |

[PATCH 8/8] maint: enable sc_trailing_blank

2012-01-17 Thread Jim Meyering
From: Jim Meyering * build-aux/pmccabe.css: Remove trailing blanks. * doc/acl-cygwin.txt: Likewise. * doc/gnu-oids.texi: Likewise * cfg.mk: Enable sc_trailing_blank. Exempt build-aux/texinfo.tex and doc/Copyright/assign.future.manual. --- ChangeLog |7 +++ build-aux/pmccabe.c

[PATCH 6/8] maint: enable sc_prohibit_cloexec_without_use

2012-01-17 Thread Jim Meyering
From: Jim Meyering * cfg.mk: Enable sc_prohibit_cloexec_without_use. * lib/dup-safer-flag.c: Don't include "cloexec.h". Not needed. --- ChangeLog|6 ++ cfg.mk |1 - lib/dup-safer-flag.c |2 -- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git

[PATCH] maint: spell file systems with two words, not one

2012-01-17 Thread Jim Meyering
FYI, >From 1685c6effebceb4d718486558ed54e794d1547ac Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Tue, 17 Jan 2012 17:30:33 +0100 Subject: [PATCH] maint: spell file systems with two words, not one * m4/ls-mntd-fs.m4 (MOUNTED_INTERIX_STATVFS): Spell file systems with two words, not one. ---

[PATCH] maint: spell file systems with two words, not one

2012-01-17 Thread Jim Meyering
Barely worth a changelog entry... >From 1685c6effebceb4d718486558ed54e794d1547ac Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Tue, 17 Jan 2012 17:30:33 +0100 Subject: [PATCH] maint: spell file systems with two words, not one * m4/ls-mntd-fs.m4 (MOUNTED_INTERIX_STATVFS): Spell file systems w