Skip to content

Conversation

@thoughton
Copy link
Contributor

A "C++98/C++11 single-header testing framework for unit tests and TDD".

A "C++98/C++11 single-header testing framework for unit tests and TDD".
@@ -0,0 +1,2 @@
onqtam/doctest@master
Copy link
Owner

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?

Copy link
Contributor Author

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?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it generally preferred for recipes not to pull from master?

Yes, as it helps keep the recipes reproducible.

Copy link
Contributor Author

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.

Copy link
Owner

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 and onqtam/doctest/2.0/package.txt and then you can refer to it as onqtam/doctest@1.0 or onqtam/doctest@2.0. Both packages are given the same name as onqtam/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.

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

Successfully merging this pull request may close these issues.

2 participants