Page MenuHomePhabricator

[CommCoreModule] Add `#include <functional>` to `NotificationsCryptoModule`
ClosedPublic

Authored by atul on Mar 6 2024, 2:28 PM.
Tags
None
Referenced Files
F3527834: D11262.id37934.diff
Tue, Dec 24, 6:55 AM
F3527831: D11262.id37935.diff
Tue, Dec 24, 6:55 AM
F3527830: D11262.id37902.diff
Tue, Dec 24, 6:54 AM
F3527823: D11262.id.diff
Tue, Dec 24, 6:54 AM
F3527820: D11262.diff
Tue, Dec 24, 6:54 AM
Unknown Object (File)
Fri, Dec 6, 8:57 AM
Unknown Object (File)
Mon, Dec 2, 7:29 AM
Unknown Object (File)
Nov 23 2024, 11:46 PM
Subscribers
None

Details

Summary

To resolve build issue after updating Xcode from 15.2 to 15.3.

Specifically, looks like there a number of changes to C++ Standard Library: https://developer.apple.com/documentation/xcode-release-notes/xcode-15_3-release-notes#C++-Standard-Library

The relevant piece might be:

The <experimental/algorithm> and <experimental/functional> headers have been removed, since all the contents have been implemented in namespace std.

We weren't previously including <experimental/functional>, but now it requires us to include <functional> in order for it to recognize std::function.

We include <functional> elsewhere in the codebase, so maybe it was getting resolved by coincidence or something in the past? Either way, this change fixes the iOS build.


Depends on D11261

Test Plan

Before: didn't build
After: does build

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul retitled this revision from [CommCoreModule] Add `#include <string>` to `NotificationsCryptoModule` to [CommCoreModule] Add `#include <functional>` to `NotificationsCryptoModule`.Mar 6 2024, 2:30 PM
ginsu published this revision for review.Mar 6 2024, 2:36 PM

rubber stamping, one thing to note is that it looks like this diff is part of an unrelated stack

This revision is now accepted and ready to land.Mar 6 2024, 2:36 PM

This is a sign that all developers on the team should update their XCode version shortly.

This revision was landed with ongoing or failed builds.Mar 7 2024, 8:37 AM
This revision was automatically updated to reflect the committed changes.