Komodo / KomodoEdit

Komodo Edit is a fast and free multi-language code editor. Written in JS, Python, C++ and based on the Mozilla platform.
http://www.komodoide.com/komodo-edit
Other
2.15k stars 302 forks source link

(Windows) Komodo crashes whenever Git or other tool updates an open file #2260

Open jimzag opened 7 years ago

jimzag commented 7 years ago

Short Summary

After Git SYNC (GitHub UI) or sometimes if changing branches I will find Komodo non responsive. The windows cursor and greyed-out appearance of the app. Need to close and re-open. I first noticed this with another tool, Pinegrow which writes web pages. If I had a particular page open in Komodo then saved that same page in Pinegrow, Komono crashes every time.

Steps to Reproduce

See above

Expected results

Actual results

Platform Information

Windows 10

Komodo Edit or IDE? IDE Komodo Version? Komodo IDE, version 10.1.4, build 89515, platform win32-x86. Built on Fri Dec 16 10:34:21 2016 Operating System (and version)?

Additional Information

-- eg. Error logs, screenshots, workarounds --

jimzag commented 7 years ago

I am curious if this problem has been reported by others. I noticed there have been no notations made to date on the status. Let me know if I can provide anything in particular: logs, etc.

Defman21 commented 7 years ago

Once you reproduce the crash again, attach your log file (pystderr.log in your profile folder)

Naatan commented 7 years ago

We use Komodo with GIT to develop Komodo and are not seeing this crash.

Does disabling GIT make a difference?

The next time it happens please enter your email in the crash report and put a link to this bug in the comments.

jimzag commented 7 years ago

The lockup happened again today.  Not as frequently recently - may depend on something specific in the files. Here is the log and screenshot.  I had just performed a 'SYNC' in the Github UI and I know it had updates some of the files which had been open in Komodo.

jimzag commented 7 years ago

Sent an example of the crash to S. Kislyakov.  Hope it is helpful in finding the cause.

jimzag commented 7 years ago

Also noticed Komodo is taking 25 - 30% CPU, so not crashed but stuck in processing.

Naatan commented 7 years ago

Please respond using the github web interface. Email responses include a lot of useless extra text and do not include attachments (we cannot see your logs).

Additionally, you did not say whether disabling GIT helped or not?

Defman21 commented 7 years ago

I believe github hides my e-mail to something that behaves like noreply. So I didn't get your message :( Please do what Nathan said :)

jimzag commented 7 years ago

komono_not_responding

pystderr.zip

Log and screen capture attached. Also, I noted that the processor was at 25-30% when Komodo was in the 'not responding' state seen in the screen capture.

Regarding whether 'Disabling GIT' makes a difference, I assume that is a setting in Komodo. I don't use the Git integration. I had a quick look around in Preferences and did not see a setting like that. Let me know where to look.

th3coop commented 7 years ago

I had a quick look around in Preferences and did not see a setting like that. Let me know where to look.

@jimzag Open the preferences and search git in the search field at the top of the left column.

Naatan commented 7 years ago

This seems fishy:

Exception in thread Thread-20:
Traceback (most recent call last):
  File "C:\Program Files (x86)\ActiveState Komodo IDE 10\lib\python\lib\threading.py", line 808, in __bootstrap_inner
    self.run()
  File "C:\Program Files (x86)\ActiveState Komodo IDE 10\lib\mozilla\python\komodo\watchdog\observers\api.py", line 186, in run
    self.queue_events(self.timeout)
  File "C:\Program Files (x86)\ActiveState Komodo IDE 10\lib\mozilla\python\komodo\watchdog\observers\read_directory_changes.py", line 73, in queue_events
    self.watch.is_recursive)
  File "C:\Program Files (x86)\ActiveState Komodo IDE 10\lib\mozilla\python\komodo\watchdog\observers\winapi_common.py", line 124, in read_directory_changes
    None)
  File "C:\Program Files (x86)\ActiveState Komodo IDE 10\lib\mozilla\python\komodo\watchdog\observers\winapi.py", line 106, in _errcheck_bool
    raise ctypes.WinError()
WindowsError: [Error 5] Access is denied.

Are you perhaps pointing Komodo at files/directories that it does not have permission to read?

jimzag commented 7 years ago

The GIT operations at that time included pressing the SYNC button (on the GIT Windows App), and possibly creating a new branch and changing the selected branch.  In GIT obviously if you switch branches some files disappear or change according to what has occurred on that branch.  Perhaps there is brief inavailability of the folder during these type of GIT operations, and perhaps a timing-dependent failure encountered if there is a certain timing with respect to processing of any updates as Komono observes folder/ file changes. I am pretty sure all files open at the time were in folders under c:\Dev\JBSDEV\BrowserApp2.  There were perhaps 6 files open.

Naatan commented 7 years ago

Please see whether disabling GIT works around the issue, see screenshot for assistance:

image

jimzag commented 7 years ago

komodo_pystderr.zip

Another crash occurred today after disabling Git integration. Here is the sequence of events:

  1. Disable Git integration
  2. close and re-open Komodo
  3. Select the Master branch in Git (from the Git UI)
  4. press Git UI SYNC button
  5. Observed that Komodo was no longer running
Naatan commented 7 years ago

See if disabling this setting makes a difference:

image

jimzag commented 7 years ago

I have made the change.   Now that file change detection is turned off, is there a way I can tell Komodo to reread all files.  I certainly don't want to be editing an old version, such that when I save the changes it actually overwrites the current version with older content.

Naatan commented 7 years ago

@jimzag the point of toggling that setting is not to fix your problem, but rather to diagnose it. Are you saying that toggling it worked around your issue? Komodo is no longer hanging/crashing?

jimzag commented 7 years ago

Right - understood.Yes, I have not seen a crash since toggling the 'detect updates' setting.  I have repeated the steps which previously would have caused the crash or hangup. Jim

Naatan commented 7 years ago

Thanks @jimzag, I'm gonna see about creating a build with increased file notification logging. Until then I would recommend you use the workaround. Note you can manually refresh files by right clicking their tab and hitting "Refresh Status".