matthiasblaesing commented on PR #8408:
URL: https://github.com/apache/netbeans/pull/8408#issuecomment-2790188390

   > looking through the code: The variable substitution runs while the config 
is scanned line-by-line, this has the consequence that properties have to be 
defined in the right order, otherwise things will be missed. Setting 
`-J-Dmyhome=${netbeans_jdkhome}` does only work if `netbeans_jdkhome` is moved 
before `netbeans_default_options`.
   
   The list of supported variables is:
   
   - `DEFAULT_USERDIR_ROOT`
   - `DEFAULT_CACHEDIR_ROOT`
   - `HOME`
   - `BASEDIR`
   
   everything else is undefined behavior (this wording makes me like C :grin: ) 
and unsupported.
   
   > Assuming Windows platform launcher is picking up the change correctly, 
incidentally, this will need the platform Unix launcher altering too - 
https://github.com/apache/netbeans/blob/master/harness/apisupport.harness/release/launchers/app.sh#L36
 That will need the same $basedir calculation adding I guess rather than 
relying on ../
   
   The platform launcher will not pickup this. The variable substitution 
described above is in the IDE launcher.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to