lourenco/mgmt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

langpuppet: Fix Cli method invocations for wrapped GAPIs

Browse Source 
Since the langpuppet GAPI creates fresh new CliContext objects,
it has to make sure to provide the original parent context, because
the child GAPIs expect to be able to access its data.
This commit is contained in:
Felix Frank
2019-02-05 16:34:55 +01:00
parent bbfeb49cdf
commit 8ba6c40f0c
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
langpuppet/gapi.go
View File

@@ -123,13 +123,13 @@ func (obj *GAPI) Cli(cliInfo *gapi.CliInfo) (*gapi.Deploy, error) {
var langDeploy *gapi.Deploy var langDeploy *gapi.Deploy
var puppetDeploy *gapi.Deploy var puppetDeploy *gapi.Deploy
langCliInfo := &gapi.CliInfo{ langCliInfo := &gapi.CliInfo{
CliContext: cli.NewContext(c.App, flagSet, nil), CliContext: cli.NewContext(c.App, flagSet, c.Parent()),
Fs: fs, Fs: fs,
Debug: debug, Debug: debug,
Logf: logf, // TODO: wrap logf? Logf: logf, // TODO: wrap logf?
} }
puppetCliInfo := &gapi.CliInfo{ puppetCliInfo := &gapi.CliInfo{
CliContext: cli.NewContext(c.App, flagSet, nil), CliContext: cli.NewContext(c.App, flagSet, c.Parent()),
Fs: fs, Fs: fs,
Debug: debug, Debug: debug,
Logf: logf, // TODO: wrap logf? Logf: logf, // TODO: wrap logf?