Skip to content

Conversation

rhubert
Copy link
Contributor

@rhubert rhubert commented Oct 20, 2025

This fixes the build of kmscube for x86_64.

These packages contain header files, which are used by the public headers of
other packages like libX11. Make them dev package is to have them provided.
When building from git sources the autotools build misses some m4 files.
Comment on lines +8 to +10
depends:
- graphics::xorg::utils::util-macros-dev

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, seems unrelated to what the commit message says. I think I even built it without this dependency. Why is it needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right, this is only necessary when building from git sources... I'll keep this as a separate commit on my side..

Comment on lines +8 to +9
- graphics::xorg::proto::xorgproto-dev
- graphics::xorg::utils::util-macros-dev
Copy link
Member

Choose a reason for hiding this comment

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

There is no provideDeps: ['*-dev'] for those. I guess this should be added?

Comment on lines +14 to +15
- graphics::xorg::proto::xorgproto-dev
- graphics::xorg::utils::util-macros-dev
Copy link
Member

Choose a reason for hiding this comment

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

There is no provideDeps: ['*-dev'] for those. I guess this should be added?

PKG_DESCRIPTION: "X transport library"

depends:
- graphics::xorg::utils::util-macros-dev
Copy link
Member

Choose a reason for hiding this comment

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

This looks unrelated as well. Why wasn't that needed before?

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.

2 participants