<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/builtin/commit.c, branch v2.11.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.11.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.11.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2017-01-17T23:11:03Z</updated>
<entry>
<title>Merge branch 'ak/commit-only-allow-empty' into maint</title>
<updated>2017-01-17T23:11:03Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2017-01-17T23:11:03Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=9d2a24864e8f0a1ab2fc6b72c64a4f1d4aa9fe94'/>
<id>urn:sha1:9d2a24864e8f0a1ab2fc6b72c64a4f1d4aa9fe94</id>
<content type='text'>
"git commit --allow-empty --only" (no pathspec) with dirty index
ought to be an acceptable way to create a new commit that does not
change any paths, but it was forbidden, perhaps because nobody
needed it so far.

* ak/commit-only-allow-empty:
  commit: remove 'Clever' message for --only --amend
  commit: make --only --allow-empty work without paths
</content>
</entry>
<entry>
<title>commit: remove 'Clever' message for --only --amend</title>
<updated>2016-12-09T18:52:46Z</updated>
<author>
<name>Andreas Krey</name>
<email>a.krey@gmx.de</email>
</author>
<published>2016-12-09T04:10:21Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=beb635ca9ced5cd8f0eb5d2def9468f14bd243bc'/>
<id>urn:sha1:beb635ca9ced5cd8f0eb5d2def9468f14bd243bc</id>
<content type='text'>
The behavior is now documented; more importantly, rewarding the user
with a "Wow, you are clever" praise afterwards is not an effective
way to advertise the feature--at that point the user already knows.

Signed-off-by: Andreas Krey &lt;a.krey@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>commit: make --only --allow-empty work without paths</title>
<updated>2016-12-05T20:41:06Z</updated>
<author>
<name>Andreas Krey</name>
<email>a.krey@gmx.de</email>
</author>
<published>2016-12-02T22:15:13Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=319d835240b752b94ccb67f6515759824955937b'/>
<id>urn:sha1:319d835240b752b94ccb67f6515759824955937b</id>
<content type='text'>
--only is implied when paths are present, and required
them unless --amend. But with --allow-empty it should
be allowed as well - it is the only way to create an
empty commit in the presence of staged changes.

Signed-off-by: Andreas Krey &lt;a.krey@gmx.de&gt;
Reviewed-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'rs/commit-pptr-simplify'</title>
<updated>2016-10-31T20:15:25Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2016-10-31T20:15:25Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2f445c17e59bb1fa49ad9197a4155f878f50d5f5'/>
<id>urn:sha1:2f445c17e59bb1fa49ad9197a4155f878f50d5f5</id>
<content type='text'>
Code simplification.

* rs/commit-pptr-simplify:
  commit: simplify building parents list
</content>
</entry>
<entry>
<title>commit: simplify building parents list</title>
<updated>2016-10-30T23:03:25Z</updated>
<author>
<name>René Scharfe</name>
<email>l.s.r@web.de</email>
</author>
<published>2016-10-29T12:55:36Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=de9f7fa3b08ae0554e57f6a6f5de2559d5bcb871'/>
<id>urn:sha1:de9f7fa3b08ae0554e57f6a6f5de2559d5bcb871</id>
<content type='text'>
Push pptr down into the FROM_MERGE branch of the if/else statement,
where it's actually used, and call commit_list_append() for appending
elements instead of playing tricks with commit_list_insert().  Call
copy_commit_list() in the amend branch instead of open-coding it.  Don't
bother setting pptr in the final branch as it's not used thereafter.

Signed-off-by: Rene Scharfe &lt;l.s.r@web.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'nd/ita-empty-commit'</title>
<updated>2016-10-27T21:58:50Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2016-10-27T21:58:50Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=650360210afbd585f33ed622d3e700b1941b1ddb'/>
<id>urn:sha1:650360210afbd585f33ed622d3e700b1941b1ddb</id>
<content type='text'>
When new paths were added by "git add -N" to the index, it was
enough to circumvent the check by "git commit" to refrain from
making an empty commit without "--allow-empty".  The same logic
prevented "git status" to show such a path as "new file" in the
"Changes not staged for commit" section.

* nd/ita-empty-commit:
  commit: don't be fooled by ita entries when creating initial commit
  commit: fix empty commit creation when there's no changes but ita entries
  diff: add --ita-[in]visible-in-index
  diff-lib: allow ita entries treated as "not yet exist in index"
</content>
</entry>
<entry>
<title>commit: don't be fooled by ita entries when creating initial commit</title>
<updated>2016-10-24T17:54:11Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2016-10-24T10:42:22Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2c49f7ffb3063fdccccf2a038ab2eee66e7395e4'/>
<id>urn:sha1:2c49f7ffb3063fdccccf2a038ab2eee66e7395e4</id>
<content type='text'>
ita entries are dropped at tree generation phase. If the entire index
consists of just ita entries, the result would be a a commit with no
entries, which should be caught unless --allow-empty is specified. The
test "!!active_nr" is not sufficient to catch this.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>commit: fix empty commit creation when there's no changes but ita entries</title>
<updated>2016-10-24T17:48:23Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2016-10-24T10:42:21Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=018ec3c8203ad1aee683840a580cfba7914419b6'/>
<id>urn:sha1:018ec3c8203ad1aee683840a580cfba7914419b6</id>
<content type='text'>
If i-t-a entries are present and there is no change between the index
and HEAD i-t-a entries, index_differs_from() still returns "dirty, new
entries" (aka, the resulting commit is not empty), but cache-tree will
skip i-t-a entries and produce the exact same tree of current
commit.

index_differs_from() is supposed to catch this so we can abort
git-commit (unless --no-empty is specified). Update it to optionally
ignore i-t-a entries when doing a diff between the index and HEAD so
that it would return "no change" in this case and abort commit.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>sequencer: use memoized sequencer directory path</title>
<updated>2016-10-17T18:52:23Z</updated>
<author>
<name>Johannes Schindelin</name>
<email>johannes.schindelin@gmx.de</email>
</author>
<published>2016-10-14T13:17:12Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=8a2a0f534141cf6d2fa63fe3f84acbc4763c6754'/>
<id>urn:sha1:8a2a0f534141cf6d2fa63fe3f84acbc4763c6754</id>
<content type='text'>
Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'tg/add-chmod+x-fix'</title>
<updated>2016-09-26T23:09:20Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2016-09-26T23:09:20Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ebc63580a1287325eef85e96c436de494406fd0a'/>
<id>urn:sha1:ebc63580a1287325eef85e96c436de494406fd0a</id>
<content type='text'>
"git add --chmod=+x &lt;pathspec&gt;" added recently only toggled the
executable bit for paths that are either new or modified. This has
been corrected to flip the executable bit for all paths that match
the given pathspec.

* tg/add-chmod+x-fix:
  t3700-add: do not check working tree file mode without POSIXPERM
  t3700-add: create subdirectory gently
  add: modify already added files when --chmod is given
  read-cache: introduce chmod_index_entry
  update-index: add test for chmod flags
</content>
</entry>
</feed>
