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
hopper: fix libxml2 dynamic linking issue #75522
Conversation
* port to qt5.callPackage / mkDerivation * format derivation
* use wrapQtAppsHook in favour for the custom qt builder * use autoPatchelfHook * cleanup build dependencies * add maintainer Enteee
db0b403
to
2f7f00b
Compare
maintainers = [ stdenv.lib.maintainers.luis ]; | ||
platforms = stdenv.lib.platforms.linux; | ||
license = licenses.unfree; | ||
maintainers = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not use with maintainers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
python27, | ||
gmpxx, | ||
}: | ||
let |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty let.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
|
||
installPhase = '' | ||
mkdir -p $out/bin | ||
mkdir -p $out/lib | ||
mkdir -p $out/share | ||
|
||
cp $sourceRoot/opt/hopper-${rev}/bin/Hopper $out/bin/hopper | ||
cp -r $sourceRoot/opt/hopper-${rev}/lib $out | ||
cp -r $sourceRoot/usr/share $out/share |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is installing files into $out/share/share
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
|
||
installPhase = '' | ||
mkdir -p $out/bin | ||
mkdir -p $out/lib | ||
mkdir -p $out/share | ||
|
||
cp $sourceRoot/opt/hopper-${rev}/bin/Hopper $out/bin/hopper | ||
cp -r $sourceRoot/opt/hopper-${rev}/lib $out |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not like all these libraries being vendored, could you perhaps only copy those that are not available in nixpkgs and add the rest to buildInputs
?
── lib
│ ├── libBlocksRuntime.so.0
│ ├── libdispatch.so -> libdispatch.so.1
│ ├── libdispatch.so.0.1.3.1
│ ├── libdispatch.so.1 -> libdispatch.so.0.1.3.1
│ ├── libform.so.5
│ ├── libgnustep-base.so -> libgnustep-base.so.1.24
│ ├── libgnustep-base.so.1.24 -> libgnustep-base.so.1.24.9
│ ├── libgnustep-base.so.1.24.9
│ ├── libHopperCore.so
│ ├── libkqueue.so.0
│ ├── libncurses.so.5
│ ├── libobjc.so -> libobjc.so.4.6
│ ├── libobjc.so.4.6
│ ├── libobjcxx.so -> libobjcxx.so.4.6
│ ├── libobjcxx.so.4.6
│ ├── libpthread_workqueue.so.0
│ └── libtinfo.so.5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
libpthread_workqueue.so.0
might come from https://github.com/mheily/libpwq
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- libBlocksRuntime.so.0
- copy
- libdispatch.so.0.1.3.1
- copy
- libform.so.5
- copy
- ncurses5.out
- (gcc-arm-embedded-4_9.out)
- (androidStudioPackages.dev.out)
- (androidStudioPackages.canary.out)
- (android-studio.out)
- (androidStudioPackages.beta.out)
- libgnustep-base.so.1.24.9
- copy
- libHopperCore.so
- copy
- libkqueue.so.0
- copy
- libncurses.so.5
- copy
- scilab-bin.out
- ncurses5.out
- (gcc-arm-embedded-4_9.out)
- (androidStudioPackages.dev.out)
- (androidStudioPackages.canary.out)
- (android-studio.out)
- (androidStudioPackages.beta.out)
- libobjc.so.4.6
- copy
- gnustep.libobjc.out
- libobjcxx.so.4.6
- copy
- libpthread_workqueue.so.0
- copy
- libtinfo.so.5
- copy
- tlf.out
- ncurses5.out
- (gcc-arm-embedded-4_9.out)
- jetbrains.clion.out
- arduino-core.out
- arduino.out
- (androidStudioPackages.dev.out)
- (androidStudioPackages.canary.out)
- (android-studio.out)
- (androidStudioPackages.beta.out)
generated with:
for l in *; do [ -L "${l}" ] && continue; echo "* ${l}"; echo " - [ ] copy"; nix-locate -1 -w "${l}" | sed -e 's/^/ - [ ] /' ; done | xclip
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
* add dependency to ncurses, gnustep * don't copy to share/share * run pre/post Install hooks * fix desktop icon * with maintainers
fb8e87e
to
bb51742
Compare
@jtojnar all review comments were implemented, please merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works fine :-)
@Lassulus could you merge this please?
hopper: fix libxml2 dynamic linking issue (cherry picked from commit 2d87562)
Motivation for this change
On channel:
nixpkgs-20.03pre202154.58fb23f72ad
hopper currently fails to start:Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @luis-hebendanz (Maintainer)
@ttuegel (Contributor)