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

goxel: 0.7.3 -> 0.8.0 #39768

Merged
merged 1 commit into from May 1, 2018
Merged

goxel: 0.7.3 -> 0.8.0 #39768

merged 1 commit into from May 1, 2018

Conversation

r-ryantm
Copy link
Contributor

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.

This update was made based on information from https://repology.org/metapackage/goxel/versions.

These checks were done:

  • built on NixOS
  • ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel --help’ got 0 exit code
  • ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel -V’ and found version 0.8.0
  • ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel --version’ and found version 0.8.0
  • ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped --help’ got 0 exit code
  • ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped -V’ and found version 0.8.0
  • ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped --version’ and found version 0.8.0
  • found 0.8.0 with grep in /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0
  • directory tree listing: https://gist.github.com/ee8a96a0b785c0293e1e477b693c483b

cc @tilpner for review

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.

This update was made based on information from https://repology.org/metapackage/goxel/versions.

These checks were done:

- built on NixOS
- ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel --help’ got 0 exit code
- ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel -V’ and found version 0.8.0
- ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel --version’ and found version 0.8.0
- ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped --help’ got 0 exit code
- ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped -V’ and found version 0.8.0
- ran ‘/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped --version’ and found version 0.8.0
- found 0.8.0 with grep in /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0
- directory tree listing: https://gist.github.com/ee8a96a0b785c0293e1e477b693c483b
@GrahamcOfBorg
Copy link

No attempt on x86_64-darwin (full log)

The following builds were skipped because they don't evaluate on x86_64-darwin: goxel

Partial log (click to expand)


a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.


@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: goxel

Partial log (click to expand)

glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/goxel
shrinking RPATHs of ELF executables and libraries in /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0
shrinking /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin/.goxel-wrapped
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0/bin
patching script interpreter paths in /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0
checking for references to /build in /nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0...
/nix/store/af30rki4lvc9cinx3nxq1nqdnfgi6g1b-goxel-0.8.0

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: goxel

Partial log (click to expand)

glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0/bin/goxel
shrinking RPATHs of ELF executables and libraries in /nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0
shrinking /nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0/bin/.goxel-wrapped
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0/bin
patching script interpreter paths in /nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0
checking for references to /build in /nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0...
/nix/store/1l4k91l7aby8gy59nrcf9dmfc1jy890l-goxel-0.8.0

Copy link
Member

@tilpner tilpner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested new features, works on NixOS

@tilpner
Copy link
Member

tilpner commented May 1, 2018

Just noticed goxel still creates imgui.ini in the current working directory. This can be disabled with

diff --git a/src/gui.cpp b/src/gui.cpp
index 9b7236c..a8a11b2 100644
--- a/src/gui.cpp
+++ b/src/gui.cpp
@@ -314,6 +314,8 @@ static void init_ImGui(const inputs_t *inputs)
     ImGuiIO& io = ImGui::GetIO();
     io.DeltaTime = 1.0f/60.0f;

+    io.IniFilename = NULL;
+
     io.KeyMap[ImGuiKey_Tab]         = KEY_TAB;
     io.KeyMap[ImGuiKey_LeftArrow]   = KEY_LEFT;
     io.KeyMap[ImGuiKey_RightArrow]  = KEY_RIGHT;

It can either be fixed right in this PR, or I'll PR this patch after this one is merged.

@xeji
Copy link
Contributor

xeji commented May 1, 2018

@tilpner Thank you. Since this PR was auto-generated, let's merge it first and do your patch separately.

@xeji xeji merged commit 70b1657 into NixOS:master May 1, 2018
@tilpner
Copy link
Member

tilpner commented May 1, 2018

I've added the patch in #39801

@r-ryantm r-ryantm deleted the auto-update/goxel branch May 1, 2018 12:29
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.

None yet

4 participants