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

pgadmin4: add darwin build #302812

Merged
merged 1 commit into from
Apr 10, 2024
Merged

pgadmin4: add darwin build #302812

merged 1 commit into from
Apr 10, 2024

Conversation

gador
Copy link
Member

@gador gador commented Apr 9, 2024

Description of changes

This fixes the build for aarch64-darwin and x86_64-darwin. (Which never really worked before..)

fixexs #281214

Please note: There is an issue on MacOS with keyring: It will ask everytime twice for the admin password when started. Then it will print an error on the logs and one can enter the pgadmin Master password. This can be skipped by saying "don't allow" when promped by MacOS and directly entering the master password.

Issue on pgadmin: pgadmin-org/pgadmin4#7076
Issue upstream: jaraco/keyring#619

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/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 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.

@gador
Copy link
Member Author

gador commented Apr 9, 2024

note: tests fail for x86-64-linux and aarch64-linux. This is also happening on master and should be fixed by #302834

@wegank
Copy link
Member

wegank commented Apr 9, 2024

@ofborg build pgadmin4 pgadmin4.passthru.tests

fixexs NixOS#281214

Signed-off-by: Florian Brandes <florian.brandes@posteo.de>
@wegank
Copy link
Member

wegank commented Apr 10, 2024

Result of nixpkgs-review pr 302812 run on aarch64-darwin 1

4 packages built:
  • pgadmin4
  • pgadmin4-desktopmode
  • pgadmin4-desktopmode.dist
  • pgadmin4.dist

@SuperSandro2000 SuperSandro2000 merged commit ca729af into NixOS:master Apr 10, 2024
23 of 24 checks passed
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