nextspace icon indicating copy to clipboard operation
nextspace copied to clipboard

Workspace Crashes

Open armm77 opened this issue 3 years ago • 10 comments

Creating a folder and renaming it, and selecting the top folder crashes the workspace. This behavior seems to me to be random, it does not always happen or it does not reproduce in the way it is proposed.

https://user-images.githubusercontent.com/3104815/140235403-3ea72968-8b4b-4963-9eeb-e534c2cbb4d7.mov

https://user-images.githubusercontent.com/3104815/140235436-b5b85870-9290-4ea9-a6cc-b945af08e1e3.mov

armm77 avatar Nov 04 '21 00:11 armm77

I think you've faced several distinct problems here and in issue #399. Several events happend at once:

  • application crashes (Terminal) and lead to Workspace's crash (I can't reproduce this - I got only Terminal crash but appicon in Dock became weird);
  • Workspace crashes
  • Login panel interprets Workspace's crash as "Power Off" request and proceeds to OS shutdown (I've made serveral changes to this part of Login);

So, I plan to stick to following plan:

  1. Fix Login problem with Workspace exit status handling.
  2. Fix problem in Terminal (I understand what the cause of a problem)
  3. Fix problem in Workspace

Well done, thank you!

trunkmaster avatar Nov 04 '21 16:11 trunkmaster

I can confirm that the VM or my physical equipment no longer shuts down when the event occurs. only a restart of the Workspace is presented.

And as an image explains more than 1000 words, I leave a video of the behavior.

https://user-images.githubusercontent.com/3104815/140519179-355581ac-691f-4cd2-91d8-e252e3f0ff75.mov

armm77 avatar Nov 05 '21 13:11 armm77

I cannot reproduce the crash issue in GWorkspace. I don't know if it depends on specific changes of Workspace or not.

rmottola avatar Jan 14 '22 09:01 rmottola

hi @rmottola, these are tests on Workspace, i have not done GWorkspace tests.

armm77 avatar Jan 14 '22 15:01 armm77

@rmottola, I'ts definitely Workspace-specific issue. I suppose it's somehow connected to using KVO in Image View. It's hard to reproduce this bug for me - I'll investigate it later.

trunkmaster avatar Jan 14 '22 16:01 trunkmaster

@armm77 I guess I've fixed this bug. I can reproduce it no more. Check it out, please.

trunkmaster avatar Jan 18 '22 10:01 trunkmaster

Hi @trunkmaster, I perceive the file manager to be more solid, but I was able to reproduce the event. I tried on Fedora 35 and CentOS Stream 8, with the latest commits https://github.com/trunkmaster/nextspace/commit/2545727f52228ba7e02a16f0a51cfecffcaf054d.

https://user-images.githubusercontent.com/3104815/150061791-3c061c35-e0d3-4168-8494-ef9ffe54fab3.mov

armm77 avatar Jan 19 '22 05:01 armm77

hi @armm77 I've created almost 20 folders without crash. Could you please run Workspace under debugger and send me backtrace? I've put script for that in Applications/Workspace/gdb_Workspace.sh. Here is the short instruction:

  1. Login into user session as armm
  2. Login via SSH as armm and cd into NEXTSPACE sources Applications/Workspace.
  3. Run script ./gdb_Workspace.sh - this attaches GDB debugger into running Workspace. Wait for GDB to complete its loading.
  4. Type c and hit Return to continue running Workspace.
  5. Proceed with actions you did in the video. Upon crash Workspace stops responding.
  6. Switch to GDB session and type btand hit Return - it show code calling backtrace.
  7. Send me all text you'll see after you've entered bt.

trunkmaster avatar Jan 19 '22 20:01 trunkmaster

hi @trunkmaster, with the last commit https://github.com/trunkmaster/nextspace/commit/bfb80b702dae3860795f285234c2045a8ef65c57 I couldn't get it to fail, but the following video support was presented.

https://user-images.githubusercontent.com/3104815/150233160-4827d79b-9688-41c7-b550-675211494682.mov

test2.txt

armm77 avatar Jan 19 '22 23:01 armm77

Confirmed, thanks.

trunkmaster avatar Jan 20 '22 22:01 trunkmaster

@armm77 the problem should be fixed with latest commit. Please check.

trunkmaster avatar Dec 21 '23 11:12 trunkmaster

Hi @trunkmaster, the initial error is no longer replicated, but I have a problem when renaming the folders, it appears that the name is changed but when creating a subfolder the parent folder returns to its original name.

armm77 avatar Dec 21 '23 13:12 armm77

@armm77 did you see that problem with Icon viewer or with Browser viewer also?

trunkmaster avatar Dec 21 '23 20:12 trunkmaster

@trunkmaster Icon Viewer

armm77 avatar Dec 21 '23 21:12 armm77

@armm77 Browser Viewer works correctly, right?

At the moment I can see some exceptions while using Icon Viewer so renaming may be incorrect with Icon Viewer. I'll try to fix it.

trunkmaster avatar Dec 21 '23 21:12 trunkmaster

@trunkmaster Yes, in browser view it works correctly.

armm77 avatar Dec 22 '23 02:12 armm77

@armm77 latest commit should fix problem with Icon Viewer you've mentioned. Please test and tell me if you'll find more inconsitensies.

trunkmaster avatar Dec 25 '23 22:12 trunkmaster

@trunkmaster, I confirm that the incident I reported no longer occurs, so I close the incident. 👍🏻

armm77 avatar Dec 25 '23 23:12 armm77