Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • xserver xserver
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 887
    • Issues 887
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 109
    • Merge requests 109
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • xorg
  • xserverxserver
  • Merge requests
  • !252

Various fixes for both meson and autoconf for building Xwayland without GLamor nor libdrm

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Olivier Fourdan requested to merge ofourdan/xserver:xwayland-build-libdrm into master Aug 01, 2019
  • Overview 7
  • Commits 4
  • Pipelines 2
  • Changes 5

While reviewing !248, Emil (@evelikov) pointed out that Xwayland now requires libdrm.

That was a bit of a surprise to me, because if building without GLamor support (which is supported by Xwayland) and without libdrm (which is possible according to the build system), there is no reason to link against libdrm as it is not required nor used.

Looking at the build system, I found the problem which is present in both autoconf and meson build systems, so I took a bit of time to try to fix that.

I think this is important, as long as we give the option to disable support for things like glamor, to actually take care of not over linking with libs that aren't required in that case.

The following series of patches aim at requiring and linking Xwayland with libdrm and epoxy only when this is needed.

Edited Aug 01, 2019 by Olivier Fourdan
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: xwayland-build-libdrm