Skip to content

docs(macOS): enable IFD CCID with one command; keep Info.plist edits as legacy; update SIP link#134

Merged
cedelavergne-ledger merged 1 commit intoLedgerHQ:developfrom
zgv163:develop
Oct 1, 2025
Merged

docs(macOS): enable IFD CCID with one command; keep Info.plist edits as legacy; update SIP link#134
cedelavergne-ledger merged 1 commit intoLedgerHQ:developfrom
zgv163:develop

Conversation

@zgv163
Copy link
Contributor

@zgv163 zgv163 commented Sep 30, 2025

Description

This PR refreshes the macOS section of the GPG Application install docs:

Adds a recommended path to enable the Apple-bundled IFD CCID driver via:

sudo defaults write /Library/Preferences/com.apple.security.smartcard useIFDCCID -bool yes

Keeps the legacy path (editing ifd-ccid.bundle/Contents/Info.plist) for advanced users only, with a SIP warning.

Leaves Linux and Windows instructions unchanged.

Keeps the XML element details in the existing “Manual update of CCID” section to avoid duplication.

Updates the SIP documentation link to Apple’s current page:
https://developer.apple.com/documentation/security/disabling-and-enabling-system-integrity-protection

Build note (sorry!): I couldn’t generate the PDF locally with ./generate.sh on macOS — sorry about that. The LaTeX build failed due to an undefined \pandocbounded macro (introduced in recent Pandoc).

Checklist

App update process has been followed

Target branch is develop

Application version has been bumped

…y; update SIP link

- Recommend enabling Apple-bundled IFD CCID via `useIFDCCID` (no SIP needed)
- Keep Linux/Windows instructions unchanged
- Point macOS “Legacy” path to existing “Manual update of CCID” section for XML keys/values
- Update SIP documentation link to the current Apple URL
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 36.07%. Comparing base (e2e6c31) to head (2662fdd).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #134   +/-   ##
========================================
  Coverage    36.07%   36.07%           
========================================
  Files            4        4           
  Lines          255      255           
  Branches        35       35           
========================================
  Hits            92       92           
  Misses         163      163           
Flag Coverage Δ
unittests 36.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cedelavergne-ledger
Copy link
Contributor

Thank you for your contribution @zgv163 !

@cedelavergne-ledger cedelavergne-ledger merged commit e9664a2 into LedgerHQ:develop Oct 1, 2025
61 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