Skip to content

[win32] Fix Mingw support#2

Closed
petrutlucian94 wants to merge 1 commit intoceph:ceph-octopusfrom
petrutlucian94:mingw
Closed

[win32] Fix Mingw support#2
petrutlucian94 wants to merge 1 commit intoceph:ceph-octopusfrom
petrutlucian94:mingw

Conversation

@petrutlucian94
Copy link

If POSIX is set, mingw will no longer prefix some function names
with an underscore, which is expected by libfmt. Rather than handling
them case by case, we'll just avoid the POSIX macros.

Signed-off-by: Lucian Petrut [email protected]

If _POSIX_ is set, mingw will no longer prefix *some* function names
with an underscore, which is expected by libfmt. Rather than handling
them case by case, we'll just avoid the POSIX macros.

Signed-off-by: Lucian Petrut <[email protected]>
@ml8mr
Copy link

ml8mr commented May 21, 2020

The Travis CI failure is due to the Android NDK Gradle build failing. This failure is due to a build config step that is missing the version parameter in the sdk use gradle [version] command. Without the version parameter, this use command is not useful and should be removed from the build config.

The changes requested in this PR work in my test environment.

@tchaikov
Copy link

@petrutlucian94 sorry, Lucian! i just noticed your PR. could you upstream this change to https://github.com/fmtlib/fmt ?

@petrutlucian94
Copy link
Author

petrutlucian94 commented May 21, 2020

@petrutlucian94 sorry, Lucian! i just noticed your PR. could you upstream this change to https://github.com/fmtlib/fmt ?

Right away.

fmtlib#1693

@petrutlucian94
Copy link
Author

@tchaikov it merged upstream: fmtlib#1696. Should I cherry-pick this commit or pull the entire master branch to ceph-octopus?

@tchaikov
Copy link

@petrutlucian94 i pushed ceph-pacific to ceph/fmt already. and created ceph/ceph#35259 to pick up your change.

@petrutlucian94
Copy link
Author

That's great, thanks! I guess I can close this PR then.

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