Skip to content

Makefile: man page: rename containerd.1 to containerd.8#4139

Merged
estesp merged 1 commit intocontainerd:masterfrom
thaJeztah:fix_manpages
Apr 2, 2020
Merged

Makefile: man page: rename containerd.1 to containerd.8#4139
estesp merged 1 commit intocontainerd:masterfrom
thaJeztah:fix_manpages

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Apr 2, 2020

The generated file was incorrectly named containerd.1 and should
be in section 8 (see MAN-PAGES(7): Sections of the manual pages)

This patch fixes the filename and updates references to containerd(1)
to refer to containerd(8).

The generated file itself already had the correct section set in its
header, so didn't need updating.

The generated file was incorrectly named containerd.1 and should
be in section 8 (see [MAN-PAGES(7)]: Sections of the manual pages)

This patch fixes the filename and updates references to containerd(1)
to refer to containerd(8).

The generated file itself already had the correct section set in its
header, so didn't need updating.

[MAN-PAGES(7)]: http://man7.org/linux/man-pages/man7/man-pages.7.html

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 2, 2020

Build succeeded.

@dims
Copy link
Copy Markdown
Member

dims commented Apr 2, 2020

LGTM

Copy link
Copy Markdown
Contributor

@Zyqsempai Zyqsempai left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit a19a357 into containerd:master Apr 2, 2020
@thaJeztah thaJeztah deleted the fix_manpages branch May 10, 2020 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants