Makefile remove redundant logic

[alrs]

The verify-gendocs target in the Makefile checks for the existence of the kops binary. By making the kops binary a dependency, this check need not exist.

I've silenced the commands in this target by prepending them with @, so all that the end-user sees is the echoed text, not the shell logic.

[alrs]

/approve
/assign chrislovecnm

[chrislovecnm]

Dumb question of the year. Did you change a flag in create or something, and test that this works?

I am sure it does, but you makefile kungfu is stronger than mine

[alrs]

@chrislovecnm What is create? To test this, I do a make clean and then a make verify-gendocs, which causes the kops and the go-bindata targets to be built, and then verifies the cli docs. To test that it can detect a stale docs/cli I've been deleting docs/cli/kops.md and re-running make verify-gendocs, and it correctly notes that kops.md only exists in the /tmp directory.

[chrislovecnm]

That will work!

[chrislovecnm]

/lgtm

[k8s-github-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alrs, chrislovecnm

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

• OWNERS [chrislovecnm]

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

[k8s-github-robot]

/test all [submit-queue is verifying that this PR is safe to merge]

[k8s-github-robot]

Automatic merge from submit-queue