Skip to content

qgis: add nixos tests#252001

Merged
sikmir merged 1 commit intoNixOS:masterfrom
imincik:qgis-nixos-test
Sep 20, 2023
Merged

qgis: add nixos tests#252001
sikmir merged 1 commit intoNixOS:masterfrom
imincik:qgis-nixos-test

Conversation

@imincik
Copy link
Copy Markdown
Contributor

@imincik imincik commented Aug 28, 2023

Description of changes

Add NixOS test for QGIS and QGIS-LTR. The test creates QGIS vector memory layer containing Nix snowflake. This proves that application can successfully start and Python bindings are working.

By default, Python script is executed in non-interactive mode and QGIS is closed after script is finished. This script can be also executed interactively by running following command:

screen-imincik-2023-08-30-12:49:16

 nix-build -A qgis
 QGIS_TEST_INTERACTIVE=True ./result/bin/qgis  --code pkgs/applications/gis/qgis/test.py

In this case, QGIS is not automatically closed.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 23.11 Release Notes (or backporting 23.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.

@github-actions github-actions bot added the 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS label Aug 28, 2023
@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 Aug 28, 2023
@imincik imincik self-assigned this Aug 30, 2023
@imincik imincik mentioned this pull request Aug 30, 2023
12 tasks
@imincik imincik force-pushed the qgis-nixos-test branch 2 times, most recently from b186fa0 to 4396e27 Compare September 1, 2023 20:56
@imincik imincik marked this pull request as ready for review September 1, 2023 20:59
@imincik
Copy link
Copy Markdown
Contributor Author

imincik commented Sep 1, 2023

@timlinux , if you can, please have a look on this QGIS package test PR. Thank you.

@imincik imincik requested review from sikmir and willcohen September 1, 2023 21:03
Add NixOS test for QGIS and QGIS-LTR. This test creates QGIS vector
memory layer containing Nix snowflake. This proves that application can
successfully start and Python bindings are working.

By default, Python script is executed in non-interactive mode and QGIS
is closed after script is finished. This script can be also executed
interactively by running following command:

```
 nix-build -A qgis
 QGIS_TEST_INTERACTIVE=True ./result/bin/qgis  --code pkgs/applications/gis/qgis/test.py
```

In this case, QGIS is not automatically closed.
@imincik
Copy link
Copy Markdown
Contributor Author

imincik commented Sep 2, 2023

@sikmir , thanks for fix !

@imincik imincik requested a review from sikmir September 17, 2023 10:46
@imincik
Copy link
Copy Markdown
Contributor Author

imincik commented Sep 18, 2023

@sikmir , are you OK with merging this PR as it is now ?

@delroth delroth added the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 18, 2023
@sikmir sikmir merged commit 09846ea into NixOS:master Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 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. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants