Postinstall scripts and bash functions

@bpavlov I think you're having an issue with JSS's Script Parameters

Sorry I should have clarified, I'm not testing in the JSS yet so it's not that. My only testing is in OS X 10.9.5, creating the package in Packages and running the built package.

@bpavlov so you're creating a PKG to locally mount a DMG to install a PKG?

Why not upload the PKG you're trying to install to Casper Admin?

Positional parameters, $0, $1, $2, $3 . . ., in bash are local variables. When you call a function, the positional parameters are defined differently in the context of that function.
http://tldp.org/LDP/abs/html/othertypesv.html

You can, outside the context of the function, define a global variable with the value of $3 in the script context. The function can then reference this variable inside the function since it is global. You can also pass the value of $3 to the function as a functional positional parameter when you call it. In this case, the position has changed and you will have to adjust your numbers accordingly. If you use the below code, it will be $1 in the context of the function.

...
if [[ ${osvers} -le 7 ]]; then
    ScanSnap "$3"
fi
...

I, personally, think the code reads better using a global variable. I try to assign variables, global or local, at the "top" of the script/function to make the code more readable and then I don't have to worry about which context I'm in.

Yep, what josh said. When you put $3 inside the function, that is different than the $3 that you are passing to the script.

The $3 in the function means the 3rd argument you are sending to the function.

#!/usr/bash

echo "Script argument #3:  $3"

function blah () {
        echo $3

}

blah "1" "2" "three"

and when we run it:

$ ./blah.sh 1 2 3
Script argument #3:  3
three

#!/bin/bash

#Working directory for script to reference resources
install_dir=`dirname $0`

# Determine OS version
osvers=$(sw_vers -productVersion | awk -F. '{print $2}')

#Variables for ScanSnap installer
ScanSnapDMG="ScanSnap.dmg"
ScanSnapDMGVolume="/Volumes/ScanSnap"
ScanSnapInstaller="ScanSnap Manager.pkg"

ScanSnap () {
    /usr/bin/hdiutil attach -nobrowse -readonly $install_dir/"$ScanSnapDMG"
    /usr/sbin/installer -dumplog -verbose -pkg "$ScanSnapDMGVolume/$ScanSnapInstaller" -target $1 -allowUntrusted
    /bin/sleep 10
    /usr/bin/hdiutil detach "$ScanSnapDMGVolume"
    /usr/bin/killall "ScanSnap Manager"
}

if [[ ${osvers} -le 7 ]]; then
    ScanSnap $3
fi

exit 0

Thanks @Josh_S and @chriscollins. That was very clear. The More You Know!

@bentoms I work of a simple rule. I try to do what I can so that if Casper is not available the package will work as is and has whatever logic is needed in it. From that simple rule I approach how I deploy packages. In this case this application requires 3 installers dependent on the OS version being run. I'm not going to create additional policies and/or smart groups when I can just package everything up and put the logic in a postinstall script. I chose to use the dmgs originally for just testing because I didn't want to take out the PKG out of it since it wasn't necessary and wanted to keep the original installers intact. I will ultimately take them out of the DMGs, but was just testing in this case. In certain situations I will take out the pkg if it needs to be modified right away, but in this case I didn't feel the need to do that since it was still a work in progress.

@bpavlov Fair do's. You have your reasons. :)