Skip to content

Conversation

vivekadishankara
Copy link

I am new to bevy, so I just cloned the repository and did a cargo build. That gave me the error:

/home/einfochips/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/wayland-sys-0.31.7/build.rs:10:47:
  called `Result::unwrap()` on an `Err` value: Failed to run command `PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 PKG_CONFIG_ALLOW_SYSTEM_CFLAGS=1 pkg-config --libs --cflags wayland-client`, because: Not a directory (os error 20)

then I discovered that my system runs on x11, but when i ran the example:

cargo run --features x11 --example hello_world

I still got the error.
This PR is trying to fix this.

Objective

Since engine runs on wayland by default, the defaults need to be disabled for running on x11.

Solution

The first example of cargo run in example/README.md needs to specify how to do this. Hence a new example is added.

Testing

I did run the cargo my self and the newly added example works on my system, the command being:

cargo run --no-default-features --features x11 --example hello_world

Copy link
Contributor

github-actions bot commented Oct 6, 2025

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@alice-i-cecile alice-i-cecile added the C-Docs An addition or correction to our documentation label Oct 6, 2025
@alice-i-cecile
Copy link
Member

You'll need to fix CI, but this seems helpful and the wording is good.

@alice-i-cecile alice-i-cecile added C-Examples An addition or correction to our examples O-Linux Specific to the Linux desktop operating system S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Oct 6, 2025
@hukasu hukasu added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Oct 7, 2025
@vivekadishankara
Copy link
Author

This might not be a good idea because when i tried to run other examples they failed to compile because other default features were required

@alice-i-cecile
Copy link
Member

Related to #21158; I think this needs to be blocked on that.

@alice-i-cecile alice-i-cecile added S-Blocked This cannot move forward until something else changes and removed S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Oct 8, 2025
@vivekadishankara
Copy link
Author

There is one problem in the current condition. The only way i can run the examples is to manually comment out the wayland default in the cargo toml. Currently there is no way to disable only one default.

@vivekadishankara
Copy link
Author

I jumped to a wayland system now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-Docs An addition or correction to our documentation C-Examples An addition or correction to our examples O-Linux Specific to the Linux desktop operating system S-Blocked This cannot move forward until something else changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants