Skip to content

feat: add features option to build script#2834

Merged
overtrue merged 4 commits into
rustfs:mainfrom
giter:main
May 6, 2026
Merged

feat: add features option to build script#2834
overtrue merged 4 commits into
rustfs:mainfrom
giter:main

Conversation

@giter
Copy link
Copy Markdown
Contributor

@giter giter commented May 6, 2026

e.g. build-rustfs.sh --features webdav

Related Issues

Summary of Changes

Verification

Impact

Additional Notes


Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md). If this is your first contribution, review the CLA document and sign it by commenting I have read and agree to the CLA. on the PR.

e.g.  build-rustfs.sh --features webdav

Signed-off-by: giter <giter@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

CLA requirements are satisfied for this pull request.

@houseme houseme requested review from Copilot and overtrue May 6, 2026 14:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the top-level build-rustfs.sh build helper to allow callers (e.g., CI or local developers) to enable Cargo features via a new --features / -f CLI option when building the rustfs binaries.

Changes:

  • Added a FEATURES option and surfaced it in --help output and example invocations.
  • Appended --features <value> to the constructed Cargo build command when provided.
  • Printed selected features in the build configuration summary output.

Comment thread build-rustfs.sh
Comment on lines +447 to +449
if [ -n "$FEATURES" ]; then
build_cmd+=" --features $FEATURES"
fi
Comment thread build-rustfs.sh
overtrue and others added 2 commits May 6, 2026 22:56
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 安正超 <anzhengchao@gmail.com>
@overtrue overtrue enabled auto-merge May 6, 2026 14:56
@overtrue overtrue added this pull request to the merge queue May 6, 2026
Merged via the queue into rustfs:main with commit 04712fb May 6, 2026
8 checks passed
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.

4 participants