Skip to content

Conversation

@ksloginov
Copy link
Contributor

Summary

Makes the ARK_setPreventWritesAfterException method publicly accessible for Swift Package Manager consumers while maintaining full backward compatibility with CocoaPods and framework builds.

Background

SPM only exposes headers in the include directory to consumers. Previously, ARK_setPreventWritesAfterException was only available in the private header, making it inaccessible to SPM users who needed to configure exception handling behavior.

Changes Made

  • Moved public API: Split NSFileHandle+ARKAdditions into public and private headers
    - Sources/CoreAardvark/include/NSFileHandle+ARKAdditions.h - Contains only ARK_setPreventWritesAfterException
    - Sources/CoreAardvark/private/NSFileHandle+ARKAdditions.h - Contains all other methods + imports public header
  • Updated umbrella header: Added NSFileHandle+ARKAdditions.h to CoreAardvark.h for all build systems

Co-Authored-By: Zachary Drayer <128868+zadr@users.noreply.github.com>
@ksloginov ksloginov requested a review from zadr October 17, 2025 14:38
@luispadron luispadron enabled auto-merge (squash) October 17, 2025 17:36
auto-merge was automatically disabled October 24, 2025 09:49

Head branch was pushed to by a user without write access

@ksloginov ksloginov force-pushed the kloginov/spm-aardvark-2 branch from f984d23 to 9a79d2d Compare October 24, 2025 11:09
…vark-2

* upstream/master:
  Fix CI and build with Xcode 16+
@luispadron luispadron merged commit 26a36de into square:master Oct 30, 2025
3 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.

3 participants