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

freetype: add package_type + fix check of custom CMake variables in test package #16456

Merged

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented Mar 8, 2023

same fix than #16389 (comment), but for test package of freetype.


@conan-center-bot

This comment has been minimized.

@SpaceIm SpaceIm closed this Mar 8, 2023
@SpaceIm SpaceIm reopened this Mar 8, 2023
@SpaceIm SpaceIm closed this Mar 8, 2023
@SpaceIm SpaceIm reopened this Mar 8, 2023
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@SpaceIm
Copy link
Contributor Author

SpaceIm commented Mar 14, 2023

This PR has been triggered again because there are existing changes in master branch that are related to these same references resulting in different recipe-revisions. The build is being triggered on the PR context in case some additional fix is needed after merging those changes.

@danimtb @uilianries c3i reported successful build of this PR both in v1 & v2 pipelines 5 days ago. Now there is this message while there was 0 modification of freetype in master branch in the meantime, nor in any dependency of freetype. I see also that continuous-integration/jenkins/pr-merge is now silently marked failed 😞

@conan-center-bot

This comment has been minimized.

@SpaceIm SpaceIm closed this Mar 14, 2023
@SpaceIm SpaceIm reopened this Mar 14, 2023
@prince-chrismc
Copy link
Contributor

v2 is running happily so far :)

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 7 (4f43acacf3145cc82f2047f9b1bc9f5536fc5fcd):

  • freetype/2.13.0@:
    All packages built successfully! (All logs)

  • freetype/2.12.1@:
    All packages built successfully! (All logs)

  • freetype/2.11.1@:
    All packages built successfully! (All logs)

  • freetype/2.10.4@:
    All packages built successfully! (All logs)


Conan v2 pipeline (informative, not required for merge) ✔️

Note: Conan v2 builds are informative and they are not required for the PR to be merged.

All green in build 6 (4f43acacf3145cc82f2047f9b1bc9f5536fc5fcd):

  • freetype/2.13.0@:
    All packages built successfully! (All logs)

  • freetype/2.12.1@:
    All packages built successfully! (All logs)

  • freetype/2.11.1@:
    All packages built successfully! (All logs)

  • freetype/2.10.4@:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit 60c993e into conan-io:master Mar 16, 2023
@SpaceIm SpaceIm deleted the fix/freetype-test-cmake-vars branch March 16, 2023 13:20
0xFireWolf pushed a commit to 0xFireWolf/conan-center-index that referenced this pull request Apr 2, 2023
…ke variables in test package

* add package_type

* fix test of custom CMake variables

* robust definition of FREETYPE_VERSION_STRING
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.

5 participants