Hi,

I just looked over your scripts and have some remarks/questions.


Linux-script:

>>echo "Please, enter your account user:"
>>read NAME
There is a variable called $USER. Why not use this?

>>sudo apt-get install -y cvs
What is this package for? I don't have this installed on my ubuntu and you don't use it in your script.

>>sudo apt-get install -y alien
>>...
>>cd /home/$NAME/fpc_tools/fpc_setup
>>wget -c http://mirror.mirimar.net/freepascal/dist/2.4.0/i386-linux/rpm/fpc-2.4.0-1.i386.rpm
>>..
>>sudo alien -i -c /home/$NAME/fpc_tools/fpc_setup/fpc-2.4.0-1.i386.rpm
I don't see a reason for installing an application, downloading a package and converting them, just to get one binary. Binaries for bootstrap can be downloaded from ftp://ftp.freepascal.org/pub/fpc/dist/2.4.0/bootstrap/. Or is there another reason?

>>svn co http://svn.freepascal.org/svn/fpc/trunk fpc
>>cd /home/$NAME/fpc_tools/fpc/
>>svn up
>>..
>>cd /home/$NAME/
>>svn co http://svn.freepascal.org/svn/lazarus/trunk lazarus
>>cd /home/$NAME/lazarus/
>>svn up
Why doing an update after a checkout?


Windows-script:

>>wget -c %FPCURL%/%FPCNAME% --directory-prefix="%PATHDEST%\tools\fpc_setup"
>>...
>>echo "Copy binutils"
>>echo.
>>C:
>>copy %PATHDEST%\fpc\2.4.0_org\bin\i386-win32\ar.exe %PATHDEST%\fpc\binutils\ >>copy %PATHDEST%\fpc\2.4.0_org\bin\i386-win32\as.exe %PATHDEST%\fpc\binutils\
>>...
Binaries for bootstrap can be downloaded from ftp://ftp.freepascal.org/pub/fpc/dist/2.4.0/bootstrap/ and binutils can be downloaded from http://svn.freepascal.org/svn/fpcbuild/binaries/i386-win32/.

>>set PATHDEST=C:\Develop
>>...
>>C:
It would be more flexible to use 2 variables e.g. "PATHDEST=\Develop" and "DRIVEDEST=C:".

>>set OLDPATH=%PATH%
>>...
>>set PATH=%OLDPATH%;%PATHDEST%\fpc\2.4.0_org\bin\i386-win32\
The PATH can be set with "set PATH=%PATH%;%PATHDEST%\fpc\2.4.0_org\bin\i386-win32\". There is no need for an additional variable.


This were just some things I don't understand why you did it this way. Also I'm very low on bandwidth, so I'm looking what's needed to download.

Hope it helps you.

regards
Ingo


--
_______________________________________________
Lazarus mailing list
[email protected]
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to