Skip to content

Fix: Compilation error #41#44

Open
koenmeersman wants to merge 1 commit into
pCloud:masterfrom
koenmeersman:test
Open

Fix: Compilation error #41#44
koenmeersman wants to merge 1 commit into
pCloud:masterfrom
koenmeersman:test

Conversation

@koenmeersman

Copy link
Copy Markdown

__has_builtin(x) defined in pcompiler.h hides definition in boost library.
renamed has_builtin to: PSYNC_HAS_BUILTIN

__has_builtin(x) defined in pcompiler.h hides definition in boost library.
renamed has_builtin to: PSYNC_HAS_BUILTIN
@plague-doctor

Copy link
Copy Markdown

Come on guys, merge this already...

@plague-doctor

Copy link
Copy Markdown

How much long will we need to wait for this fix? Please merge already

@Adriankhl Adriankhl mentioned this pull request Dec 5, 2018
@blackjackshellac

Copy link
Copy Markdown

Can someone merge this request? Pretty please.

@BartKeulen BartKeulen mentioned this pull request Mar 11, 2019
@fwinkl

fwinkl commented Aug 29, 2019

Copy link
Copy Markdown

As several people have already confirmed, this fixes a build problem on several platform. I just verified that this also fixes the build on the RPI 4 (buster). I don't think this can have any negative side effects. Could this be merged so people don't need to rediscover the same problem over and over again? I was just about to make a PR myself before seeing this one here...

@raicho81

raicho81 commented Jun 20, 2021

Copy link
Copy Markdown

I've fixed this but nobody realy cared to update the code in Git Hub. It is a low priority task there at pCloud. I've added also support for removable USB devices but this update is also not on Git Hub.

@sergeyklay

Copy link
Copy Markdown

Hello @raicho81 I'm working on my own fork too. Could you please share a link to your one.

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.

6 participants