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
crash at startup if default shell is zsh #20
Comments
not fixed.. i believe it is a different bug |
Thanks, can you do the following for me.
This will drop you into the gdb prompt, type 'run' to start the program. It should then immediately halt and bring you back to the prompt in gdb due to the error. Type the command 'backtrace' to get the backtrace and paste the backtrace into this bug report. |
|
|
even using an empty .zshrc terminix doesn't startup |
Thanks for all the info, I'll do some investigation today but without a backtrace it might be challenging to figure out what is going on. I'm not sure this is related to zsh or is something else as another user has reported that zsh is working fine for him with the latest release. I assume gnome-terminal works fine for you? |
I'm not able to reproduce, please try the new release I just pushed. If the problem still exists I'll build a special debug version that will include some tracing to try to track this down. |
gnome-terminal works correctly with both bash and zsh (with oh-my-zsh extension), terminix works only with bash. if you have a debug version I can try it. |
I just tried terminix today with zsh as default. Worked as expected on my up-to-date Arch installation. |
I've created a dub release that includes a bunch of tracing so hopefully I can get an idea where it is failing. You can download it at http://www.gexperts.com/files/terminix.zip. Note to install it you will need to unzip it into your / as per non-package release instructions as follows:
|
-> % terminix (terminix:3763): GLib-GIO-ERROR **: Settings schema 'com.gexperts.Terminix.Keybindings' does not contain a key named 'terminal-copy' |
Sorry I missed one step you need to do before running it:
That will fix the schema error, my apologies for missing that and thanks for your patience. |
now it started correctly... luca@archos [09:14:44] [~] |
That's awesome, I spent some time tweaking things to get it closer to how gnome-terminal interacts with the VTE back-end so maybe that did the trick. Can you confirm that the new 0.38.0 release in AUR works for you as well and if so close this issue. |
removed terminix from my machine and then installed 0.38.0 from AUR but it doesn't work.. |
That's disappointing, the only thing I can think of that is causing this is a compiler optimization in the release version. If you have the time to help with this, what I can do is start with the debug version and then add the release compiler switches to it one at a time to try to find which one is causing the issue. I've uploaded a baseline version at http://gexperts.com/files/terminix-test.zip, this uses a custom build setting that is functionally equivalent to the debug. Can you try this and confirm that it works and if it does I'll upload a new version with one of the switches enabled. |
it works :) |
Thanks, can you try this one next, I disabled bounds checking |
i confirm it works also this one! |
To try to speed this up a bit I went ahead and pre-built three more versions with each one having a different flag enabled: "buildOptions": ["noBoundsCheck", "releaseMode", "debugInfo"] ["noBoundsCheck", "optimize", "releaseMode", "debugInfo"] ["noBoundsCheck", "optimize", "inline", "releaseMode", "debugInfo"] Please try them out and let me know if any fail, thanks for your time in doing this, really appreciate it. |
Tried and all of these releases work correctly.
|
OK, two more versions with the last one being semantically identical to how DUB builds the release but just me specifying the flags directly. If the last version works, that means either I've accidentally fixed your issue or DUB is doing more flags in then what the documentation states. Also, that trace output is from my logging, there is a build flag to compile it out for the release version. Version 7 that I uploaded will not have this trace anymore. Finally, you don't need to compile the schema anymore, it hasn't changed in these test versions. (http://gexperts.com/files/terminix-test6.zip)[http://gexperts.com/files/terminix-test6.zip] |
both of them work correctly! |
Can you confirm one more time that the release version doesn't work? I'm in the middle of some changes now, but I'll do a build later tonite for you to test with the normal build release options in case I did something to fix this. |
I confirm that 0.38.0 from AUR doesn't work. |
Thanks, I'll do a new build in the morning as I got distracted with some effort on localization. |
Here are two more, it is the same code with 8 being compiled using the switches that were the same as release and 9 being compiled with the normal release switch. If 8 works and 9 fails I'll just keep compiling it with the switches used in 8 from now on. Note you will need to compile schemas this time as I added some new preferences. http://gexperts.com/files/terminix-test8.zip |
tested 8 and 9 and both works... |
Thanks, there will be a new release tomorrow. Let me know if the issue still exists then and if I can look into the package in more detail. |
0.40 from aur works correctly.. thanks for your time.. bug can be closed! |
crash at startup if default shell is zsh, works correctly if bash shell
gen 14 09:51:17 archos kernel: zsh[5924]: segfault at 0 ip 00007ff18051ca0f sp 00007ffd80464048 error 4 in libc-2.22.so[7ff180491000+19b000]
gen 14 09:51:17 archos systemd-coredump[5925]: Process 5924 (zsh) of user 1001 dumped core.
The text was updated successfully, but these errors were encountered: