X Tutup
The Wayback Machine - https://web.archive.org/web/20200915012951/https://github.com/mono/CppSharp/pull/1089
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

#1087 Fix stack overflow unless VS2017 is not installed #1089

Open
wants to merge 1 commit into
base: master
from

Conversation

@limenote135
Copy link

limenote135 commented May 17, 2018

Fixes #1087

@limenote135 limenote135 force-pushed the limenote135:fix_stackoverflow branch 2 times, most recently from 3a8e6dd to 0f172c0 May 17, 2018
@limenote135 limenote135 force-pushed the limenote135:fix_stackoverflow branch from 0f172c0 to f60ad1d May 17, 2018
@ddobrev
Copy link
Contributor

ddobrev commented May 17, 2018

Thank you for this fix. Unfortunately, it crashes for me with an invalid path. Assuming all tests pass for you and so does our CI, I am going to take a look myself as soon as possible. Meanwhile, please use your custom copy.

@limenote135
Copy link
Author

limenote135 commented May 17, 2018

Thank you for the checking. I'll use my change for now.
Please let me know if there are any problems!

@ddobrev ddobrev force-pushed the mono:master branch 12 times, most recently from 0464938 to 637018f Oct 15, 2018
@ddobrev ddobrev force-pushed the mono:master branch 4 times, most recently from 91e219c to 32da859 Jan 4, 2019
@tritao
Copy link
Collaborator

tritao commented Jan 14, 2019

Hey @limenote135.

I think we merged some VS fixes a little while ago.

Could you check if there is still an issue with latest revisions?

@ddobrev ddobrev force-pushed the mono:master branch from 1108e72 to e203463 Jan 15, 2019
@ddobrev ddobrev force-pushed the mono:master branch from 3164a89 to bc7f20a Feb 14, 2019
@tritao tritao force-pushed the mono:master branch 2 times, most recently from a0169c2 to 3ea7e97 Mar 10, 2019
@ddobrev ddobrev force-pushed the mono:master branch 2 times, most recently from a1559de to 304d673 Apr 10, 2019
@ddobrev ddobrev force-pushed the mono:master branch from 5f2de79 to 1d28807 May 1, 2019
@ddobrev ddobrev force-pushed the mono:master branch from 8ba8553 to 72368d5 Jun 18, 2019
@ddobrev ddobrev force-pushed the mono:master branch 6 times, most recently from 0e0b4cc to d90d328 Jun 26, 2019
@ddobrev ddobrev force-pushed the mono:master branch from 046f29c to 573272e Jul 13, 2019
@ddobrev ddobrev force-pushed the mono:master branch from ce27429 to e592a2b Sep 19, 2019
@ddobrev ddobrev force-pushed the mono:master branch 2 times, most recently from c8e9f62 to be54a8b Dec 18, 2019
@ddobrev ddobrev force-pushed the mono:master branch 6 times, most recently from 1610aa5 to 64b1efd Apr 13, 2020
@ddobrev ddobrev force-pushed the mono:master branch from 418c720 to 2acbf32 May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.
X Tutup