stevedlawrence commented on issue #934:
URL:
https://github.com/apache/daffodil-vscode/issues/934#issuecomment-1888008029
I think it's important to be careful about guessing what users want, so I'm
not sure I'm a fan of automatically adding `src/main/resources` and
`target/classes`, but I don't feel strongly about it. But maybe an alternative
would be to have a daffodilDebugClasspath wizard that you can give it project
directories, it will search for what it things are resources (e.g.
src/main/resource, target/classes, *.jar) and the user can confirm what they
want added to the classpath.
Also, another thought on a possible format of daffodilDebugClasspath, what
about just an array of strings, e.g.
```json
{
...
"daffodilDebugClasspath": [
"${workspaceFolder}/dfdl-mil-std-2045/src/main/resources",
"${workspaceFolder}/dfdl-mil-std-2045/target/classes",
"${workspaceFolder}/dfdl-tcpMessage/src/main/resources",
"${workspaceFolder}/dfdl-tcpMessage/target/classes",
"${workspaceFolder}/dfdl-pcap/src/main/resources",
"${workspaceFolder}/dfdl-pcap/target/classes",
"${workspaceFolder}/dfdl-envelope-payload/src/main/resources",
"${workspaceFolder}/dfdl-envelope-payload/target/classes",
"${workspaceFolder}/dfdl-ethernet/src/main/resources",
"${workspaceFolder}/dfdl-ethernet/target/classes"
],
...
}
```
Also, something to consider is that on Windows the classpath separator is a
semicolon instead of colon. Doing it as an array makes it so you can just join
the array with the os specific classpath separator and you don't need something
like `str.replace(":", ";")`.
--
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]
