Merge pull request #57143 from sttts/sttts-clientgen-no-base-dirs

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. client-gen: remove base input dirs Were these dirs some kind of cargo cult? Removing them causes no change to the generated code. Background: having those packages in the list of input dirs makes them mandatory to exist. If people use code-gen on a project that does not vendor them (due to vendor/ pruning), code-gen fails. Fixes https://github.com/kubernetes/sample-controller/issues/8

Merge pull request #57143 from sttts/sttts-clientgen-no-base-dirs
9a3d2795 · Kubernetes Submit Queue · GitHub · b5387eff · ed1b0109 · 9a3d2795
Unverified Commit 9a3d2795 authored Feb 08, 2018 by Kubernetes Submit Queue Committed by GitHub Feb 08, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 6 deletions

args.go ...ing/src/k8s.io/code-generator/cmd/client-gen/args/args.go +1 -6

No files found.
--- a/staging/src/k8s.io/code-generator/cmd/client-gen/args/args.go
+++ b/staging/src/k8s.io/code-generator/cmd/client-gen/args/args.go
@@ -27,12 +27,7 @@ import (
 	codegenutil "k8s.io/code-generator/pkg/util"
 )
-var DefaultInputDirs = []string{
+var DefaultInputDirs = []string{}
-	"k8s.io/apimachinery/pkg/fields",
-	"k8s.io/apimachinery/pkg/labels",
-	"k8s.io/apimachinery/pkg/watch",
-	"k8s.io/apimachinery/pkg/apimachinery/registered",
-}
 // ClientGenArgs is a wrapper for arguments to client-gen.
 type CustomArgs struct {