[Lazarus] Problems creating Lazarus installer + readme/batch patch

Martin lazarus at mfriebe.de
Wed Aug 1 14:27:38 CEST 2012


On 01/08/2012 07:20, Reinier Olislagers wrote:
> BTW, patch attached that:
> - clarify/update readme (mentioned 2.2 fpc compiler instead of 2.6.0)
> - lets you use environment variables for BUILDDIR
>
>

I skipped the build dir part.

BUILDDIR is too generic as a name, and may be set by something else. If 
so, the script would delete whatever directory that points too.

Feel free to introduce

LAZTEMPBUILDDIR

But do not replace it everywere. Initialize BUILDDIR from the former.
And maybe add a check, for empty, or none existent.

The hardcoded BUILDDIR points to a temp dir (and contains laz in the name), so it can always be deleted with little risk.

for a directory from the environment, this is not known. It could point to important data. So it should be required to bo empty already.

yes that prevents auto clean up, if a previous build fails (since on failure the content is left in place, so it can be inspected)






More information about the Lazarus mailing list