Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to use a version tag instead of always pulling from master?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm, yes the repo does seem to have tags for each of its releases, so I guess those could be used instead.
Is it generally preferred for recipes not to pull from master?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, as it helps keep the recipes reproducible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahha, I see. Yes that makes sense.
Does that also imply then, that if two different users both wanted reproducible setups, but using two different versions of a library, then we might want two different recipes for that same library (eg. with the version number in the recipe name perhaps) in cget?
Apologies for all the questions, but I'm genuinely interested in finding out what the ideal cget workflow is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cget doesn't really support installing two different versions in the same cget directory(as two packages are not allowed to install the same file).
You can also add multiple versions of recipe under the same directory, such as
onqtam/doctest/1.0/package.txt
andonqtam/doctest/2.0/package.txt
and then you can refer to it asonqtam/doctest@1.0
oronqtam/doctest@2.0
. Both packages are given the same name asonqtam/doctest
. Currently, there is no SAT solving, just first-come-first-serve(ie cget just checks if the package is already installed, and then installs). So to fix a correct version, the user should install it first or use a set of recipes that are know to work together.Of course, this recipe is only useful for picking a usable version. So I dont see the need to add multiple recipes as the user can already do
onqtam/doctest@1.0
to pick a certain tag and install.