Skip to content

Commit 993385e

Browse files
committed
add comment as to why outdated version of protobuf-net is used
1 parent 8609da9 commit 993385e

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/Aqua.protobuf-net/Aqua.protobuf-net.csproj

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,13 @@
1111
</PropertyGroup>
1212

1313
<ItemGroup>
14+
<!--
15+
NOTE: Due to restrictions in protobuf-net v3 which prevent the use of surrogates for types part of an object hierarchy,
16+
we're forced to stick to v2 since remote.linq is using a surrogate for serializing ConstantExpression.
17+
ConstantExpression is a sub-type of Expression, hence part of a type hierarchy, and a surrogate is used to handle
18+
serialization of the value property of type System.Object. While supported in v2, this cannot be done in v3
19+
and no workaround has been found yet. Contributions are welcome ;)
20+
-->
1421
<PackageReference Include="protobuf-net" Version="2.4.8" />
1522
</ItemGroup>
1623

0 commit comments

Comments
 (0)