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

Add support for custom bloopInstall scripts #359

Open
crt-31 opened this issue Oct 5, 2023 · 2 comments
Open

Add support for custom bloopInstall scripts #359

crt-31 opened this issue Oct 5, 2023 · 2 comments

Comments

@crt-31
Copy link

crt-31 commented Oct 5, 2023

Is your feature request related to a problem? Please describe.

Metals should have a way to configure a custom bloopInstall script or program. This would mean that on 'import build' Metals would call this script (which would create the bloop json files), and then Metals would then use Bloop as the build server.

Describe the solution you'd like

I'd envision adding some fields to the *.bsp file: maybe add a field called "type" and set it to "bloop", and then have another field called "bloopInstallScript". Maybe could have other parameters to configure the bloop like bloopDir, etc. Putting in a .bsp file would be natural and allow for multiple different ones.

The main purpose of this feature would be for build tools that are not currently directly supported by Metals (i.e. being developed or experimenting, or just want to be separate), to have a way for them to generate the bloop json files in same way as the supported tools do.

Thanks.

Describe alternatives you've considered

.

Additional context

No response

Search terms

bloopInstall bloop custom

@tgodzik
Copy link
Contributor

tgodzik commented Oct 5, 2023

Thanks for reporting! Potentially, you should be able to reuse one of the existing scripts in the settings such the one for mill or maven and just ignore the arguments metals sends. This is a workaround though.

In the future we might want to merge the setting into one with type (be it mill, maven, sbt, gradle or other) and with other you could potentially define the whole command.

@tgodzik
Copy link
Contributor

tgodzik commented Oct 5, 2023

Thanks for reporting! Potentially, you should be able to reuse one of the existing scripts in the settings such the one for mill or maven and just ignore the arguments metals sends. This is a workaround though.

Although you would also need a fake pom.xml or build.sc :/ for metals to detect it as such

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

No branches or pull requests

2 participants