Skip to content
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

More layouts: Generators folder, CMake Toolchain, Editable packages and more #8554

Merged
merged 135 commits into from May 13, 2021

Conversation

lasote
Copy link
Contributor

@lasote lasote commented Feb 25, 2021

Changelog: Feature: Introduced the layout() method to the recipe to be able to declare the folder structure both for the local development methods (conan source, conan build...) and in the cache. Also, associated to the folders, cppinfo objects to be used in editable packages and file pattern descriptions to enable "auto packaging".
Docs: conan-io/docs#2092

  • The layout declares a generator folder (defaulted to install_folder if not declared in layout) where the generators and toolchain files are written. The conaninfo and the other weird stuff is kept in the install_folder (probably to be removed in Conan 2.0)
  • Also new cppinfo objects in the layout entries.
  • The CMake toolchain is injected with an absolute path to the generator folder.
  • A little helper for clion layout as an example (not convinced about interface/UX)
  • Auto package() based on layout.
  • Auto package_info() based on layout.
  • Editables following the build and source layouts.

Copy link
Member

@memsharded memsharded left a comment

Choose a reason for hiding this comment

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

I have pushed a couple of minor things:

  • CMake() was broken for some local flows, because not finding the conanbuild.json file (sorry, no tests, I was checking manually)
  • Redundant layout() call, it was already run inside run_configure_method()

Please check ffaa4cb

@lasote lasote merged commit 8e1aa75 into conan-io:develop May 13, 2021
@czoido czoido added this to the 1.37 milestone May 28, 2021
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.

None yet

3 participants