Skip to content

alacritty: fix cross#225119

Merged
Mic92 merged 1 commit intomasterfrom
unknown repository
Apr 8, 2023
Merged

alacritty: fix cross#225119
Mic92 merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Apr 7, 2023

This commit fixes cross compilation of alacritty by:

  1. If xdg-utils.meta.broken (see xdg-utils: mark broken if cross #225112 (comment)), late-bind it via runtime $PATH rather than compile-time reference.

  2. Use $STRIP instead of strip

@ofborg ofborg bot added the 6.topic: cross-compilation Building packages on a different platform than they will be used on label Apr 7, 2023
This commit fixes cross compilation of alacritty by:

1. If xdg-utils.meta.broken (see
   #225112 (comment)),
   late-bind it via runtime $PATH rather than compile-time
   reference.

2. Use `$STRIP` instead of `strip`
@ghost ghost requested a review from NickCao April 7, 2023 20:54
@ofborg ofborg bot requested review from Br1ght0ne and Mic92 April 7, 2023 21:12
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Apr 7, 2023
@Mic92 Mic92 merged commit 456d819 into NixOS:master Apr 8, 2023
@ghost ghost deleted the pr/alacritty/fix-cross branch April 8, 2023 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: cross-compilation Building packages on a different platform than they will be used on 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants