also sprach Junichi Uekawa <[EMAIL PROTECTED]> [2007.07.18.0210 +0200]:
> I had an impression that the code should diff everything as committed,
> and find 'debian/changelog'. The effect should be so that, if you
> didn't include debian/changelog in your commit, you get an error and
> debcommit will barf out.  However, it isn't. If you didn't add
> debian/changelog explicitly in the command-line you get a commit
> without debian/changelog; which is a different 'bug' to the one we're
> trying to discuss.

This is not what I see. Initially, I thought the patch should call
git add on debian/changelog, but then I decided that it makes more
sense to require the user to actually stage that file along with all
other hunks. If debian/changelog is not staged, you get the
following warning when calling debcommit (with my patch):

  debcommit: unable to determine commit message using git (did you
  forget to run git add?)
  Try using the -m flag.

> There are technical people who like that detail, but imo, they can
> use git directly.

Sure, but debcommit is really a tool to extract the changelog entry
and reuse it for the commit message. git can't do that.

also sprach Adam D. Barratt <[EMAIL PROTECTED]> [2007.07.18.0814 +0200]:
> Other than changing git-diff to "git diff" as per Martin's original
> patch, does anyone have any objections to this new version? If not then
> I'll look at applying it (or something very close) to the next release.

After reading over

  http://marc.info/?t=118322948500001&r=1&w=2

again, I am not sure it is being deprecated. Let's leave it for now.

Thanks, Junichi!

-- 
 .''`.   martin f. krafft <[EMAIL PROTECTED]>
: :'  :  proud Debian developer, author, administrator, and user
`. `'`   http://people.debian.org/~madduck - http://debiansystem.info
  `-  Debian - when you have better things to do than fixing systems

Attachment: signature.asc
Description: Digital signature (GPG/PGP)

Reply via email to