Skip to content

ObjectiveC bump Xcconfig 1.1 - #718

Merged
hannesa2 merged 3 commits into
masterfrom
ObjectiveC-bump-BumpXcconfig-1.1
Sep 23, 2026
Merged

hannesa2 merged 3 commits into
masterfrom
ObjectiveC-bump-BumpXcconfig-1.1

Conversation

@hannesa2

Copy link
Copy Markdown
Contributor

@hannesa2 hannesa2 added the dependencies Pull requests that update a dependency file label Sep 22, 2026
@hannesa2
hannesa2 requested a review from goneng September 22, 2026 08:42
@hannesa2
hannesa2 force-pushed the ObjectiveC-bump-BumpXcconfig-1.1 branch from df9bada to 89b2bfc Compare September 22, 2026 09:00
hannesa2 and others added 3 commits September 23, 2026 06:15
The previous fix set CODESIGN_DIGEST_ALGORITHM=sha256, which isn't a
real Xcode build setting, so it had no effect and codesign kept
defaulting to a SHA1-only digest for ad-hoc, non-hardened-runtime test
builds. Newer codesign (Xcode 26.3+, matching CI) hard-rejects that
with "signing with only SHA1 not allowed".

OTHER_CODE_SIGN_FLAGS is what actually reaches codesign's invocation.
Setting it on the command line replaces (rather than merges with) the
xcconfig's own OTHER_CODE_SIGN_FLAGS = --timestamp=none, so both flags
are repeated here to keep timestamping disabled and switch codesign to
sha256.

Reproduced locally: xcodebuild passed --digest-algorithm=sha1 and
failed with the exact CI error; overriding OTHER_CODE_SIGN_FLAGS to
include --digest-algorithm=sha256 fixed it, and `make unit-test` now
passes all 202 GitXTests.
@hannesa2
hannesa2 force-pushed the ObjectiveC-bump-BumpXcconfig-1.1 branch from 89b2bfc to bdd0190 Compare September 23, 2026 04:15
@hannesa2
hannesa2 merged commit e9c8de1 into master Sep 23, 2026
3 checks passed
@hannesa2
hannesa2 deleted the ObjectiveC-bump-BumpXcconfig-1.1 branch September 23, 2026 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants