[#CIB-1165] Apparent typo in dev tools common.bat

Pulse

Apparent typo in dev tools common.bat

Created: 26/Jul/07 11:33 AM Updated: 02/Aug/07 07:18 AM

Component/s:

None

Affects Version/s:

1.2.33

Fix Version/s:

1.2.34

Description

« Hide

Line 19 of common.bat references a label "noJavaHomeS" that is not present in the script, causing it to abort with an error.

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

jason - 26/Jul/07 02:20 PM

Hi Richard,

Amazing, and embarrassing, that this has not yet turned up. The same common.bat is used in all packages! I guess that it only actually matters when JAVA_HOME is not set, and given that the default way to start Pulse servers on Windows is as a service it is rare to run into the scenario where this happens. Thanks for the catch.

[ Permlink | « Hide ]

jason - 26/Jul/07 02:31 PM

Fixed in change 3563.

[ Permlink | « Hide ]

jason - 02/Aug/07 07:18 AM

Unfortunately the fix breaks auto agent upgrades. Given the fact that this has not proven a problem on server installations, I have reverted to the broken batch file but added a special case to fix it for dev packages. The fix proper will be reapplied in Pulse 2.0. Refer to change 3580.