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)



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

Reply via email to