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: Compilation error #41 #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

koenmeersman
Copy link

__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

Come on guys, merge this already...

@plague-doctor
Copy link

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

Can someone merge this request? Pretty please.

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

fwinkl commented Aug 29, 2019

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
Copy link

raicho81 commented Jun 20, 2021

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

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