shanedell commented on issue #895:
URL: 
https://github.com/apache/daffodil-vscode/issues/895#issuecomment-1804792305

   @lrbarber So instead of doing:
   
   ```ts
   const extraArgs = isAtLeastJdk17
       ? ['-J--add-opens', '-Jjava.base/java.lang=ALL-UNNAMED']
       : [ ]
   ```
   
   if you did
   
   ```ts
   const extraArgs = isAtLeastJdk17
       ? ['-J--add-opens java.base/java.lang=ALL-UNNAMED']
       : [ ]
   ```
   
   does that work? If not I would ask why does windows option need surrounded 
by double quotes for
   ```ts
   const extraArgs = isAtLeastJdk17
      ? [ osCheck('-J"--add-opens java.base/java.lang=ALL-UNNAMED"',     // 
Windows
                  '-J--add-opens java.base/java.lang=ALL-UNNAMED')]      // 
Linux
      : [ ]
   ```
   ? If that is the only way it can be done that is fine just curious as the 
double quotes seem like they shouldn't be needed. Meaning instead of needing 
two different commands for the different OSes that we instead need to send 
these values as one big string instead of two separate ones like my snippet 
from before.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to