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

dbeaver-bin: fix build on aarch64-linux #381766

Merged
merged 2 commits into from
Feb 14, 2025

Conversation

gepbird
Copy link
Contributor

@gepbird gepbird commented Feb 13, 2025

Fixes https://hydra.nixos.org/build/288415437

Previously I ignored libc.so.8 (wanted on x86_64-linux) as it doesn't exist on linux and thought it was an upstream packaging issue, but after a bit of research I realized it's used on BSD. However dbeaver itself does not support BSD, so we can safely remove that. Similarly something wants a libc.so.7 on aarch64-linux, and there are a bunch of other platforms that we don't need.

directories removed
  • aix-ppc/
  • aix-ppc64/
  • dragonflybsd-x86-64/
  • freebsd-aarch64/
  • freebsd-x86/
  • freebsd-x86-64/
  • linux-arm/
  • linux-armel/
  • linux-loongarch64/
  • linux-mips64el/
  • linux-ppc/
  • linux-ppc64le/
  • linux-riscv64/
  • linux-s390x/
  • linux-x86/
  • openbsd-x86/
  • openbsd-x86-64/
  • sunos-sparc/
  • sunos-sparcv9/
  • sunos-x86/
  • sunos-x86-64/
  • win32/
  • win32-aarch64/
  • win32-x86/
  • win32-x86-64/

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@gepbird
Copy link
Contributor Author

gepbird commented Feb 13, 2025

Fixed another small issue on aarch64-linux

@gepbird gepbird marked this pull request as ready for review February 13, 2025 14:31
@gepbird gepbird changed the title dbeaver-bin: remove files for unsupported architectures dbeaver-bin: fix build on aarch64-linux Feb 13, 2025
@gepbird
Copy link
Contributor Author

gepbird commented Feb 13, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 381766


x86_64-linux

✅ 1 package built:
  • dbeaver-bin

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 381766


aarch64-linux

✅ 1 package built:
  • dbeaver-bin

@yzx9
Copy link
Contributor

yzx9 commented Feb 14, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 381766


aarch64-darwin

❌ 1 package failed to build:
  • dbeaver-bin

It looks like there's an issue with the Darwin build. I'll take a look.

@gepbird gepbird force-pushed the dbeaver-remove-unsupported-arch branch from c4b2bfa to d526500 Compare February 14, 2025 09:28
@gepbird gepbird force-pushed the dbeaver-remove-unsupported-arch branch from d526500 to e1584ee Compare February 14, 2025 10:12
@yzx9
Copy link
Contributor

yzx9 commented Feb 14, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 381766


aarch64-darwin

✅ 1 package built:
  • dbeaver-bin

@mkg20001 mkg20001 merged commit 5d18af8 into NixOS:master Feb 14, 2025
26 of 27 checks passed
@gepbird gepbird deleted the dbeaver-remove-unsupported-arch branch February 14, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants