openmainframeproject / cobol-check

A testing framework for Cobol applications
Apache License 2.0
78 stars 28 forks source link

CobolCheck cannot handle spaces in paths #292

Closed dakaa16 closed 1 year ago

dakaa16 commented 1 year ago

When running cobolcheck, the run will fail, if there is a space at any point in the full path, to the COBOL-file you are referencing or its unit test files. See example: #290. This should be fixed so that spaces are allowed in the paths.

johnathankahn commented 1 year ago

If I add a log statement for example in some file , how do i run the project with that change. I mean ./cobolcheck -p SUM will search for bin/cobol-check-0.2.7.jar which doesn't contain my changes.

johnathankahn commented 1 year ago

Ok! i think i got what i should I am running gradle build but build is failing with the following error image

johnathankahn commented 1 year ago

@dakaa16 Hey, do you have an idea what is causing the error ?

dakaa16 commented 1 year ago

Hi @johnathan2709 This is unrelated to this issue. If you run into new problems, please create a new issue, or use the cobolcheck slack channel. I am not familiar with the ArchiveName method, I'm not sure what we are using it for. It seems to be deprecated, so I will create a new issue, for getting that sorted. Until it's fixed, you can ensure that your java and gradle versions are the same as described in the wiki.

johnathankahn commented 1 year ago

@dakaa16 could you please provide the slack channel link ?

AkashKumar7902 commented 1 year ago

Can anyone please check if this issue also comes in Linux ?

AkashKumar7902 commented 1 year ago

@dakaa16 I have made a pull request with the necessary changes. pull request: #298

dakaa16 commented 1 year ago

Hi @johnathan2709 here is an invitation link to Open Mainframe Project slack channel. Here you can find the Cobol Check channel

AkashKumar7902 commented 1 year ago

@Rune-Christensen I think this issue has been resolved. If not, what's the current status ?

Rune-Christensen commented 1 year ago

@AkashKumar7902 I think you are right, I do not know why it wasn't closed with your PR.