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

simp_le: 0.6.1 -> 0.6.2 #31944

Closed
wants to merge 1 commit into from
Closed

Conversation

makefu
Copy link
Contributor

@makefu makefu commented Nov 22, 2017

Motivation for this change

according to the changelog, this release of simp_le will be
future-proof by not hard-coding the TOS checksum.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

according to the changelog, this release of simp_le will be
future-proof by not hard-coding the TOS checksum
@FRidh
Copy link
Member

FRidh commented Nov 22, 2017

cc @fpletz

@Mic92
Copy link
Member

Mic92 commented Nov 22, 2017

should we back port this?

@Mic92 Mic92 added the 9.needs: port to stable A PR needs a backport to the stable release. label Nov 22, 2017
@disassembler
Copy link
Member

@GrahamcOfBorg build simp_le

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

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

Success for system: x86_64-linux

patching script interpreter paths in /nix/store/v0ljjrmgjnkvam6zkn45673029ia89f7-simp_le-client-0.6.2
checking for references to /tmp/nix-build-simp_le-client-0.6.2.drv-0 in /nix/store/v0ljjrmgjnkvam6zkn45673029ia89f7-simp_le-client-0.6.2...
wrapping `/nix/store/v0ljjrmgjnkvam6zkn45673029ia89f7-simp_le-client-0.6.2/bin/simp_le'...
running install tests
...............................
----------------------------------------------------------------------
Ran 31 tests in 0.484s

OK
/nix/store/v0ljjrmgjnkvam6zkn45673029ia89f7-simp_le-client-0.6.2

@disassembler
Copy link
Member

I agree it should go to stable. I'm happy to merge and backport, but would like @fpletz confirmation.

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

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

Failure for system: x86_64-darwin

    raise Error('Failed to execute external script: %s' % error)
Error: Failed to execute external script: [Errno 8] Exec format error

----------------------------------------------------------------------
Ran 31 tests in 1.871s

FAILED (errors=1)
builder for '/nix/store/fw04dx00vxyqy5pd0iqdppq61d49p469-simp_le-client-0.6.2.drv' failed with exit code 2
error: build of '/nix/store/fw04dx00vxyqy5pd0iqdppq61d49p469-simp_le-client-0.6.2.drv' failed on ‘mac1’
error: build of ‘/nix/store/fw04dx00vxyqy5pd0iqdppq61d49p469-simp_le-client-0.6.2.drv’ failed

@dotlambda
Copy link
Member

I believe this should have been merged a long time ago?

@@ -2,12 +2,12 @@

pythonPackages.buildPythonApplication rec {
pname = "simp_le-client";
version = "0.6.1";
version = "0.6.2";
name = "${pname}-${version}";
Copy link
Member

Choose a reason for hiding this comment

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

No need for a name here.

@dotlambda
Copy link
Member

However, there is now version 0.8.

@makefu
Copy link
Contributor Author

makefu commented Mar 20, 2018

i am pretty sure that name was still required at that point in time. i will bump to 0.8.0

@makefu
Copy link
Contributor Author

makefu commented Mar 20, 2018

too many changes required, i will open a new PR

@makefu makefu closed this Mar 20, 2018
@makefu makefu deleted the pkgs/simp_le/0.6.2 branch March 20, 2018 21:42
@makefu makefu mentioned this pull request Mar 20, 2018
8 tasks
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

7 participants