Added a new command to filter input specs based on their properties#7899
Added a new command to filter input specs based on their properties#7899alalazo wants to merge 1 commit intospack:developfrom
Conversation
Added a new command that permits to filter specs based on their properties. The main use case is to help scripting continuous deployment pipelines.
|
Why not simply add these features to |
|
@adamjstewart The two commands match arguments in different ways:
Wouldn't it be confusing to have the functionality in the same command? |
|
In the beginning I always got confused between I'm not sure if this shouldn't all be done with one interface. I.e. only having a Not sure if this should be done here, but this might serve as a starting point for this discussion |
|
|
|
I think a new name for the command (
👍 |
| import spack | ||
| import spack.cmd | ||
|
|
||
| description = "filter specs based on their properties" |
There was a problem hiding this comment.
Given your comment
spack find matches constraints on what is installed and accepts anonymous specs.
this command accepts only non-anonymous specs and concretizes them the same way install does
this command is principally different from spack find (which is good).
Can you elaborate description to include this discussion?
There was a problem hiding this comment.
p.s. you can also give users some hints where that command might be of use
|
@alalazo anything prevents this PR from being merged? I would really like to use this nice feature, that is waiting to be merged since May 2018 ;-( @adamjstewart @tgamblin could you guys review? |
Added a new command that permits to filter specs based on their properties. The main use case is to help scripting continuous deployment pipelines.
Let me know if this can be of use outside our Jenkins pipeline. Otherwise I'll close the PR.
Example
also fixes #8033