* Martin Steigerwald <Martin@lichtvoll.de> [20090812 21:13]:
> Am Mittwoch 12 August 2009 schrieb Michael Prokop:
> > * Martin Steigerwald <ms@teamix.de> [20090812 17:18]:
> > > Hi Michael!
> [...]
> > > Michael, please review. I am at the office again beginning next week,
> > > so I subscribed to this bug report via my private address.
> > Initial reviewing done. Find fixes against git tree attached as git
> > patches. (Tip: use lintian from unstable to verify your package.)
> > Please review on your own and apply what you acknowledge, ping me if
> > you think the package is ready for upload to Debian then.
> All applied. Added a changelog entry for each one. Pushed my changes.
Thx.
> Please pull and review.
Just one minor problem: the copyright sign in your git commit isn't
UTF-8, which should be used nowadays in debian maintainer scripts.
Fix it via running:
iconv -f ISO-8859-1 -t UTF-8 debian/copyright > debian/copyright.new
mv debian/copyright.new debian/copyright
I've attached a patch (which also changes another '(C)' to '©') that
addresses the issue so you can verify the result. Then the package
should be lintian clean.
> > @Jens: the first patch
> > (0001-fio.1-fix-hyphen-used-as-minus-sign.patch) addresses a minor
> > issue in the manpage. Please feel free to apply it in your upstream
> > source.
> Since it is only one patch for upstream - except a minor patch for the
> Makefile to build in /usr - and Jens already applied it anyway I decided
> not to use quilt this time.
ACK
> You replaced dh_clean by dh_prep in
> 0003-debian-rules-drop-unnecessary-configure-target-repla.patch
> What is the reason? Is dh_clean deprecated?
You are using 'debhelper (>= 7)' in Build-Depends.
debhelper 7 deprecated dh_clean -k in favour of dh_prep.
> I forgot to add a ~/.gitconfig with my name and mail address at work. Is
> there any way to rewrite the mail address in the initial git log entries?
> If not, well its just an email address in there that won't work from
> outside of our intranet.
Demonstrating switch from grml.org to debian.org:
% git filter-branch --commit-filter '
if [ "$GIT_AUTHOR_EMAIL" = "mika@grml.org" ];
then
export GIT_AUTHOR_EMAIL="mika@debian.org";
git commit-tree "$@";
else
git commit-tree "$@";
fi' HEAD
Depending on what you'd like to set also check out
$GIT_COMMITTER_EMAIL & CO. HTH
regards,
-mika-
From 8571251a4850e86027e6ba9a73d458e9030b39fc Mon Sep 17 00:00:00 2001
From: Michael Prokop <mika@grml.org>
Date: Wed, 12 Aug 2009 22:33:40 +0200
Subject: [PATCH] =?utf-8?q?Fix=20copyright=20unicode=20char=20and=20move=20another=20(C)=20to=20=C2=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Signed-off-by: Michael Prokop <mika@grml.org>
---
debian/copyright | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/debian/copyright b/debian/copyright
index 79ab588..1beb522 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -8,7 +8,7 @@ Upstream Author:
Jens Axboe <jens.axboe@oracle.com>
-Copyright: © 2005-2006 by Jens Axboe
+Copyright: © 2005-2006 by Jens Axboe
License:
@@ -75,5 +75,5 @@ Copyrights notes:
Copyright (C) Andrew McDonald <andrew@mcdonald.org.uk>
Copyright (C) 2003 Kyle McMartin <kyle@debian.org>
-The Debian packaging is (C) 2009, Martin Steigerwald <ms@teamix.de> and
+The Debian packaging is © 2009, Martin Steigerwald <ms@teamix.de> and
is licensed under the GPL, see `/usr/share/common-licenses/GPL-2'.
--
1.6.3.3
Attachment:
signature.asc
Description: Digital signature