{"action":"closed","issue":{"url":"https://api.github.com/repos/themadinventor/esptool/issues/15","labels_url":"https://api.github.com/repos/themadinventor/esptool/issues/15/labels{/name}","comments_url":"https://api.github.com/repos/themadinventor/esptool/issues/15/comments","events_url":"https://api.github.com/repos/themadinventor/esptool/issues/15/events","html_url":"https://github.com/themadinventor/esptool/issues/15","id":53970685,"number":15,"title":"Some fixes from nodemcu","user":{"login":"sej7278","id":817564,"avatar_url":"https://avatars.githubusercontent.com/u/817564?v=3","gravatar_id":"","url":"https://api.github.com/users/sej7278","html_url":"https://github.com/sej7278","followers_url":"https://api.github.com/users/sej7278/followers","following_url":"https://api.github.com/users/sej7278/following{/other_user}","gists_url":"https://api.github.com/users/sej7278/gists{/gist_id}","starred_url":"https://api.github.com/users/sej7278/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sej7278/subscriptions","organizations_url":"https://api.github.com/users/sej7278/orgs","repos_url":"https://api.github.com/users/sej7278/repos","events_url":"https://api.github.com/users/sej7278/events{/privacy}","received_events_url":"https://api.github.com/users/sej7278/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-10T20:44:27Z","updated_at":"2015-01-11T01:26:17Z","closed_at":"2015-01-11T01:26:17Z","body":"Could you have a look at https://github.com/nodemcu/nodemcu-firmware/blob/2709258c44b4fa2397df736b12b9c2ef86dc4d89/tools/esptool.py please, and see if you can/should merge in the fixes there?\r\n\r\nbasically its setting permissions to 755 so it can be executed on unix and removing the version from the shebang from me as discussed earlier.\r\n\r\nalso from @nodemcu it seems like a fix for the windows hanging issue - basically uses xt-objcopy instead of xtensa-lx106-elf-objcopy and  xt-nm instead of xtensa-lx106-elf-nm\r\n\r\ni can't really do a direct PR as its across two different projects, nodemcu should probably use your repo as a submodule i guess, or leave it to the user to install esptool.py in the $PATH somewhere."}}