Skip to content

Commit 4acdbdb

Browse files
authored
Revert "Fully qualify protoc-generated outputs (#626)" (#653)
This reverts commit bc74670.
1 parent 7a600fc commit 4acdbdb

File tree

15 files changed

+61
-145
lines changed

15 files changed

+61
-145
lines changed

Firestore/Protos/FrameworkMaker.xcodeproj/project.pbxproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@
201201
);
202202
inputPaths = (
203203
"${SRCROOT}/Pods/Target Support Files/Pods-FrameworkMaker_iOS/Pods-FrameworkMaker_iOS-resources.sh",
204-
"${PODS_CONFIGURATION_BUILD_DIR}/gRPC/gRPCCertificates.bundle",
204+
"$PODS_CONFIGURATION_BUILD_DIR/gRPC/gRPCCertificates.bundle",
205205
);
206206
name = "[CP] Copy Pods Resources";
207207
outputPaths = (

Firestore/Protos/build-protos.sh

Lines changed: 4 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,10 @@
11
#!/bin/bash
22

33
# Copyright 2017 Google
4-
#
54
# Licensed under the Apache License, Version 2.0 (the "License");
65
# you may not use this file except in compliance with the License.
76
# You may obtain a copy of the License at
8-
#
97
# http://www.apache.org/licenses/LICENSE-2.0
10-
#
118
# Unless required by applicable law or agreed to in writing, software
129
# distributed under the License is distributed on an "AS IS" BASIS,
1310
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -17,31 +14,6 @@
1714
# Run this script from firebase-ios-sdk/Firestore/Protos to regnenerate the
1815
# Objective C files from the protos.
1916

20-
set -euo pipefail
21-
22-
function StartCopyright() {
23-
local filename="$1"
24-
25-
cat > $filename <<EOF
26-
/*
27-
* Copyright 2017 Google
28-
*
29-
* Licensed under the Apache License, Version 2.0 (the "License");
30-
* you may not use this file except in compliance with the License.
31-
* You may obtain a copy of the License at
32-
*
33-
* http://www.apache.org/licenses/LICENSE-2.0
34-
*
35-
* Unless required by applicable law or agreed to in writing, software
36-
* distributed under the License is distributed on an "AS IS" BASIS,
37-
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
38-
* See the License for the specific language governing permissions and
39-
* limitations under the License.
40-
*/
41-
42-
EOF
43-
}
44-
4517
# pod update to install protoc and the gRPC plugin compiler.
4618
rm -rf Pods
4719
rm Podfile.lock
@@ -50,18 +22,10 @@ pod update
5022
# Generate the objective C files from the protos.
5123
./Pods/!ProtoCompiler/protoc --plugin=protoc-gen-grpc=Pods/\!ProtoCompiler-gRPCPlugin/grpc_objective_c_plugin -I protos --objc_out=objc --grpc_out=objc `find protos -name *.proto -print | xargs`
5224

53-
# Adjust imports in the protos to make them compile
25+
# CocoaPods does not like paths in library imports, flatten them.
5426

5527
for i in `find objc -name "*.[mh]"` ; do
56-
StartCopyright $i.tmp
57-
58-
sed '
59-
s,#import "\(.*\.pbobjc.h\),#import "Firestore/Protos/objc/\1,;
60-
s,#import "\(.*\.pbrpc.h\),#import "Firestore/Protos/objc/\1,;
61-
s, *$,,
62-
' $i >> $i.tmp
63-
64-
mv $i.tmp $i
28+
perl -i -pe 's#import ".*/#import "#' $i;
6529
done
6630

6731
# Remove the unnecessary extensionRegistry functions.
@@ -72,8 +36,5 @@ done
7236

7337
# Remove non-buildable code from Annotations.pbobjc.*.
7438

75-
StartCopyright objc/google/api/Annotations.pbobjc.m
76-
echo "static int annotations_stub __attribute__((unused,used)) = 0;" >> objc/google/api/Annotations.pbobjc.m
77-
78-
StartCopyright objc/google/api/Annotations.pbobjc.h
79-
echo "// Empty stub file" >> objc/google/api/Annotations.pbobjc.h
39+
echo "static int annotations_stub __attribute__((unused,used)) = 0;" > objc/google/api/Annotations.pbobjc.m
40+
echo "// Empty stub file" > objc/google/api/Annotations.pbobjc.h

Firestore/Protos/objc/firestore/local/MaybeDocument.pbobjc.m

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Timestamp.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
35+
#import "Timestamp.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/firestore/local/MaybeDocument.pbobjc.h"
39-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
40-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
38+
#import "MaybeDocument.pbobjc.h"
39+
#import "Document.pbobjc.h"
40+
#import "Annotations.pbobjc.h"
4141
// @@protoc_insertion_point(imports)
4242

4343
#pragma clang diagnostic push

Firestore/Protos/objc/firestore/local/Mutation.pbobjc.m

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Timestamp.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
35+
#import "Timestamp.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/firestore/local/Mutation.pbobjc.h"
39-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.h"
40-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
38+
#import "Mutation.pbobjc.h"
39+
#import "Write.pbobjc.h"
40+
#import "Annotations.pbobjc.h"
4141
// @@protoc_insertion_point(imports)
4242

4343
#pragma clang diagnostic push

Firestore/Protos/objc/firestore/local/Target.pbobjc.m

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Timestamp.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
35+
#import "Timestamp.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/firestore/local/Target.pbobjc.h"
39-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbobjc.h"
40-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
38+
#import "Target.pbobjc.h"
39+
#import "Firestore.pbobjc.h"
40+
#import "Annotations.pbobjc.h"
4141
// @@protoc_insertion_point(imports)
4242

4343
#pragma clang diagnostic push

Firestore/Protos/objc/google/api/HTTP.pbobjc.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
#import "GPBProtocolBuffers_RuntimeSupport.h"
3030
#endif
3131

32-
#import "Firestore/Protos/objc/google/api/HTTP.pbobjc.h"
32+
#import "HTTP.pbobjc.h"
3333
// @@protoc_insertion_point(imports)
3434

3535
#pragma clang diagnostic push

Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.m

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,11 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Timestamp.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
35+
#import "Timestamp.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.h"
39-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
38+
#import "Common.pbobjc.h"
39+
#import "Annotations.pbobjc.h"
4040
// @@protoc_insertion_point(imports)
4141

4242
#pragma clang diagnostic push

Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.m

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,13 @@
3333
#import <Protobuf/Struct.pbobjc.h>
3434
#import <Protobuf/Timestamp.pbobjc.h>
3535
#else
36-
#import "Firestore/Protos/objc/google/protobuf/Struct.pbobjc.h"
37-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
36+
#import "Struct.pbobjc.h"
37+
#import "Timestamp.pbobjc.h"
3838
#endif
3939

40-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
41-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
42-
#import "Firestore/Protos/objc/google/type/Latlng.pbobjc.h"
40+
#import "Document.pbobjc.h"
41+
#import "Annotations.pbobjc.h"
42+
#import "Latlng.pbobjc.h"
4343
// @@protoc_insertion_point(imports)
4444

4545
#pragma clang diagnostic push

Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbobjc.m

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -33,17 +33,17 @@
3333
#import <Protobuf/Empty.pbobjc.h>
3434
#import <Protobuf/Timestamp.pbobjc.h>
3535
#else
36-
#import "Firestore/Protos/objc/google/protobuf/Empty.pbobjc.h"
37-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
36+
#import "Empty.pbobjc.h"
37+
#import "Timestamp.pbobjc.h"
3838
#endif
3939

40-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbobjc.h"
41-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
42-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.h"
43-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
44-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Query.pbobjc.h"
45-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.h"
46-
#import "Firestore/Protos/objc/google/rpc/Status.pbobjc.h"
40+
#import "Firestore.pbobjc.h"
41+
#import "Annotations.pbobjc.h"
42+
#import "Common.pbobjc.h"
43+
#import "Document.pbobjc.h"
44+
#import "Query.pbobjc.h"
45+
#import "Write.pbobjc.h"
46+
#import "Status.pbobjc.h"
4747
// @@protoc_insertion_point(imports)
4848

4949
#pragma clang diagnostic push

Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbrpc.h

Lines changed: 14 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -14,57 +14,29 @@
1414
* limitations under the License.
1515
*/
1616

17-
#if !GPB_GRPC_FORWARD_DECLARE_MESSAGE_PROTO
1817
#import "Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbobjc.h"
19-
#endif
2018

2119
#import <ProtoRPC/ProtoService.h>
2220
#import <ProtoRPC/ProtoRPC.h>
2321
#import <RxLibrary/GRXWriteable.h>
2422
#import <RxLibrary/GRXWriter.h>
2523

26-
#if GPB_GRPC_FORWARD_DECLARE_MESSAGE_PROTO
27-
@class GCFSBatchGetDocumentsRequest;
28-
@class GCFSBatchGetDocumentsResponse;
29-
@class GCFSBeginTransactionRequest;
30-
@class GCFSBeginTransactionResponse;
31-
@class GCFSCommitRequest;
32-
@class GCFSCommitResponse;
33-
@class GCFSCreateDocumentRequest;
34-
@class GCFSDeleteDocumentRequest;
35-
@class GCFSDocument;
36-
@class GCFSGetDocumentRequest;
37-
@class GCFSListCollectionIdsRequest;
38-
@class GCFSListCollectionIdsResponse;
39-
@class GCFSListDocumentsRequest;
40-
@class GCFSListDocumentsResponse;
41-
@class GCFSListenRequest;
42-
@class GCFSListenResponse;
43-
@class GCFSRollbackRequest;
44-
@class GCFSRunQueryRequest;
45-
@class GCFSRunQueryResponse;
46-
@class GCFSUpdateDocumentRequest;
47-
@class GCFSWriteRequest;
48-
@class GCFSWriteResponse;
49-
@class GPBEmpty;
24+
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
25+
#import "Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.h"
26+
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
27+
#import "Firestore/Protos/objc/google/firestore/v1beta1/Query.pbobjc.h"
28+
#import "Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.h"
29+
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
30+
#import <Protobuf/Empty.pbobjc.h>
31+
#else
32+
#import "Empty.pbobjc.h"
33+
#endif
34+
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
35+
#import <Protobuf/Timestamp.pbobjc.h>
5036
#else
51-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
52-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.h"
53-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
54-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Query.pbobjc.h"
55-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.h"
56-
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
57-
#import <Protobuf/Empty.pbobjc.h>
58-
#else
59-
#import "Firestore/Protos/objc/google/protobuf/Empty.pbobjc.h"
60-
#endif
61-
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
62-
#import <Protobuf/Timestamp.pbobjc.h>
63-
#else
64-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
65-
#endif
66-
#import "Firestore/Protos/objc/google/rpc/Status.pbobjc.h"
37+
#import "Timestamp.pbobjc.h"
6738
#endif
39+
#import "Firestore/Protos/objc/google/rpc/Status.pbobjc.h"
6840

6941

7042
NS_ASSUME_NONNULL_BEGIN

Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbrpc.m

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -15,26 +15,9 @@
1515
*/
1616

1717
#import "Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbrpc.h"
18-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Firestore.pbobjc.h"
1918

2019
#import <ProtoRPC/ProtoRPC.h>
2120
#import <RxLibrary/GRXWriter+Immediate.h>
22-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
23-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.h"
24-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
25-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Query.pbobjc.h"
26-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.h"
27-
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
28-
#import <Protobuf/Empty.pbobjc.h>
29-
#else
30-
#import "Firestore/Protos/objc/google/protobuf/Empty.pbobjc.h"
31-
#endif
32-
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
33-
#import <Protobuf/Timestamp.pbobjc.h>
34-
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
36-
#endif
37-
#import "Firestore/Protos/objc/google/rpc/Status.pbobjc.h"
3821

3922
@implementation GCFSFirestore
4023

Firestore/Protos/objc/google/firestore/v1beta1/Query.pbobjc.m

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Wrappers.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Wrappers.pbobjc.h"
35+
#import "Wrappers.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Query.pbobjc.h"
39-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
40-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
38+
#import "Query.pbobjc.h"
39+
#import "Annotations.pbobjc.h"
40+
#import "Document.pbobjc.h"
4141
// @@protoc_insertion_point(imports)
4242

4343
#pragma clang diagnostic push

Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.m

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,13 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Timestamp.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Timestamp.pbobjc.h"
35+
#import "Timestamp.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Write.pbobjc.h"
39-
#import "Firestore/Protos/objc/google/api/Annotations.pbobjc.h"
40-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Common.pbobjc.h"
41-
#import "Firestore/Protos/objc/google/firestore/v1beta1/Document.pbobjc.h"
38+
#import "Write.pbobjc.h"
39+
#import "Annotations.pbobjc.h"
40+
#import "Common.pbobjc.h"
41+
#import "Document.pbobjc.h"
4242
// @@protoc_insertion_point(imports)
4343

4444
#pragma clang diagnostic push

Firestore/Protos/objc/google/rpc/Status.pbobjc.m

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@
3232
#if GPB_USE_PROTOBUF_FRAMEWORK_IMPORTS
3333
#import <Protobuf/Any.pbobjc.h>
3434
#else
35-
#import "Firestore/Protos/objc/google/protobuf/Any.pbobjc.h"
35+
#import "Any.pbobjc.h"
3636
#endif
3737

38-
#import "Firestore/Protos/objc/google/rpc/Status.pbobjc.h"
38+
#import "Status.pbobjc.h"
3939
// @@protoc_insertion_point(imports)
4040

4141
#pragma clang diagnostic push

Firestore/Protos/objc/google/type/Latlng.pbobjc.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
#import "GPBProtocolBuffers_RuntimeSupport.h"
3030
#endif
3131

32-
#import "Firestore/Protos/objc/google/type/Latlng.pbobjc.h"
32+
#import "Latlng.pbobjc.h"
3333
// @@protoc_insertion_point(imports)
3434

3535
#pragma clang diagnostic push

0 commit comments

Comments
 (0)