Skip to content

Extension not activated on local installation #1637

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
dayo09 opened this issue Sep 13, 2023 · 6 comments · Fixed by #1638
Closed

Extension not activated on local installation #1637

dayo09 opened this issue Sep 13, 2023 · 6 comments · Fixed by #1638
Labels
Bug/Issue Something isn't working

Comments

@dayo09
Copy link
Contributor

dayo09 commented Sep 13, 2023

What?

The current main branch runs on debugging session very well, but it doesn't work on installation.

@dayo09 dayo09 added the Bug/Issue Something isn't working label Sep 13, 2023
@dayo09
Copy link
Contributor Author

dayo09 commented Sep 13, 2023

Searching the problematic PR

git reset --hard 0.4.0 && \
rm one-vscode-0.4.0.vsix && \
vsce package && \
code --uninstall-extension Samsung.one-vscode && \
code --install-extension one-vscode-0.4.0.vsix

git reset --hard a001610 && \
rm one-vscode-0.4.0.vsix && \
vsce package && \
code --uninstall-extension Samsung.one-vscode && \
code --install-extension one-vscode-0.4.0.vsix
version status
ed89c8e Add result variable to JobCommand (#1573) fail
90ea0b1 [MPQEditor] Implement createFromDefaultExplorer (#1527) fail
b96f6a8 [MPQEditor] Register MPQEditor (#1528) fail
8d8fed5 [MPQEditor] Implement message processing (#1526) ok
cc00550 [MPQEditor] Implement webviewPanel init. (#1525)
edef879 [MPQEditor] Implement providing model nodes info. (#1524) ok
1ce4be5 [MPQEditor] Implement 'createMPQJson' (#1521)
a0132fa [MPQEditor] Implement 'updateDocumentBy' (#1520)
0ed653a [MPQEditor] Implement 'createDefaultMPQ' (#1519)
08972a3 [MPQEditor] Implement 'findMPQName' (#1518) ok
a001610 [docs] Add docs for visual tensor view (#1504) ok
0.4.0 ok

@dayo09
Copy link
Contributor Author

dayo09 commented Sep 13, 2023

b96f6a8 [MPQEditor] Register MPQEditor (#1528) looks problematic.

Dev host

  console.log("MPQEditorProvider.register");
  MPQEditorProvider.register(context);
  console.log("MPQEditorProvider.register done");
[2023. 9. 13. 오전 11:29:49][extension.ts][info] MPQEditorProvider.register
[2023. 9. 13. 오전 11:29:49][extension.ts][info] MPQEditorProvider.register done
  • But it works well on development host.

Local host

Not showing anything in OUTPUT

It doesn't prints out activation log.
It seems the whole activation function didn't run.

Code version?

I upgraded my vscode from 1.81.1 -> 1.82.1 but it remains the same.

@dayo09
Copy link
Contributor Author

dayo09 commented Sep 13, 2023

MPQEditor module

/cc @stamalakhov

With MPQEditor related modules commented out in extensions.ts, the extension works as normally at the latest main branch.

NOTE that the symptom looks as if it 'activate' function doesn't work at the beginning, because the first line Logger.info(tag, "one-vscode activate OK"); doesn't prints out. ONE-vscode extension only shows its viewer frames, which can be done only by loadingpackage.json, not extension.ts.

@stamalakhov
Copy link
Contributor

With MPQEditor related modules commented out in extensions.ts, the extension works as normally at the latest main branch.

@dayo09
Thank you. I'll try to fix it.

@stamalakhov
Copy link
Contributor

stamalakhov commented Sep 13, 2023

@dayo09
The problem code is

glob.sync(...)

at MPQEditor.ts. With this line commented ONE-vscode extension can be successfully loaded and used. I'm investigating how it can be fixed.

@stamalakhov
Copy link
Contributor

@dayo09
As a quick fix please see #1638, seems like it's more safe as readdirSync is used a lot in the extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug/Issue Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants