Skip to content

Comments

fix link order#2

Open
jiacai2050 wants to merge 2 commits intoGreptimeTeam:mainfrom
jiacai2050:fix-compile
Open

fix link order#2
jiacai2050 wants to merge 2 commits intoGreptimeTeam:mainfrom
jiacai2050:fix-compile

Conversation

@jiacai2050
Copy link
Contributor

From https://doc.rust-lang.org/cargo/reference/build-scripts.html#outputs-of-the-build-script

the order of arguments passed to rustc may affect the order of arguments passed to the linker. Therefore, you will want to pay attention to the order of the build script’s instructions. For example, if object foo needs to link against library bar, you may want to make sure that library bar’s cargo::rustc-link-lib instruction appears after instructions to link object foo.

@jiacai2050
Copy link
Contributor Author

@killme2008 killme2008 requested a review from zhongzc November 7, 2024 02:44
@zhongzc zhongzc added this pull request to the merge queue Nov 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to Branch Protection failures Nov 7, 2024
You're not authorized to push to this branch. Visit "About protected branches" for more information.
@killme2008 killme2008 added this pull request to the merge queue Dec 5, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to Branch Protection failures Dec 5, 2025
You're not authorized to push to this branch. Visit "About protected branches" for more information.
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.

1 participant