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
Use conan-io/hello example for the new command #5069
Conversation
There could be some places in the docs where we are using this |
I have included the changes in the documentation too |
@@ -244,7 +244,7 @@ def build_local_different_folders_test(self): | |||
install_dir = os.path.join(client.current_folder, "install_x86_64") | |||
tools.replace_in_file(os.path.join(client.current_folder, "src", "hello", "CMakeLists.txt"), | |||
"${CMAKE_BINARY_DIR}/conanbuildinfo.cmake", | |||
'"%s/conanbuildinfo.cmake"' % install_dir, |
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.
why necessary? It is not failing so far
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.
I was wondering the same and I have no idea...
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.
It failed in this CI job, I would want to know what's happening, but if it is needed, this PR is ok for me and ready to merge.
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.
LGTM, just remove the cd hello
pointed out by memsharded.
Thanks!
Changelog: Feature: Updated the generated conanfile.py in
conan new
to the new [conan-io/hello].(https://github.com/conan-io/hello) repositoryDocs: conan-io/docs#1269
develop
branch, documenting this one.Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.