Skip to content

Build: Remove spawn from exec command (Windows) #3167

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

Merged
merged 1 commit into from
May 18, 2015

Conversation

Chris--A
Copy link
Contributor

It appears in ant that spawn on windows is not needed/usable with exec.

This error now is visible due to the commit here: 37e2a19, before hand it was ignored.

Currently a windows build generates this (using ant run):

windows-run:
spawn does not allow attributes related to input, output, error, result
spawn also does not allow timeout
finally, spawn is not compatible with a nested I/O

BUILD FAILED
F:\Arduino\build\build.xml:71: The following error occurred while executing this line:
F:\Arduino\build\build.xml:869: You have used an attribute or nested element which is not compatible with spawn

If I remove the spawn attribute everything compiles and runs fine.
If I remove the failonerror attribute, it also works fine.

It appears that spawn is only useful for background processes.

@ffissore ffissore added the Component: IDE The Arduino IDE label May 18, 2015
ffissore added a commit that referenced this pull request May 18, 2015
Build: Remove spawn from exec command (Windows)
@ffissore ffissore merged commit 19aa7ed into arduino:master May 18, 2015
@ffissore
Copy link
Contributor

Thanks! We need more windows hackers

@Chris--A Chris--A deleted the no-spawn branch May 18, 2015 12:04
@ffissore ffissore modified the milestone: Release 1.6.5 May 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: IDE The Arduino IDE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants