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

svector: add recipe #13170

Merged
merged 7 commits into from
Oct 4, 2022
Merged

Conversation

toge
Copy link
Contributor

@toge toge commented Sep 27, 2022

Specify library name and version: svector/1.0.2

svector is compact optimized vector library.


  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the conan-center hook activated.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

Co-authored-by: Uilian Ries <uilianries@gmail.com>
@conan-center-bot

This comment has been minimized.

Co-authored-by: Uilian Ries <uilianries@gmail.com>
@toge toge mentioned this pull request Sep 27, 2022
4 tasks
@conan-center-bot

This comment has been minimized.

@toge
Copy link
Contributor Author

toge commented Sep 27, 2022

@uilianries
Umm...
I'm afraid my understand is wrong.
This PR failed to build.
Shall I write self.info.settings routine in build_validate()?
Or shall I wait for future conan release?

@uilianries
Copy link
Member

@toge You are right, I thought it has a different behavior when testing and installing (I tested it before), but I'm wrong. Checking again, it's always None, the compiler from settings.

So I opened the issue conan-io/conan#12210 to get some from Conan devs.

I'm sure if we use self.settings.compiler it would work, but that's not what we want.

@conan-center-bot
Copy link
Collaborator

All green in build 5 (3081b13f706c8569cab093d1a6829f8d984aedda):

  • svector/1.0.2@:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit 7da1e9c into conan-io:master Oct 4, 2022
System-Arch pushed a commit to System-Arch/conan-center-index that referenced this pull request Oct 7, 2022
* svector: add recipe

* fix description, license

* remove unused function

* remove comment of template

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* use `self.info.settings`

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* use self.settings instead of self.info.settings (workaround)

Co-authored-by: Uilian Ries <uilianries@gmail.com>
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.

4 participants