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][backport]adapt to gcc 13 #3

Merged
merged 2 commits into from
Nov 8, 2024
Merged

[fix][backport]adapt to gcc 13 #3

merged 2 commits into from
Nov 8, 2024

Conversation

adzfolc
Copy link

@adzfolc adzfolc commented Nov 5, 2024

[fix][backport]adapt to gcc 13

backport from

@adzfolc adzfolc closed this Nov 6, 2024
@adzfolc adzfolc reopened this Nov 6, 2024
@adzfolc adzfolc changed the title [feat]adapt to gcc 13 [fix][backport]adapt to gcc 13 Nov 6, 2024
anand76 and others added 2 commits November 6, 2024 20:32
Summary:
Change the library order in PLATFORM_LDFLAGS to enable fbcode platform 10 build with folly. This PR also has a few fixes for platform 10 compiler errors.

Pull Request resolved: facebook#10708

Test Plan:
ROCKSDB_FBCODE_BUILD_WITH_PLATFORM010=1 USE_COROUTINES=1 make -j64 check
ROCKSDB_FBCODE_BUILD_WITH_PLATFORM010=1 USE_FOLLY=1 make -j64 check

Reviewed By: ajkr

Differential Revision: D39666590

Pulled By: anand1976

fbshipit-source-id: 256a1127ef561399cd6299a6a392ca29bd68ca44
Summary:
Like other versions before, gcc 13 moved some includes around and as a result <cstdint> is no longer transitively included [1]. Explicitly include it for uint{32,64}_t.

[1] https://gcc.gnu.org/gcc-13/porting_to.html#header-dep-changes

Pull Request resolved: facebook#11118

Reviewed By: cbi42

Differential Revision: D42711356

Pulled By: ajkr

fbshipit-source-id: 5ea257b85b7017f40fd8fdbce965336da95c55b2
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