Skip to content

fix template whitespace syntax#715

Merged
ras0219-msft merged 1 commit intomicrosoft:masterfrom
aduskett:template-fix
Aug 3, 2018
Merged

fix template whitespace syntax#715
ras0219-msft merged 1 commit intomicrosoft:masterfrom
aduskett:template-fix

Conversation

@aduskett
Copy link
Copy Markdown
Contributor

@aduskett aduskett commented Mar 9, 2018

Some files don't have a space inbetween the '<' and '::' charachters, which
will cause build failures on older toolchains. Adding a space inbetween these
two characters fixes the issue.

See http://autobuild.buildroot.net/results/797a9b5fdf6ab0f16f2249324b48292dfab61d9f/build-end.log
for more information.

Some files don't have a space inbetween the '<' and '::' charachters, which
will cause build failures on older toolchains. Adding a space inbetween these
two characters fixes the issue.

See http://autobuild.buildroot.net/results/797a9b5fdf6ab0f16f2249324b48292dfab61d9f/build-end.log
for more information.
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this pull request Mar 10, 2018
some files don't have a space inbetween the '<' and '::' charachters, which
will cause build failures on older toolchains. Adding a space inbetween these
two characters fixes the issue.

upstream-status: pending
microsoft/cpprestsdk#715

fixes:
http://autobuild.buildroot.net/results/797a9b5fdf6ab0f16f2249324b48292dfab61d9f
Signed-off-by: Adam Duskett <aduskett@gmail.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
@garethsb
Copy link
Copy Markdown
Contributor

This may also be resolved by -fpermissive on GCC 4.7 as far as I remember. See comments in #609.

@ras0219-msft ras0219-msft merged commit ae50037 into microsoft:master Aug 3, 2018
@ras0219-msft
Copy link
Copy Markdown
Contributor

Thank you for the PR, however note that we will not be testing with these old compilers in the future and make no guarantees that it won't be broken again :)

@aduskett aduskett deleted the template-fix branch August 3, 2018 17:22
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.

3 participants