-
Notifications
You must be signed in to change notification settings - Fork 534
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
OS X Installer Fixes #1629
OS X Installer Fixes #1629
Conversation
* Fix how the user is obtained (http://apple.stackexchange.com/questions/144159/how-can-i-determine-the-invoking-user-in-an-apple-installer-postinstall-script) * Do brew install/tap commands only if the packages are not found * This was we avoid errors from double installation
…links This issue probably originated around 3.3 where libaf*.dylib / libaf*.major.dylib were not being treated as symlinks but were full versions of the library on their own. This was due to CMake copying symlinks as the files they point to rather than symlinks.
@@ -14,13 +14,16 @@ if [ ! -f $brew ]; then | |||
exit 1 | |||
fi | |||
|
|||
user=$(ps aux | grep console | grep -v 'grep\|root' | cut -d' ' -f1 | head -n1) | |||
#user=$(ps aux | grep console | grep -v 'grep\|root' | cut -d' ' -f1 | head -n1) | |||
user=$(stat -f '%Su' $HOME) |
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.
👍
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.
Can we make sure that this is working as expected and the libraries are being installed in the correct location.
@@ -261,10 +296,10 @@ IF(BUILD_GRAPHICS) | |||
PKG_BUILD( PKG_NAME ForgeLibrary | |||
DEPENDS OSX_INSTALL_SETUP_FORGE_LIB | |||
TARGETS forge_lib_package | |||
INSTALL_LOCATION /usr/local/ | |||
INSTALL_LOCATION /usr/local/lib |
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 doesn't look right. Was it really installing the libraries in the /usr/local directory? I don't see a change in the PKG_BUILD function that is associated with this change.
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.
@shehzan10 clarified that this is indeed correct.
share/Forge/*
being installed both in/usr/local/*
and /usr/local/share/Forge/*` (the former was wrong).postinstall
script not working due to not having +x enabled.postinstall
script.[skip arrayfire ci]
These fixes have been retroactively been applied to the ArrayFire-v3.4.1_OSX.pkg installer available on the arrayfire.com/download site.