shanedell opened a new issue, #934:
URL: https://github.com/apache/daffodil-vscode/issues/934
Something that I noticed with testing changes for #653 is that the
`daffodilDebugClasspath` setting can become incredibly long. I think making
this setting a JSON object that will that be looped over and join the values
together separated by a colon (`:`).
So for example:
#### Snippet 1
```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",
...
}
```
Could instead be:
#### Snippet 2
```json
{
...
"daffodilDebugClasspath": {
"dfdl-mil-std-2045":
"${workspaceFolder}/dfdl-mil-std-2045/src/main/resources:${workspaceFolder}/dfdl-mil-std-2045/target/classes",
"dfdl-tcpMessage":
"${workspaceFolder}/dfdl-tcpMessage/src/main/resources:${workspaceFolder}/dfdl-tcpMessage/target/classes",
"dfdl-pcap":
"${workspaceFolder}/dfdl-pcap/src/main/resources:${workspaceFolder}/dfdl-pcap/target/classes",
"dfdl-envelope-payload":
"${workspaceFolder}/dfdl-envelope-payload/src/main/resources:${workspaceFolder}/dfdl-envelope-payload/target/classes",
"dfdl-ethernet":
"${workspaceFolder}/dfdl-ethernet/src/main/resources:${workspaceFolder}/dfdl-ethernet/target/classes",
}
...
}
```
Another thing that we could also do is test if the value passed is a
directory, if it is the extension won't add the general directory but would
instead add `${DIR_PATH}/{src/main/resources,target/classes}`. Then if say its
a file like a JAR it just leaves as is and will join in together with the other
items. This would than make the above Snippet 2 to be even simpler by only
needing this instead:
#### Snippet 3
```json
{
...
"daffodilDebugClasspath": {
"dfdl-mil-std-2045": "${workspaceFolder}/dfdl-mil-std-2045",
"dfdl-tcpMessage": "${workspaceFolder}/dfdl-tcpMessage",
"dfdl-pcap": "${workspaceFolder}/dfdl-pcap",
"dfdl-envelope-payload": "${workspaceFolder}/dfdl-envelope-payload",
"dfdl-ethernet": "${workspaceFolder}/dfdl-ethernet",
}
...
}
```
But in the end with exported variable would still contain all paths needed
like Snippet 1 manually defines.
What does everyone think about this?
@mbeckerle @stevedlawrence @scholarsmate @stricklandrbls @NolanMatt @arosien
@lrbarber @nlewis05 @hdalsania
--
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]
