Skip to content

[Build] Optimize makefile to auto remove outdated binary tools #214

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

yansun1996
Copy link
Contributor

  • Issues:
    Currently when we changed the binary tools version, the old binaries won't get auto updated. For example we have updated the controller-gen version over different branches, when we switch among branches to do the build, the outdated controller-gen could result in massive auto-generated files differences.

  • Improvement:
    In the Makefile, if the binary tool is existing, check the version. If the version mismatch with the desired version, re-download the binary tool so that the correct version tool could be used for building the project.

@yansun1996 yansun1996 force-pushed the auto_update_binary branch from 1f31d17 to 63ea84d Compare June 17, 2025 06:37
Copy link

@shiv-tyagi shiv-tyagi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The docker/shell target would also need fixing. The bash -c from the front needs to go away.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants