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

subsurface: 4.7.2 -> 4.7.5 #33251

Merged
merged 1 commit into from Jan 1, 2018
Merged

subsurface: 4.7.2 -> 4.7.5 #33251

merged 1 commit into from Jan 1, 2018

Conversation

bkchr
Copy link
Contributor

@bkchr bkchr commented Dec 31, 2017

Also updates googlemaps to 2017-12-15 and fixes compilation with qt5.10.

Motivation for this change

Preparation for qt5.10

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.

Also updates googlemaps to 2017-12-15 and fixes compilation with qt5.10.
@lukateras
Copy link
Member

@GrahamcOfBorg build subsurface

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

while evaluating the attribute ‘__impureHostDeps’ of the derivation ‘qtconnectivity-5.9.3’ at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/stdenv/generic/make-derivation.nix:98:11:
while evaluating ‘unique’ at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/lists.nix:455:12, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/stdenv/generic/make-derivation.nix:89:13:
while evaluating ‘concatMap’ at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/lists.nix:102:18, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/stdenv/generic/make-derivation.nix:89:25:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/stdenv/generic/make-derivation.nix:89:40, called from undefined position:
while evaluating ‘getOutput’ at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/attrsets.nix:450:23, called from undefined position:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/stdenv/generic/make-derivation.nix:63:15, called from undefined position:
while evaluating ‘callPackageWith’ at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/customisation.nix:113:35, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/top-level/all-packages.nix:12401:12:
while evaluating ‘makeOverridable’ at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/customisation.nix:72:24, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/customisation.nix:117:8:
while evaluating anonymous function at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/os-specific/linux/bluez/default.nix:1:1, called from /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/customisation.nix:74:12:
assertion failed at /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/pkgs/os-specific/linux/bluez/default.nix:5:1

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

glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/hjy6p3l1ip65kx3dbxb5qsw66f87v8qi-subsurface-4.7.5
shrinking /nix/store/hjy6p3l1ip65kx3dbxb5qsw66f87v8qi-subsurface-4.7.5/bin/.subsurface-wrapped
stripping (with flags -S) in /nix/store/hjy6p3l1ip65kx3dbxb5qsw66f87v8qi-subsurface-4.7.5/bin 
patching script interpreter paths in /nix/store/hjy6p3l1ip65kx3dbxb5qsw66f87v8qi-subsurface-4.7.5
checking for references to /tmp/nix-build-subsurface-4.7.5.drv-0 in /nix/store/hjy6p3l1ip65kx3dbxb5qsw66f87v8qi-subsurface-4.7.5...
postPatchMkspecs
postMoveQtStaticLibs
/nix/store/hjy6p3l1ip65kx3dbxb5qsw66f87v8qi-subsurface-4.7.5

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: aarch64-linux

glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/yss7gmxlnqbgkp1wkwk33dm1r4xivasj-subsurface-4.7.5
shrinking /nix/store/yss7gmxlnqbgkp1wkwk33dm1r4xivasj-subsurface-4.7.5/bin/.subsurface-wrapped
stripping (with flags -S) in /nix/store/yss7gmxlnqbgkp1wkwk33dm1r4xivasj-subsurface-4.7.5/bin
patching script interpreter paths in /nix/store/yss7gmxlnqbgkp1wkwk33dm1r4xivasj-subsurface-4.7.5
checking for references to /build in /nix/store/yss7gmxlnqbgkp1wkwk33dm1r4xivasj-subsurface-4.7.5...
postPatchMkspecs
postMoveQtStaticLibs
/nix/store/yss7gmxlnqbgkp1wkwk33dm1r4xivasj-subsurface-4.7.5

@adisbladis
Copy link
Member

It seems like you based this on an old master. subsurface is already at 4.7.5.

@lukateras
Copy link
Member

lukateras commented Jan 1, 2018

Still, this includes a compilation fix for Qt5.10 (presumably adding libXcomposite).

@adisbladis
Copy link
Member

@yegortimoshenko It's a good fix. It just has to be rebased on current master with a new commit message.

@lukateras
Copy link
Member

GitHub seems to be able to merge it without rebase. New commit message can be supplied when using "Squash and merge".

@lukateras lukateras merged commit d9e26a0 into NixOS:master Jan 1, 2018
@bkchr
Copy link
Contributor Author

bkchr commented Jan 1, 2018

Thanks for merging it anyway :)

@bkchr
Copy link
Contributor Author

bkchr commented Jan 1, 2018

And yeah, the googlemaps upgrade is needed for qt 5.10.

@flokli
Copy link
Contributor

flokli commented Jan 22, 2018

@bkchr Does the map widget work for you?

I get

qrc:/MapWidget.qml:21: Error: Cannot assign [undefined] to QDeclarativeGeoMapType*

during runtime, and only a white map instead of tiles…

Not that into QT, but it seems like the type used in MapWidget.qml:21 doesn't exist

@bkchr
Copy link
Contributor Author

bkchr commented Jan 22, 2018

@flokli are you on Linux or MacOs? The following error seems to be similar: subsurface/subsurface#588

Maybe we also need to fix that on our side.

@flokli
Copy link
Contributor

flokli commented Jan 22, 2018

@bkchr I'm on NixOS, not MacOS.

MapWidget.qml: cannot find a plugin named: googlemaps

is also shown in the stdout - sorry, missed this one.

So it looks pretty much the same as the linked issue.

Build logs show:

-- Cannot find libqtgeoservices_googlemaps.so

So yes, this might need patching from our side.

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

5 participants