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

Build options leak when building a target's dependency. #27456

Open
ericonr opened this issue Dec 26, 2020 · 6 comments
Open

Build options leak when building a target's dependency. #27456

ericonr opened this issue Dec 26, 2020 · 6 comments
Labels
bug Something isn't working

Comments

@ericonr
Copy link
Member

ericonr commented Dec 26, 2020

My test case is with efl and enlightenment; to reproduce, you simply need to increase the revision in both packages, then try to build enlightenment, with ./xbps-src pkg enlightenment. When it doesn't find the up to date efl dependency, it will attempt to build it, and fail, as seen here:

=> xbps-src: updating repositories for host (x86_64)...
[*] Updating repository `https://mirror.clarkson.edu/voidlinux/current/x86_64-repodata' ...
[*] Updating repository `https://mirror.clarkson.edu/voidlinux/current/nonfree/x86_64-repodata' ...
[*] Updating repository `https://mirror.clarkson.edu/voidlinux/current/debug/x86_64-repodata' ...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/multilib/x86_64-repodata' ...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/multilib/nonfree/x86_64-repodata' ...
=> xbps-src: updating software in / masterdir...
=> xbps-src: cleaning up / masterdir...
=> enlightenment-0.24.2_1: removing autodeps, please wait...
=> enlightenment-0.24.2_1: the following build options are set:
   wayland: Enable support for the Wayland backend (ON)
=> enlightenment-0.24.2_1: building [meson] for x86_64...
   [host] efl-1.24.4_1: not found
   [host] pkg-config-0.29.2_2: found (https://mirror.clarkson.edu/voidlinux/current)
   [host] gettext-0.21_2: found (https://mirror.clarkson.edu/voidlinux/current)
   [host] xorg-server-xwayland-1.20.10_2: found (https://mirror.clarkson.edu/voidlinux/current)
   [host] meson-0.55.3_3: found (https://mirror.clarkson.edu/voidlinux/current)
   [target] gettext-devel-0.21_2: found (https://mirror.clarkson.edu/voidlinux/current)
   [target] efl-devel-1.24.4_1: not found
   [target] pam-devel-1.3.0_2: found (https://mirror.clarkson.edu/voidlinux/current)
   [target] xkeyboard-config-2.31_1: found (https://mirror.clarkson.edu/voidlinux/current)
   [target] libxkbcommon-devel-1.0.3_1: found (https://mirror.clarkson.edu/voidlinux/current)
   [target] MesaLib-devel-20.3.1_1: found (https://mirror.clarkson.edu/voidlinux/current)
   [target] wayland-devel-1.18.0_3: found (https://mirror.clarkson.edu/voidlinux/current)
   [runtime] desktop-file-utils-0.26_1: found (https://mirror.clarkson.edu/voidlinux/current)
   [runtime] hicolor-icon-theme-0.17_3: found (https://mirror.clarkson.edu/voidlinux/current)
   [runtime] xkeyboard-config-2.31_1: found (https://mirror.clarkson.edu/voidlinux/current)
   [runtime] xorg-server-xwayland-1.20.10_2: found (https://mirror.clarkson.edu/voidlinux/current)
   [runtime] efl-devel-1.24.4_1: not found
=> ERROR: efl-1.24.4_1: cannot be built, it's currently broken; see the build log:
=> ERROR: efl-1.24.4_1: wayland requires drm

I assume this happens because enlightenment has build_options_default=wayland, while efl has build_options_default="elogind harfbuzz pulseaudio gstreamer x11 wayland opengl gles2 drm glib ibus". efl's options are overridden, and we (luckily) get an error, because the wayland build option requires drm.

@ericonr ericonr added the bug Something isn't working label Dec 26, 2020
@fosslinux
Copy link
Contributor

Oddly, even when I set XBPS_PKG_OPTS_efl, this issue still occurs.

@ericonr
Copy link
Member Author

ericonr commented Aug 12, 2021

https://build.voidlinux.org/builders/aarch64_builder/builds/34101/steps/shell_3/logs/stdio shows another instance of leakage. enchant2 for the host fails to build because it was launched from within a cross build for OpenRCT2.

IMO we need to completely revamp how unavailable dependencies are built from within xbps-src.

@Duncaen
Copy link
Member

Duncaen commented Aug 12, 2021

Yes would be nice to rework xbps-src to just fork instead of trying to source and later unset a bunch of stuff, there is really no benefit to it afaik.

@github-actions
Copy link

github-actions bot commented May 1, 2022

Issues become stale 90 days after last activity and are closed 14 days after that. If this issue is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label May 1, 2022
@ericonr
Copy link
Member Author

ericonr commented May 1, 2022

Still an issue

@ericonr ericonr removed the Stale label May 1, 2022
@github-actions
Copy link

Issues become stale 90 days after last activity and are closed 14 days after that. If this issue is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label Jul 31, 2022
@q66 q66 removed the Stale label Jul 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants