Skip to content
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

Could not find 'docker'. The program may not be installed error #2145

Closed
3 tasks done
pedromzadeh opened this issue May 29, 2018 · 9 comments
Closed
3 tasks done

Could not find 'docker'. The program may not be installed error #2145

pedromzadeh opened this issue May 29, 2018 · 9 comments

Comments

@pedromzadeh
Copy link

pedromzadeh commented May 29, 2018

Description

I have installed atom-beautify from the Atom packages tab like any other normal package. I have installed uncrustify on my machine too (win 10). I can confirm that it works because where.exe uncrustify returns the correct path to the executable. I have given atom-beautify the correct path to the configuration files for uncrustify. Nonetheless, when I try to save my .cpp files, I get the error "Could not find docker. The program may not be installed."

Indeed, the issue is obvious. I need to install docker. That's not what I'm asking here. I have been using atom-beautify for a while now and have gone through setting up twice. I have never had this issue with docker. It's the first time I'm hearing about it.

Is this a bug? Or, is this a recent new implementation that now requires me to install some third-party app (so called docker) to get it working?

Just saying, before it used to be install atom-beatify, install uncrustify, give the correct configuration files' paths, and it's done. Now, there's docker! Note that #1687 and #1441 seem to be close relatives of my problem, but I wasn't able to seek what I was looking for there. My advanced apologies if somehow I'm posting a duplicate question.

Debug

Here is a link to the debug.md

Checklist

I have:

  • Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
  • Reloaded (or restarted) Atom to ensure it is not a caching issue
  • Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
    so I know this is not a duplicate issue
@ghost
Copy link

ghost commented May 29, 2018

Please follow the issue template provided. More specifically, update the original comment for this issue by adding a link to the required debug.md gist which includes debugging information that answers our most commonly asked questions. Thank you.

@stevenzeck
Copy link
Contributor

Ideally it shouldn't be displaying an error around Docker, that is likely caused by #2129. @Glavin001 we should fix that.

The relevant error from the log is:

2018-05-29T19:30:26.413Z - debug: [] error Error: spawn uncrustify ENOENT
    at exports._errnoException (util.js:1050:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
    at onErrorNT (internal/child_process.js:367:16)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)
2018-05-29T19:30:26.413Z - debug: [] error Error: spawn uncrustify ENOENT
    at exports._errnoException (util.js:1050:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
    at onErrorNT (internal/child_process.js:367:16)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)

@Ptheguy can you try putting the full path to the Uncrustify executable into Settings -> Executables -> Uncrustify?

@pedromzadeh
Copy link
Author

@szeck87 Yes, that solved my issue. Thank you. Should I close the thread? Or, leave it open? You seem to have called for something more fundamental to be fixed.

@stevenzeck
Copy link
Contributor

Yea I'll open a separate issue. Thanks, glad to hear it's working.

@Glavin001
Copy link
Owner

Thank you for reporting this issue, @Ptheguy !

Is this a bug? Or, is this a recent new implementation that now requires me to install some third-party app (so called docker) to get it working?

Docker support was added as part of #1707 and #1687. The idea is to improve the installation by leveraging's portable Docker features (i.e. works the same on Windows, Mac, Linux, etc).

However, Docker is definitely not required, it should be a last resort / fallback. I'll take a look at #2146 and see about fixing this for yourself and future users. Thanks to @szeck87 , it sounds like you're no longer blocked, @Ptheguy ? Please let us know if you have any more issues. Thanks again!

@mukeshbasira
Copy link

It is asking for docker on c++ And I don't know what it is. why is this necessary for this package?

@stevenzeck
Copy link
Contributor

It's not required, it's a bug that was fixed in #2146 which hasn't been released yet.

@yoopersteeze
Copy link

yoopersteeze commented Aug 1, 2018

Seeing this with beautify version 0.32.5. when trying to beautify a shell script running Atom v1.29.0 on macOS 10.13.4

What I've tried:

  • Uninstalling and re-installing Atom Beautify. Uninstalled via settings in GUI and re-installed with the command: apm install atom-beautify
  • Reloaded / restarted Atom and also applied the update that was available
  • Ensure that "auto beautify shell files on save" is not enabled from the beautify package settings
  • Ran which docker from Terminal, which returned nothing

Is anyone else seeing this issue with shell scripts?
screenshot 2018-07-31 20 04 05

@simonmelouah
Copy link

Having this issue as well with Python3

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

No branches or pull requests

6 participants