Summary
gradle.properties ships a hardcoded, machine-specific default for snakemakeWrappersRepoPath that points at a JetBrains developer's home directory:
|
snakemakeWrappersRepoPath=/Users/romeo/work/git_repos/snakecharm_related/snakemake-wrappers |
snakemakeWrappersRepoPath=/Users/romeo/work/git_repos/snakecharm_related/snakemake-wrappers
Because this path doesn't exist on anyone else's machine, ./gradlew buildPlugin fails for outside contributors at :buildWrappersBundle (which runs SmkWrapperCrawler with this path):
Execution failed for task ':buildWrappersBundle'.
> Process 'command '.../bin/java'' finished with non-zero exit value 1
./gradlew test is unaffected (it uses buildTestWrappersBundle with the in-repo testData/wrappers_storage), so only the distributable-ZIP path is broken.
Suggestion
Make the default contributor-friendly, e.g. one of:
- default
snakemakeWrappersRepoPath to a repo-relative location (or leave it unset and have buildWrappersBundle skip/emit an empty bundle when it's missing), and document the -PsnakemakeWrappersRepoPath=<path> override; and/or
- document in
DEVELOPER.md that buildPlugin requires a local snakemake-wrappers checkout (tag matching snakemakeWrappersRepoVersion, currently v7.2.0) with the path overridden.
There's also a commented machine-specific path at gradle.properties#L65 (#platformLocalPath=/Users/romeo/...) that could be cleaned up at the same time.
Found while building the plugin locally to test 2026.1 compatibility (see #569 / #570).
Summary
gradle.propertiesships a hardcoded, machine-specific default forsnakemakeWrappersRepoPaththat points at a JetBrains developer's home directory:snakecharm/gradle.properties
Line 109 in aa529e6
snakemakeWrappersRepoPath=/Users/romeo/work/git_repos/snakecharm_related/snakemake-wrappersBecause this path doesn't exist on anyone else's machine,
./gradlew buildPluginfails for outside contributors at:buildWrappersBundle(which runsSmkWrapperCrawlerwith this path):./gradlew testis unaffected (it usesbuildTestWrappersBundlewith the in-repotestData/wrappers_storage), so only the distributable-ZIP path is broken.Suggestion
Make the default contributor-friendly, e.g. one of:
snakemakeWrappersRepoPathto a repo-relative location (or leave it unset and havebuildWrappersBundleskip/emit an empty bundle when it's missing), and document the-PsnakemakeWrappersRepoPath=<path>override; and/orDEVELOPER.mdthatbuildPluginrequires a localsnakemake-wrapperscheckout (tag matchingsnakemakeWrappersRepoVersion, currentlyv7.2.0) with the path overridden.There's also a commented machine-specific path at
gradle.properties#L65(#platformLocalPath=/Users/romeo/...) that could be cleaned up at the same time.Found while building the plugin locally to test 2026.1 compatibility (see #569 / #570).