The things that go in square brackets like that are Windows Installer property names, either standard ones or those that you define. You can't just put the name of a system environment variable in there.
If that really is an actual system environment variable name, you use the syntax:
[%ENVIRONMENTVARIABLENAME]
to tell Windows that this is not a MSI property name, it'san environment variable name.
Phil Wilson