threads / patch / 53498

patchgit-bugreport.txt: adjust reference to strftime(3)

Subject: [PATCH] git-bugreport.txt: adjust reference to strftime(3)

## tl;dr

3 messages between May 18, 2020 and May 18, 2020. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Todd Zullinger· May 18, 2020, 19:37 UTC · lore

The strftime(3) man page is outside of the Git suite. Refererence it as we do other external man pages and avoid creating a broken link when generating the HTML documentation.

Signed-off-by: Todd Zullinger <tmz@pobox.com>
---
Hi Emily,

While building 2.27.0-rc0, I noticed a broken link crept into the HTML docs for git-bugreport. I believe this change matches most of our existing references to external man pages.

Some are enclosed in backticks, while most are not. My first instinct was to use backticks. I avoided that to be consistent with the majority of other references. But I didn't look closely to see if newer additions use backticks versus older changes do not.

The current "link:strftime[3]" tries to create a link to strftime named 3, which doesn't seem like what we want. :)

Thanks, Todd

 Documentation/git-bugreport.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to Documentation/git-bugreport.txt +1 −1
diff --git a/Documentation/git-bugreport.txt b/Documentation/git-bugreport.txt
index 7fe9aef34e..9edad66a63 100644
--- a/Documentation/git-bugreport.txt
+++ b/Documentation/git-bugreport.txt
@@ -46,7 +46,7 @@ OPTIONS
 --suffix <format>::
 	Specify an alternate suffix for the bugreport name, to create a file
 	named 'git-bugreport-<formatted suffix>'. This should take the form of a
-	link:strftime[3] format string; the current local time will be used.
+	strftime(3) format string; the current local time will be used.
 
 GIT
 ---
-- 
2.26.1
Junio C Hamano· May 18, 2020, 19:53 UTC · re: Todd Zullinger · lore

Re: [PATCH] git-bugreport.txt: adjust reference to strftime(3)

Todd Zullinger <tmz@pobox.com> writes:
Show 12 quoted lines
> The strftime(3) man page is outside of the Git suite.  Refererence it as
> we do other external man pages and avoid creating a broken link when
> generating the HTML documentation.
>
> Signed-off-by: Todd Zullinger <tmz@pobox.com>
> ---
>
> Hi Emily,
>
> While building 2.27.0-rc0, I noticed a broken link crept into the HTML
> docs for git-bugreport.  I believe this change matches most of our
> existing references to external man pages.

Thanks. This was noticed elsewhere in a different thread but let's take this as a patch independent from the other documentation topic.

Show 13 quoted lines
> diff --git a/Documentation/git-bugreport.txt b/Documentation/git-bugreport.txt
> index 7fe9aef34e..9edad66a63 100644
> --- a/Documentation/git-bugreport.txt
> +++ b/Documentation/git-bugreport.txt
> @@ -46,7 +46,7 @@ OPTIONS
>  --suffix <format>::
>  	Specify an alternate suffix for the bugreport name, to create a file
>  	named 'git-bugreport-<formatted suffix>'. This should take the form of a
> -	link:strftime[3] format string; the current local time will be used.
> +	strftime(3) format string; the current local time will be used.
>  
>  GIT
>  ---
Todd Zullinger· May 18, 2020, 20:18 UTC · re: Junio C Hamano · lore

Re: [PATCH] git-bugreport.txt: adjust reference to strftime(3)

Junio C Hamano wrote:
Show 6 quoted lines
> Todd Zullinger <tmz@pobox.com> writes:
> 
>> The strftime(3) man page is outside of the Git suite.  Refererence it as
>> we do other external man pages and avoid creating a broken link when
>> generating the HTML documentation.
>>
[...]
> Thanks.  This was noticed elsewhere in a different thread but let's
> take this as a patch independent from the other documentation topic.

Ahh, I have gotten behind on my reading (and only just got to testing rc0¹). I'm glad this unintended redundancy worked out well, by chance.

Thank you.
¹ FWIW, this was the only issue I found in building for the
  various architectures Fedora supports: aarch64, armv7hl,
  ppc64le, x86, x86_64, and, last -- but by far the most
  likely to turn up issues -- s390x.
-- 
Todd

← back to recent threads