search

codefori / vscode-ibmi

🌍 IBM i development extension for VS Code
https://codefori.github.io/docs/#/
MIT License
283 stars 92 forks source link

SEP to support update production #2196

Open worksofliam opened 2 months ago

worksofliam commented 2 months ago

This command we call to the debugger doesn't look like we pass in update debug like we do for batch.

We need an additional Boolean parameter to ibmidebug.create-service-entry-point-with-prompt to control this.

https://github.com/codefori/vscode-ibmi/blob/master/src/api/debug/index.ts#L457

mkwan01 commented 2 months ago

This is an enhancement we can add in the next release. As a complete user story, only adding a parameter to ibmidebug.create-service-entry-point-with-prompt is not enough. We also need to add an action on an SEP object to toggle this setting and also show the setting value in the SEP view.

mkwan01 commented 1 month ago

I will use a simpler implementation, which just honours the "update production files" setting from the code-for-i debug settings page when setting an SEP. We will only go with the more expensive solution of making this setting configurable for each SEP, if there is such a requirement later.

mkwan01 commented 1 month ago

Just figured out that we also need to update the debug service for this one. SEP debug uses the attach launch configuration. We are not passing the updateProductionFiles setting in the attach launch right now. I will discuss with the team on when we can rebuild the debug service PTF.

McJablesSupreme commented 1 week ago

I've been unable to debug any programs that need updprod(*yes). Is there a workaround for this issue while we wait for the update to fix this?

mkwan01 commented 1 week ago

@McJablesSupreme There is no workaround for the updprod issue in SEP debug. We will need an update to the debug service PTF.

McJablesSupreme commented 1 week ago

@mkwan01, that was a really fast reply! Thanks for letting me know. I'll keep an eye out for that.