[BUG] Containerless-Kantra : [Windows-Only - Java Project analysis] Fails to create static-report

[kthatipally]

Is there an existing issue for this?

• I have searched the existing issues

Konveyor version

https://github.com/konveyor/kantra/releases/tag/v0.6.0-alpha.2

Priority

Undefined (Default)

Current Behavior

I have run the analysis using the command:

.\windows-kantra.exe analyze-bin --input C:\testing\spring-petclinic\ --output .\output-spc --target cloud-readiness --overwrite

The files Dependencies.yaml and output.yaml have been successfully created. However, there's a problem with generating the static-report due to an issue with the cp command for the static-report. Let me know if you need any more info.

See the error below:

Expected Behavior

Analysis is run successfully, and the static-report is generated.

How Reproducible

Always (Default)

Steps To Reproduce

1. OS: Windows
2. Run Command: .\windows-kantra.exe analyze-bin --input C:\testing\spring-petclinic\ --output .\output-spc --target cloud-readiness --overwrite
3. Run via windows-powershell

Environment

• OS: Windows

Anything else?

No response

[konveyor-ci-bot]

This issue is currently awaiting triage.
If contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.
The triage/accepted label can be added by org members.

[eemcmullan]

@kthatipally Thank you for finding this issue. This is interesting because the cp command for the static report is only (supposed) to be used by the container analyze command, as we don't use an executable for the containerless version. Do you have the analysis.log and kantra logs?

[kthatipally]

Hi @eemcmullan ,
Please find the attached analysis.log.
static-report.log is empty. Let me know if you need more logs. analysis.log

This is the line that is causing the error:

kantra/cmd/analyze-bin.go

Line 565 in d91946d

cmd := exec.Command("cp", "-r", outputFileDestPath, a.output)

[eemcmullan]

@kthatipally Sorry, forgot we did that there. Is it fair to say xcopy would be the equivalent command?

[kthatipally]

@eemcmullan , Yes, the xcopy command works on Windows. Would it be worth considering a platform-agnostic approach? We might be able to use or adapt a function like (

kantra/cmd/analyze.go

Line 1077 in d91946d

func copyFileContents(src string, dst string) (err error) {

) to ensure cross-platform compatibility.

[eemcmullan]

Fixed in #366