Re: [Savannah-help-public] address@hidden: [SCM] GNU Inetutils branch, master, updated. release_1_5-220-gd3080c5]

[Sylvain Beucler]

Hi,

I upgraded the commit hook to use what we're using for news projects.

I displays the diff using 'git show -C COMMIT', which apparently gives
better results in this case:
http://git.savannah.gnu.org/cgit/inetutils.git/commit/?id=d3080c58a18f3f10f3e3372eb6f3ca5f1032dda7

In this new version it's easier to customize the output, currently the config 
is:

   showrev = "echo 
http://git.savannah.gnu.org/cgit/inetutils.git/commit/?id=%s; echo; echo; git 
show -C %s; echo"

-- 
Sylvain

On Sat, Nov 14, 2009 at 05:10:39AM -0500, Alfred M. Szmidt wrote:
> I got the following strange commit message from savannah,
> 
> | diff --git a/hostname/Makefile.am b/cfg.mk
> | similarity index 71%
> | copy from hostname/Makefile.am
> | copy to cfg.mk
> | index c7d78f8..6cfdbb9 100644
> | - --- a/hostname/Makefile.am
> | +++ b/cfg.mk
> 
> The change Simon did was not an rename though, we guessed that there
> is a switch for git that makes it look harder for rewrites of files.
> Would it be possible to fix this?
> 
> ------- Start of forwarded message -------
> From: "Simon Josefsson" <address@hidden>
> To: address@hidden
> Date: Thu, 12 Nov 2009 21:29:02 +0000
> Subject: [SCM] GNU Inetutils  branch, master,
>       updated. release_1_5-220-gd3080c5
> 
> This is an automated email from the git hooks/post-receive script. It was
> generated because a ref change was pushed to the repository containing
> the project "GNU Inetutils ".
> 
> http://git.savannah.gnu.org/cgit/inetutils.git/commit/?id=d3080c58a18f3f10f3e3372eb6f3ca5f1032dda7
> 
> The branch, master has been updated
>        via  d3080c58a18f3f10f3e3372eb6f3ca5f1032dda7 (commit)
>       from  d6848fcc4af10e83a70712415bb1a487a56899af (commit)
> 
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email; so we list those
> revisions in full, below.
> 
> - - Log -----------------------------------------------------------------
> commit d3080c58a18f3f10f3e3372eb6f3ca5f1032dda7
> Author: Simon Josefsson <address@hidden>
> Date:   Thu Nov 12 22:28:43 2009 +0100
> 
>     Add coverage related rules.
> 
> - -----------------------------------------------------------------------
> 
> Summary of changes:
>  ChangeLog                      |    6 ++++++
>  bootstrap.conf                 |    1 +
>  hostname/Makefile.am => cfg.mk |   16 ++++++++--------
>  3 files changed, 15 insertions(+), 8 deletions(-)
>  copy hostname/Makefile.am => cfg.mk (71%)
> 
> diff --git a/ChangeLog b/ChangeLog
> index 818f648..8b73928 100644
> - --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,9 @@
> +2009-11-12  Simon Josefsson  <address@hidden>
> +
> +     * bootstrap.conf (gnulib_modules): Add maintainer-makefile, to get
> +     coverage generating rules.
> +     * cfg.mk: New file, with coverage uploading rules.
> +
>  2009-11-08  Alfred M. Szmidt  <address@hidden>
>  
>       * hostname/hostname.c (_hostname_options): Struct renamed to ..
> diff --git a/bootstrap.conf b/bootstrap.conf
> index e219ac1..1586115 100644
> - --- a/bootstrap.conf
> +++ b/bootstrap.conf
> @@ -42,6 +42,7 @@ gettext-h
>  getusershell
>  glob
>  inttostr
> +maintainer-makefile
>  malloc
>  mempcpy
>  minmax
> diff --git a/hostname/Makefile.am b/cfg.mk
> similarity index 71%
> copy from hostname/Makefile.am
> copy to cfg.mk
> index c7d78f8..6cfdbb9 100644
> - --- a/hostname/Makefile.am
> +++ b/cfg.mk
> @@ -1,5 +1,5 @@
>  #
> - -# Copyright (C) 2008, 2009 Free Software Foundation, Inc.
> +# Copyright (C) 2009 Free Software Foundation, Inc.
>  #
>  # This file is part of GNU Inetutils.
>  #
> @@ -16,12 +16,12 @@
>  # You should have received a copy of the GNU General Public License
>  # along with this program.  If not, see `http://www.gnu.org/licenses/'.
>  
> - -bin_PROGRAMS = @hostname_BUILD@
> +htmldir = ../www-$(PACKAGE)
>  
> - -EXTRA_PROGRAMS = hostname
> +web-coverage:
> +     rm -fv `find $(htmldir)/coverage -type f | grep -v CVS`
> +     cp -rv doc/coverage/* $(htmldir)/coverage/
>  
> - -hostname_SOURCES = hostname.c
> - -
> - -INCLUDES = -I$(top_srcdir)/lib -I$(top_srcdir)/libinetutils
> - -
> - -LDADD = -L$(top_builddir)/lib -lgnu
> +upload-web-coverage:
> +     cd $(htmldir) && \
> +             cvs commit -m "Update." coverage
> 
> 
> hooks/post-receive
> - -- 
> GNU Inetutils 
> 
> 
> _______________________________________________
> Commit-inetutils mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/commit-inetutils
> ------- End of forwarded message -------
> 
>