Skip to content

Document Conan ruby 3.2.2 packaging issues (#5666) - #5667

Open
anchapin wants to merge 1 commit into
developfrom
fix/5666-conan-ruby-packaging
Open

anchapin wants to merge 1 commit into
developfrom
fix/5666-conan-ruby-packaging

Conversation

@anchapin

Copy link
Copy Markdown
Collaborator

Pull Request overview

Adds a "Known issues" section to BUILDING.md documenting the eager rubygems/spec_fetcher require (securerandom 0.2.2 pre-activation breaking Bundler.setup) and the AppleDouble ._* files, and sets COPYFILE_DISABLE=1 on the python artifact tar step in python_bindings.yml.

Rubber-duck review noted the workflow change only affects the Python dist/ archive (that job builds with with_ruby=False); it is a harmless hardening, not a fix for the Ruby tarball.

Validation: docs/workflow-only; no tests run.

Pull Request Author

  • Model API Changes / Additions
  • All new and existing tests passes

Review Checklist

  • Perform a Code Review on GitHub
  • CI status: all green or justified

… files in python artifact tar

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@anchapin
anchapin requested a balanced review from Copilot October 15, 2026 13:16
@anchapin anchapin added the Pull Request - Ready for CI This pull request if finalized and is ready for continuous integration verification prior to merge. label Oct 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation matches the linked issue, and the narrowly scoped workflow hardening has no identified blocking issues.

0 open findings

What changed in this PR

Documents upstream Conan Ruby packaging issues and hardens Python artifact creation without claiming to fix the external Ruby packages.

Changes:

  • Explains the Bundler activation conflict and AppleDouble archive files.
  • Sets COPYFILE_DISABLE=1 for the Python artifact archive step.
File Description
BUILDING.md Documents known Conan Ruby issues and upstream remedies.
.github/​workflows/​python_bindings.yml Disables AppleDouble metadata generation when archiving Python artifacts.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Pull Request - Ready for CI This pull request if finalized and is ready for continuous integration verification prior to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants