You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[Bug]: About lib\vscode\extensions\node_modules\.bin #5379
lib\vscode\extensions\node_modules.bin should not be packaged as the path these soft link files referring to do not exist
Actual
I'm not quite sure if your team have realized it and decided to keep it but just from my view, tsc & tsserver(soft link file) in lib\vscode\extensions\node_modules.bin are meaningless and should not be packaged
As yarn run release:standalone will call postinstall.mjs script and it will delete all files in typescript lib & bin, the soft link files in screenshot refer to a non-existing path.
Sorry to ask more but could you tell me why we need to run yarn again in build-standalone-release.sh. Maybe it's useful in vscode source code but for built-in extension, they are packaged by webpack and all node_modules should have been added to each extension's dist folder. Why do we need to run the following command again to install node_modules?
yarn --production --frozen-lockfile --non-interactive
Sorry to disturb you guys in your busy time, looking forward to your reply
Logs
No response
Screenshot/Video
No response
Does this issue happen in VS Code or GitHub Codespaces?
Yeah maybe we should delete them. We do delete them for the deb and rpm packages that we create through nfpm but we only did that because otherwise nfpm will fail. We try not to mess with the upstream build too much otherwise maintenance burden increases but since we are already doing this for some packages I think we can go ahead and do them for the others.
build-standalone-release.sh is run on different platforms so we run yarn again so the native modules are compiled for the right platform (for example spdlog, keytar, argon, and some others I think). If there was a way to only recompile native modules that would be ideal but yarn does not seem to have that capability so we have to do a full install.
Activity
jsjoeio commentedon Aug 1, 2022
@code-asher might be able to provide an answer to this
code-asher commentedon Aug 2, 2022
build-standalone-release.shis run on different platforms so we runyarnagain so the native modules are compiled for the right platform (for example spdlog, keytar, argon, and some others I think). If there was a way to only recompile native modules that would be ideal butyarndoes not seem to have that capability so we have to do a full install.