Skip to content

firefox: late-bind xdg-utils if broken#225115

Merged
mweinelt merged 1 commit intomasterfrom
unknown repository
Jul 21, 2023
Merged

firefox: late-bind xdg-utils if broken#225115
mweinelt merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Apr 7, 2023

xdg-utils does not cross-compile. Let's late-bind (using the runtime $PATH) in this scenario so a native-built xdg-utils can be used instead.

See also:

With this PR and #225112 I am able to nix build pkgsCross.aarch64-multiplatform.firefox.

xdg-utils does not cross-compile.  Let's late-bind (using the
runtime $PATH) in this scenario so a native-built xdg-utils can be
used instead.
@ghost ghost requested a review from mweinelt as a code owner April 7, 2023 07:25
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Apr 7, 2023
@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 20, 2023

@mweinelt do you have any objections to this?

@mweinelt
Copy link
Copy Markdown
Member

As I understand it, this will break opening URIs through xdg-open, which is meh. Is xdg-utils cross really unfixable?

@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 21, 2023

As I understand it, this will break opening URIs through xdg-open

... instead of breaking the entire build of firefox. Yes.

Is xdg-utils cross really unfixable?

It's a pretty awful mess of Perl that was not written with cross-compilation in mind.

@mweinelt mweinelt merged commit b6e5966 into NixOS:master Jul 21, 2023
@ghost ghost deleted the pr/firefox/late-bind-xdg-utils branch July 24, 2023 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant