Skip to content

yojimbo: conan 2 support with new premake generator #27827

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

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

perseoGI
Copy link
Contributor

@perseoGI perseoGI commented Jul 1, 2025

Summary

Changes to the recipe: yojimbo/1.2.5

This recipe now utilizes the new Premake Generator, which was merged in conan-io/conan#17898 and released in Conan v2.18.

A complete refactor of the recipe has been performed by incorporating the PremakeToolchain, significantly simplifying the previous version.

Additionally, the version has been updated from 1.2.1 to 1.2.5, removing the restriction that limited it to the x86_64 target only.

@perseoGI perseoGI marked this pull request as draft July 2, 2025 07:16
from conan.tools.premake import Premake, PremakeDeps, PremakeToolchain


required_conan_version = ">=2.18.0"
Copy link

Choose a reason for hiding this comment

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

Suggested change
required_conan_version = ">=2.18.0"
required_conan_version = ">=2.19.0"

Looks like conan-io/conan#18572 is required to fix this build issue.

Error: C:/workspace/cci_prod_PR-27827/conan-home/p/b/yojim66f270c3c143d/b/build-release/conanfile.premake5.lua:3: invalid escape sequence near '"C:\w'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh yes thank you,

I'm aware of it @Werni2A!
I was just trying to fix Linux CI.

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