Hi Jan,

Thanks for putting some time on the issues. Very much appreciated.

Change #1:  
        Absolutely fine please go for it.

Change #2:  
        Not sure why -ne doesn’t work for you. I have tested it on Raspbian 
which is based on ubuntu and is working fine. In any case if you would like to 
change the if statement to something more general in terms of the script 
language compatibility I don’t mind. Just make sure that  if the UID should be 
changed to  EUID to check for effective user not the original and it should be 
equal to 0 that means the user is root.

Change #3:
        having the function identifier is just a personal choice and taking it 
away doesn’t have any advantage or disadvantage so I would suggest not to 
change them unless there is a good reason.

Change #4:
        Good call. Please make the change to apache 1.6.2. instead.

Change #5:
        Sure. Please make the change.

Thanks,
-Amirhossein

> On 21 Feb 2017, at 23:15, [email protected] wrote:
> 
> Hi all,
> 
> could I add this script to the deployments/standalone folder (or suggest 
> better)? It's updated deployment script for Ubuntu 16.10. in Azure for 
> test/development.
> There are 5 changes related to the original standalone deployment script:
> 
> Change #1:
> 
> replaced row 159:
> wget -P $R 
> https://download.libsodium.org/libsodium/releases/libsodium-1.0.3.tar.gz
>    cd $R
>    tar xzf libsodium-1.0.3.tar.gz
>    rm $R/libsodium-1.0.3.tar.gz
>    cd $R/libsodium-1.0.3/
>    ./configure
>    make
>    make install
>    cd
>    rm -r $R/libsodium-1.0.3/
> 
> to:
>    wget -P $R 
> https://download.libsodium.org/libsodium/releases/libsodium-1.0.4.tar.gz
>    cd $R
>    tar xzf libsodium-1.0.4.tar.gz
>    rm $R/libsodium-1.0.4.tar.gz
>    cd $R/libsodium-1.0.4/
>    ./configure
>    make
>    make install
>    cd
>    rm -r $R/libsodium-1.0.4/
> 
> reason: version 1.0.3 was already removed from download list (the latest 
> version 1.0.11 was compiled with errors)
> 
> Change #2:
> 
> replaced row 34 (parameter -ne is unknown; in Ubuntu, not sure other distro) 
> - change:
> if [ ${UID} -ne 0 ]; then
>    printRed "Please start the script as root."
>    exit 1
> fi
> 
> to:
> if [ "${UID}" = "0" ]
>    then
>    printRed "Please start the script as root."
>    exit 1
> fi
> 
> Change #3:
> 
> removed "function" identification of each function in the script (Ubuntu)
> 
> Change #4:
> replace row 197:
> wget -P $R 
> http://apache.mirror.anlx.net/spark/spark-1.6.1/spark-1.6.1-bin-hadoop2.6.tgz
> 
> to:
> wget -P $R http://d3kbcqa49mib13.cloudfront.net/spark-1.6.1-bin-hadoop2.6.tgz
> 
> reason: repository apache.mirror.anlx.net is not available
> 
> Change #5:
> replace row 201:
> cat <<EOF >> $R/etc/bash.bashrc
> 
> to:
> cat <<EOF >> /etc/bash.bashrc
> 
> reason:
> to hit the correct file
> 
> Kind regards
> Jan Rock<deploy_standalone_ubuntu1610_azure.sh>

Reply via email to