-
Notifications
You must be signed in to change notification settings - Fork 186
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
Command "hostname" is not portable #1212
Comments
What cases are fixed by this, I'd rather think it's the other way around ( Also, if one go for the environment variable, is there some "this runs on the target machine" magic that would explain the execution rather than picking it up from the environment variables directly? |
An alternative is to use the Would that be a better solution @pontus @pditommaso ? |
I'm mostly confused, so I'm not sure. In my view An optimisation (if there is no magic involved) could be to look at the environment first (in groovy) and run the command if there is no hostname, don't think it's worth the complexity though. |
Ok. This should be fixed in #1213 Can change if there is another, better solution :) |
@Zethson likely can provide more details about the env in which |
A fresh Arch installation without |
May you check if the var |
It's empty. |
Then |
Changed here #1214 |
The
hostname
command may not exist in all environments and cause the pipeline execution to failtools/nf_core/pipeline-template/lib/NfcoreTemplate.groovy
Line 27 in b989701
Consider replacing it with
x-ref nextflow-io/nextflow#2199
The text was updated successfully, but these errors were encountered: