-
Notifications
You must be signed in to change notification settings - Fork 108
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
network/nordvpn: Fixed compatibility issues #8753
base: master
Are you sure you want to change the base?
Conversation
reverse dependencies - network/nordvpnNone |
✅ sbolint - network/nordvpn ✅
|
network/nordvpn/doinst.sh
Outdated
# Patched by Peter Christy <[email protected]> | ||
|
||
# update configuration and shared library cache for linker to find .so files | ||
echo "/usr/lib64/nordvpn" > /etc/ld.so.conf.d/nordvpn.conf |
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.
The info file has a download for both 32 and 64bit. But this is hardcoded to lib64?
network/nordvpn/nordvpn.SlackBuild
Outdated
@@ -88,6 +90,12 @@ mkdir -p $PRGNAM-$VERSION | |||
cd $PKG | |||
ar p $CWD/${PRGNAM}_${VERSION}_${DEBARCH}.deb data.tar.gz | tar zxv | |||
|
|||
# nordvpn-3.19.x appears to need additional libraries: | |||
mkdir -p usr/lib64/nordvpn |
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.
Here also lib64
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.
please have a look on the comments
There might be binary helper programs in there for the app to fully function. Or I did it wrong. But I read about them, they should be used by the main program? edit: yes I was wrong the app should know the path to those and they should not be exposed to the users, btw there was an update now its 3.19.1 lol. |
@willysr I should fix the issues and test it properly, but I cannot right now. Iwill make new changes this weekend. Please keep this PR open if possible. |
d2130db
to
e2751ba
Compare
Signed-off-by: Reza Talebi <[email protected]>
Signed-off-by: Reza Talebi <[email protected]>
e2751ba
to
bc52885
Compare
Signed-off-by: Reza Talebi <[email protected]>
Signed-off-by: Reza Talebi <[email protected]>
@sbo-bot: build network/nordvpn |
buildpkg i586 network/nordvpn
Snipped Build Failure
|
buildpkg x86_64 network/nordvpn
Snipped Build Log
|
sbopkglint x86_64 network/nordvpn
|
@RezaT4795 please review the code for i586 |
No description provided.