This repository was archived by the owner on Aug 30, 2020. It is now read-only.
Conversation
compiler and linker paths.
- PEP8 style doc string
- Simplified patching (uses only re.sub)
- Notification is printed when the patching function is invoked
Resolves #53.
…as already been upgraded
Fixed AUR URL. AUR4 was temporary.
Addded avr, arm symlinks to fake toolchain
Add compatibility with Python2 and Python3
…83ecfc6 In the FAQ of https://github.com/Valloric/YouCompleteMe, there was an issue of C++ standard library headers not found (ycm-core/YouCompleteMe#303) that has been resolved with a workaround, and @cpradog has kindly implemented it. However, this workaround has not yet been incorporated into template.py in https://github.com/rdnetto/YCM-Generator, so adding this becomes the main purpose of this fork.
Fixed the issue of C/C++ std headers not found on OSX
sorted() fails in python3 due to comparison of String and Tuple
This patch updates the README to reflect the support for the meson (Ninja) build system.
Also removed spurious semicolons
Added meson + ninja support
rdnetto
suggested changes
Nov 9, 2017
Owner
rdnetto
left a comment
rdnetto
left a comment
There was a problem hiding this comment.
Ignore what I said earlier, the wrong target branch confused me.
Please raise the PR against develop so I can review your changes in isolation.
Author
|
I want to add support for ESP32 xtensa compilers. For that I need access to
repo, Can u provide that. I have these changes on a clone @
onb3891/YCM-Generator
…On Thu, Nov 9, 2017 at 2:47 PM, rdnetto ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Ignore what I said earlier, the wrong target branch confused me.
Please raise the PR against develop so I can review your changes in
isolation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#110 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AftJP3Z4AEH9oW0MqmV5a-vieJX_U8deks5s0sMugaJpZM4QMOw5>
.
|
Owner
You don't need write access to the repo - it should be sufficient for me to merge your changes in the PR. The problem is that you've forked from the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ESP32 xtensa toolchain