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

Fix detection of xtensa ar tool #6

Merged
merged 1 commit into from
Apr 11, 2015
Merged

Fix detection of xtensa ar tool #6

merged 1 commit into from
Apr 11, 2015

Conversation

hebnern
Copy link
Contributor

@hebnern hebnern commented Jan 8, 2015

Fixes build error referenced in issue #5

@jholster
Copy link

I can confirm that this patch works on OS X Yosemite.

@bjpirt
Copy link

bjpirt commented Mar 3, 2015

I can also confirm this patch works - please merge!

@straend
Copy link

straend commented Mar 15, 2015

Confirms that this works on Yosemite 10.10.2.

@hebnern
Copy link
Contributor Author

hebnern commented Apr 11, 2015

Hey @tommie any hope of getting this merged so that building esp-open-sdk is less of a pain on OS X?

@tommie
Copy link
Owner

tommie commented Apr 11, 2015

Ouch, sorry. For some reason I didn't receive email notifications for this repo. Looks good.

tommie added a commit that referenced this pull request Apr 11, 2015
Fix detection of xtensa ar tool.
@tommie tommie merged commit bcb75b6 into tommie:master Apr 11, 2015
@hebnern
Copy link
Contributor Author

hebnern commented Apr 11, 2015

Sweet, thanks!

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

Successfully merging this pull request may close these issues.

5 participants