Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Aug 15, 2018

In the section 2.3, if one wants to see the changes made by the author
Junio Hamano, the author name cannot be gitster. Fixed the same.

In the section 2.3, if one wants to see the changes made by the author
Junio Hamano, the author name cannot be gitster. Fixed the same.
[source,console]
----
$ git log --pretty="%h - %s" --author=gitster --since="2008-10-01" \
$ git log --pretty="%h - %s" --author=Junio Hamano --since="2008-10-01" \
Copy link
Member

@ben ben Aug 15, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably this should be --author='Junio Hamano'.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works without quotes; at least for me :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ git log --pretty="%h - %s" --author=Junio Hamano --since="2008-10-01"
fatal: ambiguous argument 'Hamano': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

In any case, even if it works in some places, we should follow good practices and wrap arguments in quotes.

Also:

$ git log --pretty="%h - %s" --author='Junio Hamano' --since="2008-10-01" | wc -l
       2

Looks like Junio signs his commits as Junio C Hamano most of the time. Can you fix that as well?

In the section 2.3, if one wants to see the changes made by the author
Junio C Hamano, the author name cannot be gitster. Fixed the same.
@ghost
Copy link
Author

ghost commented Aug 24, 2018

@ben Done. I think it is safe to merge now.

@ben
Copy link
Member

ben commented Aug 24, 2018

🌈

@ben ben merged commit 966b02f into progit:master Aug 24, 2018
@jnavila jnavila added the translation backport Tags a PR for translators to backport in their own repo label Aug 24, 2018
aollier added a commit to aollier/progit2-fr that referenced this pull request Oct 31, 2018
gitster est l'identifiant de Junio C Hamano
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation backport Tags a PR for translators to backport in their own repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants