eclipse-che4z / che-che4z-lsp-for-cobol

COBOL Language Support provides autocomplete, highlighting and diagnostics for COBOL code and copybooks
Other
80 stars 55 forks source link

Need for configurable .c4z folder #1114

Closed GitMensch closed 1 year ago

GitMensch commented 3 years ago

Is your feature request related to a problem? Totally - I'm on a read-only workspace - and get

File error: Error: EPERM: operation not permitted, mkdir 'x:\workspace1\folder1.c4z'

Describe the solution you'd like A new setting to define the folder (in my case it would be set to C:\Userspace\c4z\workspace1), ideally with resolving other configuration variables (in my case C:\Userspace\c4z\${config:my.wsName})

Related: #336 which got the fixed ".c4z folder" added.

GitMensch commented 2 years ago

As requested: pinging @zimlu02 about this to be one of the issues I see.

I think there was an API to write to "extension specific storage" that may could be used instead (if I remember correctly that would write somewhere in the local user application data); this may could be used, too - otherwise I'm totally fine with a configurable folder.

GitMensch commented 2 years ago

another friendly ping @zimlu02

zimlu02 commented 2 years ago

another friendly ping @zimlu02

Hi Simon, I am really sorry for this long silence. We've finally got to this issue and I will follow up on this ticket this week with more info.

zimlu02 commented 2 years ago

A current code refactorization of the copybooks infrastructure is a blocker for this feature at the moment. However, this effort should be done for the next upcoming release 1.1.0. Once released and unblocked, I will set a priority for this request. Once again, an apologies for not updating this ticket for long.

VitGottwald commented 1 year ago

Hello @GitMensch, are you using the copybook download feature to retrieve copybooks from z/OS?

GitMensch commented 1 year ago

No, I'm only doing PC-based COBOL (with GnuCOBOL, including big iron dialects [Enterprise COBOL, MVS, BS200, GCOS]).

VitGottwald commented 1 year ago

Thank you @GitMensch . I created a bug https://github.com/eclipse/che-che4z-lsp-for-cobol/issues/1716 to fix the current behaviour.