search

codefori / vscode-ibmi

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

ILEDITOR temporary file clean up errors with only sucessful deletions #2233

Open m-tyler opened 2 months ago

m-tyler commented 2 months ago

During the connection phase, the clean up of library ILEDITOR of temporary files O_*, I often receive the error Temporary data not cleared from ILEDITOR.

Monitoring only for message id CPF2125 is not enough to prevent false errors being reported.

Context Version
Code for IBM i version 2.12.1
Visual Studio Code version 1.92.2
Operating System win32_x64
Active extensions ``` Code Spell Checker (code-spell-checker): 3.0.1 Code for IBM i User Spooled Files (vscode-ibmi-UserSpooledFiles): 0.0.9 Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.5.0 Configuration Editing (configuration-editing): 1.0.0 Db2 for IBM i (vscode-db2i): 1.3.1 Emmet (emmet): 1.0.0 Error Lens (errorlens): 3.20.0 Excel Viewer (gc-excelviewer): 4.2.61 Git (git): 1.0.0 Git Base (git-base): 1.0.0 GitHub (github): 0.0.1 GitHub Authentication (github-authentication): 0.0.2 Hawkeye Pathfinder (vscode-ibmi-hawkeye): 0.0.12 JSON Language Features (json-language-features): 1.0.0 Merge Conflict (merge-conflict): 1.0.0 Node Debug Auto-attach (debug-auto-launch): 1.0.0 Overtype (overtype): 0.5.0 Partial Diff (partial-diff): 1.4.3 Rainbow CSV (rainbow-csv): 3.12.0 Remove empty lines (remove-empty-lines): 1.0.1 TODO Highlight (vscode-todo-highlight): 1.0.5 Text Marker (Highlighter) (text-marker): 1.11.0 TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0 Version Lens (vscode-versionlens): 1.14.2 classdiagram-ts (classdiagram-ts): 0.1.67 undefined (indent-rainbow): 8.3.1 ```
Remote system |Setting|Value| |-|-| |IBM i OS|V7R5M0| |Tech Refresh|4| |CCSID Origin|37| |Runtime CCSID|37| |Default CCSID|37| |SQL|Enabled |Source dates|Enabled ### Enabled features |/QOpenSys/pkgs/bin|/usr/bin|/QSYS.LIB|/QIBM/ProdData/IBMiDebugService/bin| |-|-|-|-| |bash|attr|QZDFMDB2.PGM|startDebugService.sh| |find|iconv||| |git|setccsid||| |grep|||| |ls|||| |md5sum|||| |sort|||| |stat|||| |tar|||| |tn5250||||
Shell env ```bash ... PATH=/QOpenSys/pkgs/bin:/QOpenSys/usr/bin:/usr/ccs/bin:/QOpenSys/usr/bin/X11:/usr/sbin:.:/usr/bin PCR=PCR1708800,PCR1976200,PCR1935500,PCR1919800,PCR1959800 SHELL=/QOpenSys/pkgs/bin/bash SHLVL=1 SSH_CLIENT=10.10.0.134 55630 22 SSH_CONNECTION=10.10.0.134 55630 10.0.4.31 22 TZ=7,M3.2.0,M11.1.0 _=/QOpenSys/pkgs/bin/env ... ```
Variants ```json { "american": "#@$", "local": "#@$" } ```
Errors ```json [ system "DLTOBJ OBJ(ILEDITOR/O_*) OBJTYPE(*FILE)" { "code": 0, "signal": null, "stdout": "", "stderr": "CPC2191: Object O_AJQTYRMO in ILEDITOR type *FILE deleted.\nCPC2191: Object O_FVH3UMSN in ILEDITOR type *FILE deleted.\nCPC2191: Object O_JDJBJACG in ILEDITOR type *FILE deleted.\nCPC2191: Object O_VMGYGKEH in ILEDITOR type *FILE deleted.\nCPC2116: All 4 objects of type *FILE deleted." } ] ```
sebjulliand commented 2 months ago

I guess checking the return code should be enough...the dialog is showing because standard error is not blank.

m-tyler commented 2 months ago

I wonder if I really need to know the library temporary objects were not all cleaned up.

I think there is a case where a file is locked for use while others are free to delete and the status returns 0. I have seen it one time but I cannot get a test case working right now.

But as side item, it seems possible that with multiple users using one working library, I could start up VS Code and clean up someone else's working data.