Skip to content
This repository was archived by the owner on Jul 5, 2024. It is now read-only.

Add rustdocflags config for macos in Makefile #1412

Merged
merged 2 commits into from
May 16, 2023
Merged

Conversation

han0110
Copy link
Contributor

@han0110 han0110 commented May 16, 2023

Description

Add rustdocflags override in Makefile when it's macos to make doc test work.

Note that config.toml doesn't support to specify rustdocflags under [target.'cfg(target_os = "macos")'] (see rust-lang/cargo#11323) so we have to use command line override to add it manually.

Issue Link

Resolves #1410

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

On macos, run make test_doc.

@han0110 han0110 force-pushed the fix/rustdocflags-for-macos branch from 7285e42 to da66a8f Compare May 16, 2023 09:25
@han0110 han0110 force-pushed the fix/rustdocflags-for-macos branch from da66a8f to 7af9746 Compare May 16, 2023 09:26
Copy link
Contributor

@hero78119 hero78119 left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix!
Verified locally 👍

Copy link
Collaborator

@ChihChengLiang ChihChengLiang left a comment

Choose a reason for hiding this comment

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

Tested locally and it worked for me too. Thanks for the fix!

@han0110 han0110 enabled auto-merge May 16, 2023 10:29
@han0110 han0110 added this pull request to the merge queue May 16, 2023
Merged via the queue into main with commit 255a9bc May 16, 2023
@han0110 han0110 deleted the fix/rustdocflags-for-macos branch May 17, 2023 04:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Couldn't compile a test in MacOS
3 participants