Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Error: Process execution timed out #169

Closed
jjlorenzo opened this issue Mar 7, 2016 · 20 comments
Closed

Error: Process execution timed out #169

jjlorenzo opened this issue Mar 7, 2016 · 20 comments

Comments

@jjlorenzo
Copy link
Contributor

Error: Process execution timed out
    at /Users/jjlorenzo/.atom/packages/linter-flake8/node_modules/atom-linter/lib/helpers.js:79:18

After reinstalling linter-flake8, #168 was fixed, but now appears this.

OSX 10.11.3
Atom 1.6.0-beta7

@Arcanemagus
Copy link
Member

You should only be hitting this if flake8 is taking longer than 10 seconds to run 😮

Does flake8 take that long from the Terminal?

@jjlorenzo
Copy link
Contributor Author

jjlorenzo commented Mar 8, 2016 via email

@Arcanemagus
Copy link
Member

I'm adding an option that disables throwing the error, most likely what is going on is a process is getting stuck (for whatever reason), everything else moves on, and then after a bit it's getting killed and throwing this... which we don't really care about.

@jjlorenzo
Copy link
Contributor Author

Thanks!

@ryanisnan
Copy link

I'm also having this issue, when flake8 runs fine from terminal.

I've also noticed that this is an issue with https://github.com/AtomLinter/linter-phpcs/ and https://github.com/AtomLinter/linter-csslint/.

All three of these projects seem to have the issue in the same file

.atom\packages\<specific_atom_linter_here>\node_modules\atom-linter\lib\helpers.js:79:18

@v3ss0n
Copy link

v3ss0n commented Mar 28, 2016

I am having this issue , anytime it lints python file. Flake8 runs perfectly fine in terminal.
It happens everytime when linter is ran (when saving the file)

@v3ss0n
Copy link

v3ss0n commented Mar 28, 2016

Also it fails to do any code analysis.

@ryanisnan
Copy link

@Arcanemagus is there any way the AtomLinter team can look into this, given that it's happening in multiple linters?

@Arcanemagus
Copy link
Member

@ryanisnan I'm actually working on the PR I was talking about earlier, sorry about the delay.

For most linters this isn't an issue, for a few that need to legitimately run for that long an option should be provided, for most where this is triggering though it's a sign that the linter itself is getting stuck running for some reason.

As this particular linter seems to fall into both of the latter categories I'm simply going to provide an option that disables the timeout if the user desires.

@ryanisnan
Copy link

Thanks for the reply @Arcanemagus. I'm interested to see the PR, however, I do not think that the runtime of flake8 is the issue. Given your comments, I think we are solving the wrong problem (at least in my instance, and others in the thread).

I can run flake8 just fine from terminal on the current file I'm working in, which is 1146 lines long, sub 1 second. Saving the file from within Atom triggers the Process execution timed out errors.

There very likely is a use-case for linters that do take longer and would require the work you're proposing, but I have no reason to suspect that this is the issue being discussed here.

@ryanisnan
Copy link

Also, given that it is happening to some people and not others, it is possibly an environment-specific issue (Atom can't find the linter, permission issue perhaps, etc.). I would love to help debug the issue in any way possible, but am an atom newb.

@Arcanemagus
Copy link
Member

@ryanisnan:

  • How long does flake8 take to run on a file that Atom triggers the error on?
  • What OS are you on?
  • Do you ever get results? Or do all lints fail?

@ryanisnan
Copy link

@Arcanemagus:

  • flake8 run on files that fail in Atom as expected. I have tested a few files ranging in line count and complexity, and in every case it runs within 1 second.
  • Currently on OS X 10.11.4 - also experienced it on 10.11.3
  • I never get results from this linter. I do however receive linting from the linter-pyflakes linter.

@Arcanemagus
Copy link
Member

Can you try out the latest master, enable the new option, and see if you get any results?

To do that:

git clone https://github.com/AtomLinter/linter-flake8.git
cd linter-flake8
apm install
apm uninstall linter-flake8
apm link

Then restart/reload Atom.

To switch back to the regular version:

cd linter-flake8
apm unlink
apm install linter-flake8

@ryanisnan
Copy link

@Arcanemagus Just tried that. Enabled the new Disable Timeout setting, made some changes that should have caused different flake8 errors, saved the file, waited ~20 seconds, and received no linting.

TIL about apm, thanks!

@ryanisnan
Copy link

@Arcanemagus as you may not be able to replicate the issue, let me know if I can help in any way. ryanisnan on freenode.

@Arcanemagus
Copy link
Member

@ryanisnan Can you place a breakpoint here and see if it is ever returning? 😕

@Arcanemagus
Copy link
Member

Marking as closed since the initial error would be fixed by the option and there was never a response to the secondary issue.

@v3ss0n
Copy link

v3ss0n commented Sep 2, 2016

Confirmed fixed.

@AjayJohnAlex
Copy link

@Arcanemagus
git clone https://github.com/AtomLinter/linter-flake8.git
cd linter-flake8
apm install
apm uninstall linter-flake8
apm link
i tried this code but im getting
'apm' is not recognized as an internal or external command,
operable program or batch file.
what should i do ??

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants