thx, for the PR I have a look when time permits. The CI test however are failing with an assertion issue in java.yaml

======================================================================
FAIL: testArtifactDefinitionsValidator (validator_test.ArtifactDefinitionsValidatorTest)
Runs the validator over all the YAML artifact definitions files.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/artifacts/tests/validator_test.py", line 27, in testArtifactDefinitionsValidator
    result, msg='in definitions file: {0}'.format(definitions_file))
AssertionError: in definitions file: data/java.yaml
----------------------------------------------------------------------

Interesting, the test fails because it does not recognize %%users.localappdata_low%%, but that is actually defined in the provides-section of WindowsUserShellFolders.

Yeah, that might be strict check to catch GRR legacy stuff (also see #275). I'll have a look when time permits.

The reason for the error is that %localappdata_low% does not exist as environment variable

@DeKe42 thx for the changes. Overall changes LGTM. I've reverted the changes to Java.yaml and created a new PR #372 to trigger the tests on current HEAD. Closing this PR in favor of the new one.