Skip to content

SignersNewMarketIds - #170

Closed
mihaimarcu2004 wants to merge 1 commit into
mainfrom
signers_python
Closed

SignersNewMarketIds#170
mihaimarcu2004 wants to merge 1 commit into
mainfrom
signers_python

Conversation

@mihaimarcu2004

@mihaimarcu2004 mihaimarcu2004 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

🐛 1 issue in files not directly in the diff

🐛 Intel macOS signer cannot load

On Intel macOS, __get_shared_library selects this signer, but its Mach-O CPU type is ARM64. The loader rejects it before any signer operation.

Devin Review

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔍 Signer headers remain inconsistently generated

All Linux and macOS binaries changed, but only one generated header changed. External C consumers cannot verify that unchanged declarations match each replacement.

Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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