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

Grpc upgrade 1.70 synced #410

Draft
wants to merge 5 commits into
base: users/sacheta/grpc_upgrade_1.70.0
Choose a base branch
from

Conversation

Sacheta-NI
Copy link
Collaborator

No description provided.

@Sacheta-NI Sacheta-NI marked this pull request as draft February 18, 2025 09:00
@Sacheta-NI Sacheta-NI changed the base branch from users/sacheta/grpc_upgrade_1.70.0 to master February 18, 2025 14:26
@Sacheta-NI Sacheta-NI marked this pull request as ready for review February 18, 2025 14:28
@Sacheta-NI Sacheta-NI changed the base branch from master to users/sacheta/grpc_upgrade_1.70.0 February 18, 2025 14:30
@Sacheta-NI Sacheta-NI changed the base branch from users/sacheta/grpc_upgrade_1.70.0 to master February 19, 2025 05:58
@Sacheta-NI Sacheta-NI changed the base branch from master to users/sacheta/grpc_upgrade_1.70.0 February 19, 2025 08:19
@@ -26,9 +26,9 @@ jobs:
# improvements to hosting and exports of the NILRT toolchain are made.
- name: Install NI Linux RT CC Toolchain
run: |
wget -nv https://download.ni.com/support/softlib/labview/labview_rt/2018/Linux%20Toolchains/linux/oecore-x86_64-core2-64-toolchain-6.0.sh

Choose a reason for hiding this comment

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

Was this upgrade made to make your product compatible with latest nilrt distros? We're currently trying to figure that out ourselves (discussion)

Copy link
Collaborator Author

@Sacheta-NI Sacheta-NI Feb 20, 2025

Choose a reason for hiding this comment

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

Hey, we were trying to upgrade gRPC version, the previous toolchain was using older version of gcc, we needed 7 or above. Hence needed to upgrade this link to newer toolchain.
Apart from this, I also tried to not use the link directly but building toolchain from nilrt repo. Was facing some issues related to memory that way. So yeah, decided to use the later version of it directly.
Also, this PR is not final yet. There would be some changes.

@Sacheta-NI Sacheta-NI marked this pull request as draft February 20, 2025 11:08
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.

2 participants