Skip to content

Commit eac9dc2

Browse files
authored
Merge pull request #4144 from thaJeztah/1.3_backport_fix_manpages
[release/1.3 backport] man-page fixes
2 parents 3a4acfb + 57f41a2 commit eac9dc2

4 files changed

Lines changed: 25 additions & 18 deletions

File tree

Makefile

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ endif
8787

8888
# Project binaries.
8989
COMMANDS=ctr containerd containerd-stress
90-
MANPAGES=ctr.1 containerd.1 containerd-config.1 containerd-config.toml.5
90+
MANPAGES=ctr.8 containerd.8 containerd-config.8 containerd-config.toml.5
9191

9292
ifdef BUILDTAGS
9393
GO_BUILDTAGS = ${BUILDTAGS}
@@ -209,15 +209,15 @@ mandir:
209209
@mkdir -p man
210210

211211
# Kept for backwards compatability
212-
genman: man/containerd.1 man/ctr.1
212+
genman: man/containerd.8 man/ctr.8
213213

214-
man/containerd.1: FORCE
214+
man/containerd.8: FORCE
215215
@echo "$(WHALE) $@"
216-
go run cmd/gen-manpages/main.go containerd man/
216+
go run cmd/gen-manpages/main.go $(@F) $(@D)
217217

218-
man/ctr.1: FORCE
218+
man/ctr.8: FORCE
219219
@echo "$(WHALE) $@"
220-
go run cmd/gen-manpages/main.go ctr man/
220+
go run cmd/gen-manpages/main.go $(@F) $(@D)
221221

222222
man/%: docs/man/%.md FORCE
223223
@echo "$(WHALE) $@"

cmd/gen-manpages/main.go

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ import (
2222
"io/ioutil"
2323
"os"
2424
"path/filepath"
25+
"strings"
2526

2627
"github.com/containerd/containerd/cmd/containerd/command"
2728
"github.com/containerd/containerd/cmd/ctr/app"
@@ -41,22 +42,28 @@ func run() error {
4142
"containerd": command.App(),
4243
"ctr": app.New(),
4344
}
44-
name := flag.Arg(0)
4545
dir := flag.Arg(1)
46-
app, ok := apps[name]
46+
47+
parts := strings.SplitN(flag.Arg(0), ".", 2)
48+
if len(parts) != 2 {
49+
return fmt.Errorf("invalid name '%s': name does not contain man page section", flag.Arg(0))
50+
}
51+
name, section := parts[0], parts[1]
52+
53+
appName, ok := apps[name]
4754
if !ok {
48-
return fmt.Errorf("Invalid application '%s'", name)
55+
return fmt.Errorf("invalid application '%s'", name)
4956
}
57+
5058
// clear out the usage as we use banners that do not display in man pages
51-
app.Usage = ""
52-
data, err := app.ToMan()
59+
appName.Usage = ""
60+
appName.ToMan()
61+
data, err := appName.ToMan()
5362
if err != nil {
5463
return err
5564
}
56-
if _, err := os.Stat(dir); os.IsNotExist(err) {
57-
os.Mkdir(dir, os.ModePerm)
58-
}
59-
if err := ioutil.WriteFile(filepath.Join(dir, fmt.Sprintf("%s.1", name)), []byte(data), 0644); err != nil {
65+
_ = os.MkdirAll(dir, os.ModePerm)
66+
if err := ioutil.WriteFile(filepath.Join(dir, fmt.Sprintf("%s.%s", name, section)), []byte(data), 0644); err != nil {
6067
return err
6168
}
6269
return nil
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# containerd-config 1 01/30/2018
1+
# containerd-config 8 01/30/2018
22

33
## NAME
44

@@ -38,4 +38,4 @@ Phil Estes <[email protected]>
3838

3939
## SEE ALSO
4040

41-
ctr(1), containerd(1), containerd-config.toml(5)
41+
ctr(8), containerd(8), containerd-config.toml(5)

docs/man/containerd-config.toml.5.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,4 +140,4 @@ Phil Estes <[email protected]>
140140

141141
## SEE ALSO
142142

143-
ctr(1), containerd-config(1), containerd(1)
143+
ctr(8), containerd-config(8), containerd(8)

0 commit comments

Comments
 (0)