arosien commented on code in PR #694:
URL: https://github.com/apache/daffodil-vscode/pull/694#discussion_r1260043078
##########
src/daffodilDebugger/utils.ts:
##########
@@ -33,32 +32,45 @@ export const daffodilArtifact = (version: string): Artifact
=> {
return new Artifact('daffodil-debugger', version, 'daffodil-debugger')
}
-export async function buildDebugger(baseFolder: string, filePath: string) {
- if (!filePath.includes('.vscode/extension')) {
- if (!fs.existsSync(filePath)) {
- let command = osCheck(
- 'sbt universal:packageBin',
- '/bin/bash --login -c "sbt universal:packageBin"'
- ) // Needed --login so it could resolve sbt command
+export async function buildDebugger(
Review Comment:
Hmm/grrr. I will think more, but I still think this should be in
build-space, not extension-space.
--
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]
