Skip to content
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

Allow specification of the binary name/path #1028

Closed
wants to merge 1 commit into from

Conversation

aidy
Copy link
Contributor

@aidy aidy commented Apr 28, 2023

Some container usage expects to run an executable in a specific location, and some projects aren't well laid out to generate binaries with the expected names.

This allows the binary name to be specified in configuration (in a similar way to goreleaser), which should allow for easier migration from other build mechanisms (and usage patterns of their images) to ko.

This does not affect the name or path for Windows builds - the path structure is fundamentally different, and I think that if it's desirable to change the name/path for them, we should use a separate config argument.

connects #944

Some container usage expects to run an executable in a specific
location, and some projects aren't well laid out to generate binaries
with the expected names.

This allows the binary name to be specified in configuration (in a
similar way to goreleaser), which should allow for easier migration from
other build mechanisms (and usage patterns of their images) to `ko`.

This does not affect the name or path for Windows builds - the path
structure is fundamentally different, and I think that if it's desirable
to change the name/path for them, we should use a separate config
argument.
@github-actions
Copy link

This Pull Request is stale because it has been open for 90 days with
no activity. It will automatically close after 30 more days of
inactivity. Keep fresh with the 'lifecycle/frozen' label.

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

Successfully merging this pull request may close these issues.

1 participant