Ticket #935 (new defect)

Opened 8 months ago

Make perllib/IncrementalBuildUtils.pm reuse existing code

Reported by: ak19 Owned by: ak19
Priority: very high Milestone: 3.09 Release
Component: Collection Building Severity: trivial
Keywords: Cc:

Description

Related to commits 31972 and 31973. Commit http://trac.greenstone.org/changeset/31972 contained a bug fix to perllib/IncrementalBuildUtils.pm by Dr Bainbridge.

IncrementalBuildUtils? determines the PATH separator based on whether $GSDLOS contained the "win" substring denoting Windows. However, for the Mac OS, $GSDLOS has the value "darwin", which contains the "win" substring too. So Dr Bainbridge fixed the test to check for win as OS name prefix.

This part of the code will need to be reworked in future, such as to use util::envvar_prepend() and envvar_append() functions instead of concatenating something that may already be on the PATH. Also, check for any existing methods to test if OS is windows or not to put it all in one place.

Make the changes that reuse code BEFORE starting on GS3 release testing.

Note: See TracTickets for help on using tickets.