-
Notifications
You must be signed in to change notification settings - Fork 9
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
Experience crashes when I try to switch back by clicking icon #38
Comments
seems like this is a race condition taht should be reproducible with iterm, did you tried?
… On 31 Jan 2018, at 18:47, MindTooth ***@***.***> wrote:
setProfile
|
Reverted now to 0.1.2, and haven't had a crash yet. Replicating the steps from before. |
let me know if you find the offending commit.
the affected code is in your crashlog (3 com.pancake.therm 0x000000010eecf2ab -[PTYSession setProfile:] + 159
)
are you using a special profile configuration or you have a clean environment when testing this?
… On 31 Jan 2018, at 20:28, MindTooth ***@***.***> wrote:
Reverted now to 0.1.2, and haven't had a crash yet. Replicating the steps from before.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#38 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGIjGzeLGrSVscEs0lT5qJYcM8jnKWPTks5tQL7ygaJpZM4R0YZz>.
|
i can reproduce it |
|
Good. So I'm not the only one. P.S.: What about enabling Travis on the repo? Just needs to register the project I guess. |
wanna send a pr enabling travis here? can you confirm master works for you? i can upload a build if you want. |
I can tro compile myself first. Give me 2 sec Travis is already enabled I believe (the old config files lies there). You only need to add the app itself to the GitHub repo. Ref. Settings > Integrations & services > Add service > Pick Travis CI. |
Nice. Pushed the update to Caskroom as well. Installed it via it. |
Thanks!!
… On 31 Jan 2018, at 22:48, MindTooth ***@***.***> wrote:
Nice. Pushed the update to Caskroom as well. Installed it via it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Been doing some testing with Travis-CI. But I encounter a issue. https://travis-ci.org/MindTooth/Therm/ Had to test it out yesterday after we spoke about it :P
The command is taken from Apple: https://developer.apple.com/library/content/technotes/tn2339/_index.html#//apple_ref/doc/uid/DTS40014588-CH1-UNIT
|
Seems to randomly crash. It crashes after I've opened the windows twice and closed them twice using
exit
. And then switching between other windows (Safarai & Mail).Using the latest 0.1.3. Not sure I had the issue with 0.1.2. I'll try to revert to see if it persist.
The text was updated successfully, but these errors were encountered: