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

xorg: publish libdrm #8302

Closed
wants to merge 3 commits into from
Closed

Conversation

ericLemanissier
Copy link
Contributor

@ericLemanissier ericLemanissier commented Dec 2, 2021

it is already installed on ubuntu

Specify library name and version: xorg/system

it is used by qtwebengine
also, related to #8024


  • 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.

it is already installed
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@Croydon
Copy link
Contributor

Croydon commented Dec 2, 2021

This is the userspace part of drm, right? Can't we port the Bincrafters recipe for it?

What is the relationship between xorg and drm?

@ericLemanissier
Copy link
Contributor Author

ericLemanissier commented Dec 2, 2021

@Croydon

This is the userspace part of drm, right? Can't we port the Bincrafters recipe for it?

yes, and yes we could, but I'm afraid we'll get back to the problems we had with x11 packages, which led us back to system packages instead of conan built packages. https://gitlab.freedesktop.org/mesa/drm says New functionality in the kernel DRM drivers typically requires a new libdrm, which could be interpreted as "you need to upgrade libdrm with each kernel DRM upgrade", or "you need to upgrade libdrm if you want to use a new feature of kernel DRM driver"

What is the relationship between xorg and drm?

I don't know, but I assume there is one, because apt already install libdrm-dev when installing xorg/system (it is an indirect dependency of xorg-dev)

@conan-center-bot
Copy link
Collaborator

All green in build 3 (d936de87157d7040383325204a6cc612157eeab2):

  • xorg/system@:
    All packages built successfully! (All logs)

@ericLemanissier
Copy link
Contributor Author

I asked the question about system package vs conan package upstream: https://gitlab.freedesktop.org/mesa/drm/-/issues/75

@ericLemanissier ericLemanissier mentioned this pull request Dec 3, 2021
4 tasks
@ericLemanissier
Copy link
Contributor Author

Please hold your reviews why we decide if it's system or not

@prince-chrismc
Copy link
Contributor

Please convert as draft!

@ericLemanissier ericLemanissier marked this pull request as draft December 5, 2021 21:32
@Croydon
Copy link
Contributor

Croydon commented Dec 6, 2021

I asked the question about system package vs conan package upstream: https://gitlab.freedesktop.org/mesa/drm/-/issues/75

Thanks for asking upstream!

Independent from the question system package or not, I think that it should be separated from the xorg package. I believe that Wayland is using libdrm too

@ghost
Copy link

ghost commented Dec 10, 2021

I detected other pull requests that are modifying xorg/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.

@ghost ghost mentioned this pull request Dec 10, 2021
4 tasks
@ericLemanissier ericLemanissier deleted the patch-5 branch December 10, 2021 14:32
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