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

zlib: fix MinGW clang #14644

Closed
wants to merge 2 commits into from
Closed

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented Dec 8, 2022

closes #14440


recipes/zlib/all/conanfile.py Outdated Show resolved Hide resolved
@SpaceIm SpaceIm marked this pull request as draft December 8, 2022 14:10
recipes/zlib/all/conanfile.py Outdated Show resolved Hide resolved
@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline

All green in build 3 (da0729bf396fe0a349984c4a1f7bf83cce3937f9):

  • zlib/1.2.13@:
    All packages built successfully! (All logs)

  • zlib/1.2.11@:
    All packages built successfully! (All logs)

  • zlib/1.2.12@:
    All packages built successfully! (All logs)

host_os = self.settings.os
host_compiler = self.settings.get_safe("compiler")
mingw_gcc = host_os == "Windows" and host_compiler == "gcc"
mingw_clang = host_os == "Windows" and host_compiler == "clang" and \
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please added a comment explaining why not that comes next is important for futures contributors who might not know I'd like to try avoiding breaking this again

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe just add a link to conan-io/conan#12678

Copy link
Contributor

Choose a reason for hiding this comment

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

Sure, "for more details see..." it good

@stale
Copy link

stale bot commented Mar 11, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Mar 11, 2023
@stale
Copy link

stale bot commented Apr 11, 2023

This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions.

@CLAassistant
Copy link

CLAassistant commented May 18, 2023

CLA assistant check
All committers have signed the CLA.

@stale stale bot removed the stale label May 18, 2023
@ghost ghost mentioned this pull request Jun 10, 2023
3 tasks
@stale
Copy link

stale bot commented Jun 18, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 18, 2023
@ghost ghost mentioned this pull request Jul 2, 2023
3 tasks
@stale
Copy link

stale bot commented Aug 10, 2023

This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[package] minizip/1.2.13: Windows MinGW Clang fails to compile
4 participants