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.

Sign up for GitHub

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

Jump to bottom

[BPT] Header only libs have limited job matrix. #1391

Open
grafikrobot opened this issue May 2, 2021 · 2 comments
Open

[BPT] Header only libs have limited job matrix. #1391

grafikrobot opened this issue May 2, 2021 · 2 comments
Labels
bug

Comments

@grafikrobot
Copy link
Member

Description of Problem, Request, or Question

The job matrix generated by BPT for header only libraries seems to only contain four builds: Linux release+debugs, and Windows release+debug. They..

  • Are unlabeled "generic" build, i.e. without compiler, architecture, etc.
  • Do not include MacOS with Xcode.

Package and Environment Details (include every applicable attribute)

  • Conan version: Conan 1.36.0
  • Python version: Python 3.9.4
@grafikrobot grafikrobot added the bug label May 2, 2021
grafikrobot added a commit to bincrafters/bincrafters-package-tools that referenced this issue May 2, 2021
@grafikrobot
Test remove special matrix for header only.
6cedbee 
This temporarily removes the headers only matrix config to see what it
would take to address bincrafters/community#1391
@Croydon
Copy link
Member

Croydon commented May 2, 2021

What are you trying to archive with that? Adding all potential build jobs for header_only does not seem reasonable I think 🤔

@grafikrobot
Copy link
Member Author

The issue is that some header only libraries have per configuration definitions and linking requirements. And outside of one person locally testing all those requirements CI is the only other way to check that they work. For example the olc-pge library that I've been trying to get into CCI conan-io/conan-center-index#5283. Which I'm now trying to provide outside of CCI https://github.com/bfgroup/conan/actions/runs/803704935. In those bfgroup/conan results I hacked BPT to do the full matrix bincrafters/bincrafters-package-tools@6cedbee.

grafikrobot added a commit to bincrafters/bincrafters-package-tools that referenced this issue Jul 14, 2021
@grafikrobot
Makes it possible for header only libs to be buidl tested.
0c22e0b 
This adds the concept of "testable header only" recipes. Those are
recipes for header only libraries that also include a "test_package"
that should be built across configurations to check for correct
functioning of header libs that have platform and/or system
requirements. This resolves issue
bincrafters/community#1391
@grafikrobot grafikrobot mentioned this issue Jul 14, 2021
Closed
@Croydon Croydon mentioned this issue Oct 3, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Croydon @grafikrobot