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

Update fbos deps to v2024.05.20.00 #9968

Closed
wants to merge 1 commit into from

Conversation

czentgr
Copy link
Collaborator

@czentgr czentgr commented May 29, 2024

The main fix is for CentOS9 to find libdwarf in the correct system install path (found in folder libdwarf-0). This was added at the end of April here facebook/folly@ebd05bf

I decided not to use the latest due to an error with MacOS building fbthrift

CMake Error at thrift/compiler/CMakeLists.txt:58 (add_library):
  No SOURCES given to target: compiler_ast

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 29, 2024
Copy link

netlify bot commented May 29, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit c94ba03
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/666c78d07e87bb000836b3a8

@czentgr
Copy link
Collaborator Author

czentgr commented May 29, 2024

@majetideepak FYI.

@czentgr czentgr force-pushed the cz_advance_fbos_deps branch from 702382f to 1df7a17 Compare May 29, 2024 20:56
@majetideepak majetideepak marked this pull request as ready for review May 30, 2024 04:38
@czentgr czentgr force-pushed the cz_advance_fbos_deps branch from 1df7a17 to 99394f8 Compare May 31, 2024 17:49
Copy link
Collaborator

@assignUser assignUser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@czentgr czentgr force-pushed the cz_advance_fbos_deps branch 4 times, most recently from 43f7011 to 86328ef Compare June 10, 2024 15:19
@czentgr
Copy link
Collaborator Author

czentgr commented Jun 10, 2024

@kgpai Can you please help commit this?

@majetideepak
Copy link
Collaborator

@czentgr, @kgpai Let's merge this after the Centos9 update. That should fix the Build & Push Docker Images job as well.

@pedroerp
Copy link
Contributor

@majetideepak @assignUser please add the "ready-to-merge" whenever you want this merged :)

@assignUser
Copy link
Collaborator

@pedroerp yes, the label has been working well! We are waiting on a rebase so CI will be green :)

@majetideepak
Copy link
Collaborator

@czentgr setup-centos9.sh needs an update as well.

@czentgr czentgr force-pushed the cz_advance_fbos_deps branch from 86328ef to c94ba03 Compare June 14, 2024 17:07
@majetideepak majetideepak added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Jun 17, 2024
@facebook-github-bot
Copy link
Contributor

@kagamiori has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@kagamiori merged this pull request in 9558dcb.

Copy link

Conbench analyzed the 1 benchmark run on commit 9558dcb7.

There was 1 benchmark result indicating a performance regression:

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants