Skip to content

enable CGO on installation of tokensdk #1325

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

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

Conversation

MohammadrezaNasrabadi
Copy link

According to the issue #1273 , installation of tokensdk might be not successful. After research, I've found the CGO_ENABLED variable should be set to 1.

In addition, after merging the changes of #1315 there were some dependency errors when building the tokensdk images.

Signed-off-by: Mohammad Reza Nasrabadi <[email protected]

@MohammadrezaNasrabadi MohammadrezaNasrabadi requested a review from a team as a code owner April 14, 2025 22:18
@MohammadrezaNasrabadi MohammadrezaNasrabadi force-pushed the update-sdk-dependencies branch 2 times, most recently from b7037f2 to 9841407 Compare April 14, 2025 22:27
@C0rWin
Copy link
Contributor

C0rWin commented Apr 15, 2025

According to the issue #1273 , installation of tokensdk might be not successful. After research, I've found the CGO_ENABLED variable should be set to 1.

In addition, after merging the changes of #1315 there were some dependency errors when building the tokensdk images.

could you split those two updates into separate PRs? first of all, they are not directly related.

then, I wrote the first part, as suggested in #1273; this might be some dependency issue that needs to be localized and understood; switching to static compilation only shadows the issue and does not really solve it.

@MohammadrezaNasrabadi
Copy link
Author

MohammadrezaNasrabadi commented Apr 15, 2025

According to the issue #1273 , installation of tokensdk might be not successful. After research, I've found the CGO_ENABLED variable should be set to 1.
In addition, after merging the changes of #1315 there were some dependency errors when building the tokensdk images.

could you split those two updates into separate PRs? first of all, they are not directly related.

then, I wrote the first part, as suggested in #1273; this might be some dependency issue that needs to be localized and understood; switching to static compilation only shadows the issue and does not really solve it.

Allright. The changes related to tokensdk applications dependencies were transported to PR #1326 .

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