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

libxft: conan v2 support #17485

Closed
wants to merge 1 commit into from
Closed

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented May 9, 2023

closes #17818


@conan-center-bot

This comment has been minimized.

@SpaceIm
Copy link
Contributor Author

SpaceIm commented May 9, 2023

checking for gcc... gcc
checking whether the C compiler works... no
configure: error: in `/home/conan/w/prod-v2/BuildSingleReference/p/t/libxfa520101bafe08/b/build-release':
configure: error: C compiler cannot create executables
See `config.log' for more details

libxft/2.3.4: ERROR: 
Package '24839a77c7fe0258fd2778813f88fa1914ece8ce' build failed
libxft/2.3.4: WARN: Build folder /home/conan/w/prod-v2/BuildSingleReference/p/t/libxfa520101bafe08/b/build-release

Works fine for me on Linux, I can't know what's wrong here without the content of config.log

@CLAassistant
Copy link

CLAassistant commented May 18, 2023

CLA assistant check
All committers have signed the CLA.

@ghost
Copy link

ghost commented May 22, 2023

I detected other pull requests that are modifying libxft/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@conan-center-bot

This comment has been minimized.

@jasuramme
Copy link

@SpaceIm here you go, pal!
config.log

@jasuramme
Copy link

jasuramme commented Jun 6, 2023

@SpaceIm
I think it links across system xorg libraries, but takes libuuid from conan repo.
And Xorg libraries need libuuid1.0, but in conan we have 1.03
It is a conflict between xorg/system and fontconfig/2.14.2
UPD:
The problem happens when we build with */*:shared. I believe there should be some proper mechanism to link system libraries statically only. Otherwise, we would have dependencies conflicts all the way down

@ericLemanissier
Copy link
Contributor

See #17664

@jasuramme
Copy link

See #17664

@ericLemanissier should we wait until util-linux-libuuid is released and add it to the requirements after that? Is it correct?

@ericLemanissier
Copy link
Contributor

If we have no reason to think that libxft uses libuuid directly, then we should not add it to its requirements. Instead, we should wait for util-linux-libuuid to be merged, then upgrade the fontconfig recipe to require util-linux-libuuid instead of libuuid. Then libxft should pass

@jasuramme
Copy link

If we have no reason to think that libxft uses libuuid directly, then we should not add it to its requirements. Instead, we should wait for util-linux-libuuid to be merged, then upgrade the fontconfig recipe to require util-linux-libuuid instead of libuuid. Then libxft should pass

Oh sure, fontconfig, not the libxft. I've lost context after the long working day. Thanks!

@ericLemanissier

This comment was marked as resolved.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@ericLemanissier
Copy link
Contributor

this one can be re-triggered now that #17995 is merged

@ghost ghost mentioned this pull request Jul 2, 2023
3 tasks
@ghost ghost mentioned this pull request Jul 15, 2023
@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Sorry, the system is under maintenance and it doesn't accept builds right now.
Please, check https://status.conan.io to obtain more information.
Thanks for your understanding and help with the Conan Center Index!


Conan v2 pipeline ❌

Note: Conan v2 builds may be required once they are on the v2 ready list

The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future.

See details:

Sorry, the system is under maintenance and it doesn't accept builds right now.
Please, check https://status.conan.io to obtain more information.
Thanks for your understanding and help with the Conan Center Index!

@stale
Copy link

stale bot commented Oct 15, 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 Oct 15, 2023
@ghost ghost mentioned this pull request Oct 28, 2023
3 tasks
@ghost ghost mentioned this pull request Nov 20, 2023
3 tasks
@ghost ghost mentioned this pull request Jan 17, 2024
3 tasks
@jwillikers
Copy link
Contributor

Can this be triggered again?

@stale stale bot removed the stale label Jan 17, 2024
@jwillikers jwillikers mentioned this pull request Jan 31, 2024
7 tasks
Copy link
Contributor

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.

@github-actions github-actions bot added the stale label Feb 24, 2024
@github-actions github-actions bot removed the stale label Feb 26, 2024
@jcar87
Copy link
Contributor

jcar87 commented Mar 6, 2024

Closing in favour of #18789.

Please feel free to reopen and update if something was missing.

@jcar87 jcar87 closed this Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[request] LibXft/2.3.8 conan2 migration
9 participants