Browse Source

Fix C# source generation shell script and remove out of date notes

The conformance tests now use types which are part of src/google/protobuf, so we need to include src in the proto path.
The notes around "fix-ups" have been out of date for some time now.
Jon Skeet 9 năm trước cách đây
mục cha
commit
d49b9c81ec
2 tập tin đã thay đổi với 164 bổ sung125 xóa
  1. 1 11
      csharp/generate_protos.sh
  2. 163 114
      csharp/src/Google.Protobuf.Conformance/Conformance.cs

+ 1 - 11
csharp/generate_protos.sh

@@ -3,16 +3,6 @@
 # You first need to make sure protoc has been built (see instructions on
 # building protoc in root of this repository)
 
-# This script performs a few fix-ups as part of generation. These are:
-# - descriptor.proto is renamed to descriptor_proto_file.proto before
-#   generation, to avoid the naming collision between the class for the file
-#   descriptor and its Descriptor property
-# - This change also impacts UnittestCustomOptions, which expects to
-#   use a class of Descriptor when it's actually been renamed to
-#   DescriptorProtoFile.
-# - Issue 307 (codegen for double-nested types) breaks Unittest.proto and
-#   its lite equivalents.
-
 set -ex
 
 # cd to repository root
@@ -68,5 +58,5 @@ $PROTOC -Icsharp/protos --csharp_out=csharp/src/Google.Protobuf.Test \
 $PROTOC -Iexamples --csharp_out=csharp/src/AddressBook \
     examples/addressbook.proto
 
-$PROTOC -Iconformance --csharp_out=csharp/src/Google.Protobuf.Conformance \
+$PROTOC -Iconformance -Isrc --csharp_out=csharp/src/Google.Protobuf.Conformance \
     conformance/conformance.proto

Những thai đổi đã bị hủy bỏ vì nó quá lớn
+ 163 - 114
csharp/src/Google.Protobuf.Conformance/Conformance.cs


Một số tệp đã không được hiển thị bởi vì quá nhiều tập tin thay đổi trong này khác