README-release

2012-03-20 Thread Reuben Thomas
A slight simplification:

>From 4ab7998acbf4f829fdecabd83dc75beedcdffea1 Mon Sep 17 00:00:00 2001
From: Reuben Thomas 
Date: Tue, 20 Mar 2012 13:37:21 +
Subject: [PATCH] README-release: simplify slightly

* top/README-release: only run "git checkout master" once.
---
 ChangeLog  |5 +
 top/README-release |8 
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index d2370a0..35d1662 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2012-03-20  Reuben Thomas  
+
+   README-release: simplify slightly
+   * top/README-release: only run "git checkout master" once.
+
 2012-03-18  Andreas Oberritter  
 
git-version-gen: don't let "prefix" envvar cause trouble
diff --git a/top/README-release b/top/README-release
index 680547b..29ba260 100644
--- a/top/README-release
+++ b/top/README-release
@@ -1,6 +1,6 @@
 Here are most of the steps we (maintainers) follow when making a release.
 
-* Start from a clean, up-to-date git directory:
+* Start from a clean, up-to-date git directory on "master":
 
 make -k maintainer-clean || { ./configure && make maintainer-clean; }
 git checkout master
@@ -10,10 +10,10 @@ Here are most of the steps we (maintainers) follow when 
making a release.
   are in your PATH.  See the buildreq list in bootstrap.conf for
   the complete list of tools.
 
-* Ensure that you're on "master" with no uncommitted diffs.
-  This should produce no output:
+* Ensure that you have no uncommitted diffs.  This should produce no
+  output:
 
-git checkout master; git diff
+git diff
 
 * Ensure that you've pushed all changes that belong in the release:
 
-- 
1.7.9.1


-- 
http://rrt.sc3d.org/



Re: [PATCH] git-merge-changelog: add specific example on how to use with hg.

2012-03-20 Thread Eric Blake
On 03/16/2012 12:08 PM, Mark Wielaard wrote:
>   * lib/git-merge-changelog.c: Add example on how to use in .hgrc.
> ---
>  ChangeLog |5 +
>  lib/git-merge-changelog.c |   10 +-
>  2 files changed, 14 insertions(+), 1 deletions(-)
> 
> diff --git a/ChangeLog b/ChangeLog
> index e600a1d..9650fc3 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,8 @@
> +2012-03-15  Mark Wielaard  
> +
> + git-merge-changelog: add specific example on how to use with hg.
> + * lib/git-merge-changelog.c: Add example on how to use in .hgrc.

Thanks; pushed.

-- 
Eric Blake   ebl...@redhat.com+1-919-301-3266
Libvirt virtualization library http://libvirt.org



signature.asc
Description: OpenPGP digital signature


Re: [PATCH] git-merge-changelog: Report bugs to bug-gnulib@gnu.org.

2012-03-20 Thread Eric Blake
On 03/18/2012 05:52 AM, Mark Wielaard wrote:
> git-merge-changelog is maintained in gnulib, so bug reports should go
> to bug-gnulib@gnu.org, not bug-gnu-gett...@gnu.org.
> 
> ---
>  ChangeLog |4 
>  lib/git-merge-changelog.c |2 +-
>  2 files changed, 5 insertions(+), 1 deletions(-)
> 
> diff --git a/ChangeLog b/ChangeLog
> index 128acda..31bf40d 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,7 @@
> +2012-03-18  Mark Wielaard  
> +
> + lib/git-merge-changelog.c (status): Report bugs to bug-gnulib@gnu.org.

Thanks; pushed.

-- 
Eric Blake   ebl...@redhat.com+1-919-301-3266
Libvirt virtualization library http://libvirt.org



signature.asc
Description: OpenPGP digital signature


Re: README-release

2012-03-20 Thread Jim Meyering
Reuben Thomas wrote:
> A slight simplification:
>
>>From 4ab7998acbf4f829fdecabd83dc75beedcdffea1 Mon Sep 17 00:00:00 2001
> From: Reuben Thomas 
> Date: Tue, 20 Mar 2012 13:37:21 +
> Subject: [PATCH] README-release: simplify slightly
>
> * top/README-release: only run "git checkout master" once.

Thanks.  Pushed with this tweak:

  * top/README-release: Run "git checkout master" only once.



[PATCH] tests: avoid gcc warnings about argv vs. const initializers

2012-03-20 Thread Jim Meyering
Pulling a bunch of GNULIB_POSIXCHECK-inspired modules into coreutils
made it so that package is now compiling a few more test programs,
which evoked these failures:

  test-posix_spawn1.c: In function 'main':
  test-posix_spawn1.c:78:3: error: initialization discards 'const' qualifier 
from pointer target type [-Werror]
  test-posix_spawn1.c:78:3: error: initialization discards 'const' qualifier 
from pointer target type [-Werror]
  (and the same for test-posix_spawn2.c)

I've pushed this:

>From 880a2f6db9e37ef84347a47b15f88bef741c03da Mon Sep 17 00:00:00 2001
From: Jim Meyering 
Date: Tue, 20 Mar 2012 20:35:44 +0100
Subject: [PATCH] tests: avoid gcc warnings about argv vs. const initializers

* tests/test-posix_spawn1.c (main): Cast to "(char *)" to avoid two
warnings about discarding 'const' qualifier from pointer target type.
* tests/test-posix_spawn2.c (main): Likewise.
---
 ChangeLog |7 +++
 tests/test-posix_spawn1.c |2 +-
 tests/test-posix_spawn2.c |2 +-
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 748f098..7b67d0f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2012-03-20  Jim Meyering  
+
+   tests: avoid gcc warnings about argv vs. const initializers
+   * tests/test-posix_spawn1.c (main): Cast to "(char *)" to avoid two
+   warnings about discarding 'const' qualifier from pointer target type.
+   * tests/test-posix_spawn2.c (main): Likewise.
+
 2012-03-20  Reuben Thomas  

README-release: simplify slightly
diff --git a/tests/test-posix_spawn1.c b/tests/test-posix_spawn1.c
index 9c41039..5152d5c 100644
--- a/tests/test-posix_spawn1.c
+++ b/tests/test-posix_spawn1.c
@@ -75,7 +75,7 @@ fd_safer (int fd)
 int
 main ()
 {
-  char *argv[3] = { "/bin/sh", CHILD_PROGRAM_FILENAME, NULL };
+  char *argv[3] = { (char *) "/bin/sh", (char *) CHILD_PROGRAM_FILENAME, NULL 
};
   int ifd[2];
   sigset_t blocked_signals;
   sigset_t fatal_signal_set;
diff --git a/tests/test-posix_spawn2.c b/tests/test-posix_spawn2.c
index 39479eb..e66c9f2 100644
--- a/tests/test-posix_spawn2.c
+++ b/tests/test-posix_spawn2.c
@@ -53,7 +53,7 @@ fd_safer (int fd)
 int
 main ()
 {
-  char *argv[3] = { "/bin/sh", CHILD_PROGRAM_FILENAME, NULL };
+  char *argv[3] = { (char *) "/bin/sh", (char *) CHILD_PROGRAM_FILENAME, NULL 
};
   int ofd[2];
   sigset_t blocked_signals;
   sigset_t fatal_signal_set;
--
1.7.10.rc1.23.g16a10



Pushing tags in README-release

2012-03-20 Thread Reuben Thomas
Unless I'm missing something, there's nothing that does "git push
--tags" in README-release. Is that right? If so, I'll provide a patch.

-- 
http://rrt.sc3d.org



Re: Pushing tags in README-release

2012-03-20 Thread Jim Meyering
Reuben Thomas wrote:
> Unless I'm missing something, there's nothing that does "git push
> --tags" in README-release. Is that right? If so, I'll provide a patch.

You don't want to push all tags.
This pushes the just-created tag:

* Push the NEWS-updating changes and the new tag:

v=$(cat .prev-version)
git push origin master tag v$v



Re: Pushing tags in README-release

2012-03-20 Thread Eric Blake
On 03/20/2012 01:52 PM, Reuben Thomas wrote:
> Unless I'm missing something, there's nothing that does "git push
> --tags" in README-release. Is that right? If so, I'll provide a patch.

Pushing all tags may be too drastic (I tend to keep local tags that I
don't want pushed).  It's more appropriate to push just the release tag,
as in 'git push origin v1.0' if you just released 1.0.

-- 
Eric Blake   ebl...@redhat.com+1-919-301-3266
Libvirt virtualization library http://libvirt.org



signature.asc
Description: OpenPGP digital signature


Fix build-aux/announce-gen for Perl 5.14

2012-03-20 Thread Reuben Thomas
The following patch copes with a renamed module:

>From 670ec3363bbbfefe927de0100a4be6f5f556a332 Mon Sep 17 00:00:00 2001
From: Reuben Thomas 
Date: Tue, 20 Mar 2012 20:17:22 +
Subject: [PATCH] announce-gen: make it work with Perl 5.14.

* build-aux/announce-gen: Use either Digest::SHA1 or Digest::SHA
according to Perl version.
---
 ChangeLog  |6 ++
 build-aux/announce-gen |   10 --
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 748f098..a356632 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2012-03-20  Reuben Thomas  
 
+   announce-gen: make it work with Perl 5.14.
+   * build-aux/announce-gen: Use either Digest::SHA1 or Digest::SHA
+   according to Perl version.
+
+2012-03-20  Reuben Thomas  
+
README-release: simplify slightly
* top/README-release: Run "git checkout master" only once.
 
diff --git a/build-aux/announce-gen b/build-aux/announce-gen
index 3ca90a9..4f28049 100755
--- a/build-aux/announce-gen
+++ b/build-aux/announce-gen
@@ -3,7 +3,7 @@ eval '(exit $?0)' && eval 'exec perl -wS "$0" ${1+"$@"}'
 if 0;
 # Generate a release announcement message.
 
-my $VERSION = '2012-01-06 07:46'; # UTC
+my $VERSION = '2012-03-20 20:14'; # UTC
 # The definition above must lie within the first 8 lines in order
 # for the Emacs time-stamp write hook (at end) to update it.
 # If you change this file with Emacs, please let the write hook
@@ -30,7 +30,13 @@ use strict;
 
 use Getopt::Long;
 use Digest::MD5;
-use Digest::SHA1;
+if (!$^V or $^V lt v5.14.0) {
+  require Digest::SHA1;
+  Digest::SHA1->import();
+} else {
+  use Digest::SHA;
+  Digest::SHA->import();
+}
 use POSIX qw(strftime);
 
 (my $ME = $0) =~ s|.*/||;
-- 
1.7.9.1



-- 
http://rrt.sc3d.org/



Gsoc_Project:-

2012-03-20 Thread mandeep
hello sir...i am interested in your project REIMPLEMENT GNULIB TOOL IN
PYTHON. i am good in java and even i had done a project in java(Student
database management system) and it was good.May i know is python required
for this,beacause i don't know much about python..hoping a positive
response..
Thanks a lot...
About myself:-i'm mandeep from india..


Re: Fix build-aux/announce-gen for Perl 5.14

2012-03-20 Thread Jim Meyering
Reuben Thomas wrote:
> The following patch copes with a renamed module:

It appears that it hasn't been renamed, but rather replaced,
at least in Debian.

  
http://wiki.debian.org/Teams/DebianPerlGroup/OpenTasks/Transitions/DigestSHA1ToDigestSHA

> From: Reuben Thomas 
> Date: Tue, 20 Mar 2012 20:17:22 +
> Subject: [PATCH] announce-gen: make it work with Perl 5.14.
...
>  use Getopt::Long;
>  use Digest::MD5;
> -use Digest::SHA1;
> +if (!$^V or $^V lt v5.14.0) {
> +  require Digest::SHA1;
> +  Digest::SHA1->import();
> +} else {
> +  use Digest::SHA;
> +  Digest::SHA->import();
> +}

How about this instead, since Digest::SHA is preferable, when it exists?
(no need for ->import, afaics)

  eval { require Digest::SHA }
or eval { require Digest::SHA1 };

Besides, we must not "use Digest::SHA;" when the module does not exist.
Otherwise, something like this would happen, even when it is in an
inactive "else" clause:

$ perl -e 'if (0) { use FOO::BAR }'
Can't locate FOO/BAR.pm in @INC (@...

Please adjust your subject/log, too.
It has always worked fine with Fedora's 5.14.x



Revised announce-gen patch

2012-03-20 Thread Reuben Thomas

Implementing Jim's suggestions.

>From 3e6c32b5834341ee8d1870f7b85db546b1a1f2d8 Mon Sep 17 00:00:00 2001
From: Reuben Thomas 
Date: Tue, 20 Mar 2012 20:17:22 +
Subject: [PATCH] announce-gen: make it use Digest::SHA where available.

* build-aux/announce-gen: Use either Digest::SHA1 or Digest::SHA
according to availability thereof.
---
 ChangeLog  |6 ++
 build-aux/announce-gen |5 +++--
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 7b67d0f..29a0e09 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2012-03-20  Reuben Thomas  
+
+   announce-gen: make it use Digest::SHA where available.
+   * build-aux/announce-gen: Use either Digest::SHA1 or Digest::SHA
+   according to availability thereof.
+
 2012-03-20  Jim Meyering  
 
tests: avoid gcc warnings about argv vs. const initializers
diff --git a/build-aux/announce-gen b/build-aux/announce-gen
index 3ca90a9..b9c9360 100755
--- a/build-aux/announce-gen
+++ b/build-aux/announce-gen
@@ -3,7 +3,7 @@ eval '(exit $?0)' && eval 'exec perl -wS "$0" ${1+"$@"}'
 if 0;
 # Generate a release announcement message.
 
-my $VERSION = '2012-01-06 07:46'; # UTC
+my $VERSION = '2012-03-20 23:17'; # UTC
 # The definition above must lie within the first 8 lines in order
 # for the Emacs time-stamp write hook (at end) to update it.
 # If you change this file with Emacs, please let the write hook
@@ -30,7 +30,8 @@ use strict;
 
 use Getopt::Long;
 use Digest::MD5;
-use Digest::SHA1;
+eval { require Digest::SHA; }
+  or eval { use Digest::SHA1; };
 use POSIX qw(strftime);
 
 (my $ME = $0) =~ s|.*/||;
-- 
1.7.9.1



-- 
http://rrt.sc3d.org/



Re: Gsoc_Project:-

2012-03-20 Thread Bruno Haible
Hi Mandeep,

> i am interested in your project REIMPLEMENT GNULIB TOOL IN
> PYTHON.

Sure, please, we're awaiting your application!

> i am good in java and even i had done a project in java(Student
> database management system) and it was good.May i know is python required
> for this,beacause i don't know much about python..hoping a positive
> response..

Yes, the answer is positive: Python *is* the required language for this
project. A good opportunity to learn it in more depth if you have only
used it casually up to now.

For submitting a proper application within the deadlines given by Google,
you will want to read the procedure explanations from Google [1] and from
GNU [2].

Also, obviously, you will look into what gnulib-tool is and how it is used
[3], and have a peek at the source code of the existing shell script
implementation [4].

And then we await your application submission!

Bruno

[1] 
http://www.google-melange.com/document/show/gsoc_program/google/gsoc2012/faqs
[2] http://www.gnu.org/software/soc-projects/guidelines.html
[3] 
http://www.gnu.org/software/gnulib/manual/html_node/Invoking-gnulib_002dtool.html
[4] http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=tree





Re: Revised announce-gen patch

2012-03-20 Thread Eric Wong
How about using the generic Digest module instead?

Something like this (totally untested) patch:

diff --git a/build-aux/announce-gen b/build-aux/announce-gen
index 3ca90a9..c453d44 100755
--- a/build-aux/announce-gen
+++ b/build-aux/announce-gen
@@ -29,8 +29,7 @@ my $VERSION = '2012-01-06 07:46'; # UTC
 use strict;
 
 use Getopt::Long;
-use Digest::MD5;
-use Digest::SHA1;
+use Digest;
 use POSIX qw(strftime);
 
 (my $ME = $0) =~ s|.*/||;
@@ -151,17 +150,14 @@ sub print_checksums (@)
   print "Here are the MD5 and SHA1 checksums:\n";
   print "\n";
 
-  foreach my $meth (qw (md5 sha1))
+  foreach my $meth (qw (MD5 SHA-1))
 {
   foreach my $f (@file)
 {
   open IN, '<', $f
 or die "$ME: $f: cannot open for reading: $!\n";
   binmode IN;
-  my $dig =
-($meth eq 'md5'
- ? Digest::MD5->new->addfile(*IN)->hexdigest
- : Digest::SHA1->new->addfile(*IN)->hexdigest);
+  my $dig = Digest->new($meth)->addfile(*IN)->hexdigest;
   close IN;
   print "$dig  $f\n";
 }
-- 
Eric Wong