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

Update error reporting #17

Open
chengsun opened this issue Jan 7, 2015 · 0 comments
Open

Update error reporting #17

chengsun opened this issue Jan 7, 2015 · 0 comments

Comments

@chengsun
Copy link
Contributor

chengsun commented Jan 7, 2015

Right now error reporting is done by calling ui.Error to print a red message. However, this message can easily get lost amongst the noise of any other builders running simultaneously. This is why other builders also do a state.Put("error", err) before returning multistep.ActionHalt -- this way Packer can repeat the error at the end, when all the builders have terminated.

(I avoided doing this at first because it would introduce a huge amount of additional code noise, so if there's a clean way to do this then that would be good.)

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

1 participant