-
-
Notifications
You must be signed in to change notification settings - Fork 685
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
fix(feat): disable_automatic_asset_installation
#1226
fix(feat): disable_automatic_asset_installation
#1226
Conversation
My bad, |
|
I'm calming down now... fix(feat): disable mkdir in wasm_wm fixup! fix unused code warnings fix(setup): properly show features in setup Should I keep initial patch? |
@CaptainBloodz |
Ouch:0.26.0 here. |
Failing the same on 0.26.1... |
It gets squashed anyway on merge. So if you want to I can squash it. |
92d4f74
to
ea6ad5d
Compare
@CaptainBloodz |
This fixes a regression in the feature system: The asset installation didn't get turned off by the feature. Add error logging to the install functions. Properly show features in setup * fix(feat): disable `mkdir` in wasm_wm The `wasm_vm` creates the `plugin_dir` before mounting it. Turn that behavior off on `feature-disable-asset-installation`. Alternative: Is this even needed? We make sure the directory is there upon the normal asset installation. fixes zellij-org#1130
ea6ad5d
to
22cff54
Compare
Patching fine with 4 patch files... |
fails as in 0.26.0;
|
Could you check please with |
Rebuilding with all-in-one single patch file... |
0.26.0:
|
|
Do you have the tab bar located in |
|
|
Could you try changing your plugin config to (in
Thanks for all the help so far! I think we are getting very close. |
Which one? |
Oh, sorry it's the |
Isn't there a right issue on .../plugins dir and/or content? Once config.yaml created & modified:
|
|
Do you mind trying to run it with:
|
zellij --data-dir /usr/share/zellij Error occurred in server:
Error: thread 'wasm' panicked at 'called |
Thank you! Do you mind trying again with the |
This is to be expected. The |
Plz confitm those two patches, Are additional to all-in-one patch file, or should new ones should replace parts in all-in-one?
Elaborate, plz.
Currently checking about that, as it is out of tree... |
Currently building all-in-one+2 new patches... |
|
be35cf5
to
1d46a86
Compare
Rebuild with all-in-one + 2 patches + 1d46a86 queued. |
@CaptainBloodz Yes! |
disable `mkdir` in `wasm_vm` on `feature-disable-asset-installation`
1d46a86
to
f820a8f
Compare
Current compilation snippet:
|
Working here, without any ~/.config/zellij dir.
|
This fixes a regression in the feature system:
The asset installation didn't get turned off by the feature.
fixes #1130