Skip to content

Commit b232cfe

Browse files
authored
Cause Firestore build error if non-Android headers get included in Android builds (#580)
1 parent 3ddd997 commit b232cfe

23 files changed

+92
-0
lines changed

firestore/src/main/collection_reference_main.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@
1010
#include "firestore/src/main/promise_factory_main.h"
1111
#include "firestore/src/main/query_main.h"
1212

13+
#if defined(__ANDROID__)
14+
#error "This header should not be used on Android."
15+
#endif
16+
1317
namespace firebase {
1418
namespace firestore {
1519

firestore/src/main/converter_main.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,10 @@
2929
#include "firestore/src/main/transaction_main.h"
3030
#include "firestore/src/main/write_batch_main.h"
3131

32+
#if defined(__ANDROID__)
33+
#error "This header should not be used on Android."
34+
#endif
35+
3236
namespace firebase {
3337
namespace firestore {
3438

firestore/src/main/create_credentials_provider.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@
77

88
#include "Firestore/core/src/auth/credentials_provider.h"
99

10+
#if defined(__ANDROID__)
11+
#error "This header should not be used on Android."
12+
#endif
13+
1014
namespace firebase {
1115

1216
class App;

firestore/src/main/create_firebase_metadata_provider.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@
77

88
#include "Firestore/core/src/remote/firebase_metadata_provider.h"
99

10+
#if defined(__ANDROID__)
11+
#error "This header should not be used on Android."
12+
#endif
13+
1014
namespace firebase {
1115

1216
class App;

firestore/src/main/credentials_provider_desktop.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@
1414
#include "app/src/include/firebase/app.h"
1515
#include "app/src/include/firebase/future.h"
1616

17+
#if defined(__ANDROID__)
18+
#error "This header should not be used on Android."
19+
#endif
20+
1721
namespace firebase {
1822
namespace firestore {
1923

firestore/src/main/document_change_main.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99
#include "firestore/src/include/firebase/firestore/document_change.h"
1010
#include "firestore/src/main/firestore_main.h"
1111

12+
#if defined(__ANDROID__)
13+
#error "This header should not be used on Android."
14+
#endif
15+
1216
namespace firebase {
1317
namespace firestore {
1418

firestore/src/main/document_reference_main.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@
1515
#include "firestore/src/main/promise_factory_main.h"
1616
#include "firestore/src/main/user_data_converter_main.h"
1717

18+
#if defined(__ANDROID__)
19+
#error "This header should not be used on Android."
20+
#endif
21+
1822
namespace firebase {
1923
namespace firestore {
2024

firestore/src/main/document_snapshot_main.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@
1616
#include "firestore/src/include/firebase/firestore/snapshot_metadata.h"
1717
#include "firestore/src/main/firestore_main.h"
1818

19+
#if defined(__ANDROID__)
20+
#error "This header should not be used on Android."
21+
#endif
22+
1923
namespace firebase {
2024
namespace firestore {
2125

firestore/src/main/field_value_main.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@
1616
#include "firestore/src/include/firebase/firestore/field_value.h"
1717
#include "firestore/src/main/firestore_main.h"
1818

19+
#if defined(__ANDROID__)
20+
#error "This header should not be used on Android."
21+
#endif
22+
1923
namespace firebase {
2024
namespace firestore {
2125

firestore/src/main/firebase_metadata_provider_desktop.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55

66
#include "Firestore/core/src/remote/firebase_metadata_provider.h"
77

8+
#if defined(__ANDROID__)
9+
#error "This header should not be used on Android."
10+
#endif
11+
812
namespace firebase {
913

1014
namespace firestore {

0 commit comments

Comments
 (0)