mirror of
https://gitea.com/Lydanne/buildx.git
synced 2025-05-21 03:07:46 +08:00
build: don't generate metadata file when print flag is used
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com> (cherry picked from commit ba264138d62e79f3f740b272e4b6152fa0fc5199)
This commit is contained in:
parent
b77648d5f8
commit
f6e57cf5b5
@ -122,27 +122,26 @@ func (o *buildOptions) toControllerOptions() (*controllerapi.BuildOptions, error
|
|||||||
}
|
}
|
||||||
|
|
||||||
opts := controllerapi.BuildOptions{
|
opts := controllerapi.BuildOptions{
|
||||||
Allow: o.allow,
|
Allow: o.allow,
|
||||||
Annotations: o.annotations,
|
Annotations: o.annotations,
|
||||||
BuildArgs: buildArgs,
|
BuildArgs: buildArgs,
|
||||||
CgroupParent: o.cgroupParent,
|
CgroupParent: o.cgroupParent,
|
||||||
ContextPath: o.contextPath,
|
ContextPath: o.contextPath,
|
||||||
DockerfileName: o.dockerfileName,
|
DockerfileName: o.dockerfileName,
|
||||||
ExtraHosts: o.extraHosts,
|
ExtraHosts: o.extraHosts,
|
||||||
Labels: labels,
|
Labels: labels,
|
||||||
NetworkMode: o.networkMode,
|
NetworkMode: o.networkMode,
|
||||||
NoCacheFilter: o.noCacheFilter,
|
NoCacheFilter: o.noCacheFilter,
|
||||||
Platforms: o.platforms,
|
Platforms: o.platforms,
|
||||||
ShmSize: int64(o.shmSize),
|
ShmSize: int64(o.shmSize),
|
||||||
Tags: o.tags,
|
Tags: o.tags,
|
||||||
Target: o.target,
|
Target: o.target,
|
||||||
Ulimits: dockerUlimitToControllerUlimit(o.ulimits),
|
Ulimits: dockerUlimitToControllerUlimit(o.ulimits),
|
||||||
Builder: o.builder,
|
Builder: o.builder,
|
||||||
NoCache: o.noCache,
|
NoCache: o.noCache,
|
||||||
Pull: o.pull,
|
Pull: o.pull,
|
||||||
ExportPush: o.exportPush,
|
ExportPush: o.exportPush,
|
||||||
ExportLoad: o.exportLoad,
|
ExportLoad: o.exportLoad,
|
||||||
WithProvenanceResponse: len(o.metadataFile) > 0,
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// TODO: extract env var parsing to a method easily usable by library consumers
|
// TODO: extract env var parsing to a method easily usable by library consumers
|
||||||
@ -207,6 +206,8 @@ func (o *buildOptions) toControllerOptions() (*controllerapi.BuildOptions, error
|
|||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
opts.WithProvenanceResponse = opts.PrintFunc == nil && len(o.metadataFile) > 0
|
||||||
|
|
||||||
return &opts, nil
|
return &opts, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -365,15 +366,14 @@ func runBuild(ctx context.Context, dockerCli command.Cli, options buildOptions)
|
|||||||
return errors.Wrap(err, "writing image ID file")
|
return errors.Wrap(err, "writing image ID file")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if options.metadataFile != "" {
|
|
||||||
if err := writeMetadataFile(options.metadataFile, decodeExporterResponse(resp.ExporterResponse)); err != nil {
|
|
||||||
return err
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if opts.PrintFunc != nil {
|
if opts.PrintFunc != nil {
|
||||||
if err := printResult(opts.PrintFunc, resp.ExporterResponse); err != nil {
|
if err := printResult(opts.PrintFunc, resp.ExporterResponse); err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
} else if options.metadataFile != "" {
|
||||||
|
if err := writeMetadataFile(options.metadataFile, decodeExporterResponse(resp.ExporterResponse)); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
}
|
}
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user