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

pyo3-pack: 0.3.8 -> 0.5.0 #56032

Merged
merged 1 commit into from
Apr 13, 2019
Merged

pyo3-pack: 0.3.8 -> 0.5.0 #56032

merged 1 commit into from
Apr 13, 2019

Conversation

robertodr
Copy link
Contributor

Motivation for this change

Update pyo3-pack to its latest version.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@veprbl
Copy link
Member

veprbl commented Feb 19, 2019

cc @danieldk

@danieldk
Copy link
Contributor

danieldk commented Feb 19, 2019

Thanks for the update! Currently testing against our project, will report back within the hour or so...

@danieldk
Copy link
Contributor

danieldk commented Feb 19, 2019

Unfortunately, this change does not work on (my) darwin (macOS 10.14.3):

$ pyo3-pack 
dyld: Library not loaded: @rpath/libproc_macro-d4266636a11dd50a.dylib
  Referenced from: /nix/store/r6zm9vv1lygq3faw35mahxrcs7l3nxl6-pyo3-pack-0.5.0/bin/pyo3-pack
  Reason: image not found
Abort trap: 6

It seems that it tries to dynamically link the dependent crates:

$ otool -L `which pyo3-pack`
/nix/store/r6zm9vv1lygq3faw35mahxrcs7l3nxl6-pyo3-pack-0.5.0/bin/pyo3-pack:
	@rpath/libproc_macro-d4266636a11dd50a.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libsyntax-8e769437e3648ad4.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/librustc_target-7ff8eedff7ca3637.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/librustc_errors-5a0caaf966aacf01.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libsyntax_pos-cdcb6ed983a20bc1.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libarena-b7aab6b6a5bea115.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/librustc_data_structures-1084c8bd030620f2.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/librustc_cratesio_shim-d3f5d7e303f42eb8.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libgraphviz-83eb64cca0343abc.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libserialize-0bd1112a715df097.dylib (compatibility version 0.0.0, current version 0.0.0)
	@rpath/libstd-6ee24e3247ed8c3c.dylib (compatibility version 0.0.0, current version 0.0.0)
	/System/Library/Frameworks/Security.framework/Versions/A/Security (compatibility version 1.0.0, current version 58286.240.4)
	/nix/store/0bi9n2vchd9ddf8xnris9rsfaflf18ml-Libsystem-osx-10.11.6/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1226.10.1)
	/nix/store/0bi9n2vchd9ddf8xnris9rsfaflf18ml-Libsystem-osx-10.11.6/lib/libresolv.9.dylib (compatibility version 1.0.0, current version 1.0.0)

The version that is currently in nixpkgs still seems to build and work without problems.

@robertodr
Copy link
Contributor Author

I am afraid I don't know how to debug that failure on Darwin. The diff between v0.3.8 and v0.5.0 doesn't show anything in particular regarding dynamically linking dependent crates.

@konstin
Copy link

konstin commented Feb 19, 2019

That's a bug in older versions of rustc. The latest stable release fixes that. (Full details: mozilla/cbindgen#203)

@veprbl
Copy link
Member

veprbl commented Mar 7, 2019

@GrahamcOfBorg build pyo3-pack

@veprbl
Copy link
Member

veprbl commented Mar 12, 2019

We are currently on rustc 1.32.0, so, I guess, we are waiting for 1.33.0?

@infinisil infinisil added the 8.has: package (update) This PR updates a package to a newer version label Mar 30, 2019
@veprbl
Copy link
Member

veprbl commented Apr 11, 2019

@GrahamcOfBorg build pyo3-pack

@veprbl
Copy link
Member

veprbl commented Apr 12, 2019

Let's give it another spin:
@GrahamcOfBorg build pyo3-pack

Copy link
Member

@veprbl veprbl left a comment

Choose a reason for hiding this comment

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

Builds locally on darwin

@veprbl veprbl merged commit ee742f9 into NixOS:master Apr 13, 2019
@robertodr robertodr deleted the update_pyo3-pack branch April 13, 2019 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants