-
Notifications
You must be signed in to change notification settings - Fork 97
Set AR with the archiver from cc-rs #132
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
base: main
Are you sure you want to change the base?
Conversation
|
Welcome @reneleonhardt! It looks like this is your first PR to tikv/jemallocator 🎉 |
d2ffa5b to
a2d5fb0
Compare
cc-rs offers some variables to set the archiver program (commonly ar) that should be used for a given target. This is commonly used in cross-compilation situations, to use a version of AR compatible with the target linker. Signed-off-by: roblabla <[email protected]> Signed-off-by: Rene Leonhardt <[email protected]>
069e4bd to
92499b0
Compare
Signed-off-by: Rene Leonhardt <[email protected]>
Signed-off-by: Rene Leonhardt <[email protected]>
92499b0 to
126b754
Compare
| @@ -0,0 +1,23 @@ | |||
| version: 2 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don' introduce this bot as it will spam the PR lists.
| NOBGT: ${{ matrix.nobgt }} | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just actions/checkout@v6 is fine.
cc-rs offers some variables to set the archiver program (commonly ar) that should be used for a given target. This is commonly used in cross-compilation situations, to use a version of AR compatible with the target linker.
Rebased version of #124