You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by "pdxcodemonkey (GitHub)" <gi...@apache.org> on 2019/01/22 23:32:42 UTC
[GitHub] [geode-native] pdxcodemonkey opened pull request #438:
GEODE-6211: Add dotnet transaction example
Co-authored-by: Ivan Godwin <ig...@pivotal.io>
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] igodwin commented on pull request #438:
GEODE-6211: Add dotnet transaction example
Posted by "igodwin (GitHub)" <gi...@apache.org>.
@pivotal-jbarrett , it's C#. Not sure how it started.
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] pdxcodemonkey commented on pull request #438:
GEODE-6211: Add dotnet transaction example
Posted by "pdxcodemonkey (GitHub)" <gi...@apache.org>.
The configure and install targets use paths based on the cmake variable ADD_EXAMPLE_NAME, so the names have to match for now. We could extricate the two things from each other, but that's IMO work beyond the scope of this PR.
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] pivotal-jbarrett commented on pull request
#438: GEODE-6211: Add dotnet transaction example
Posted by "pivotal-jbarrett (GitHub)" <gi...@apache.org>.
Missing terminating new line.
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] pdxcodemonkey closed pull request #438:
GEODE-6211: Add dotnet transaction example
Posted by "pdxcodemonkey (GitHub)" <gi...@apache.org>.
[ pull request closed by pdxcodemonkey ]
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] pdxcodemonkey commented on pull request #438:
GEODE-6211: Add dotnet transaction example
Posted by "pdxcodemonkey (GitHub)" <gi...@apache.org>.
Turns out, you can’t have two projects in the same VS solution whose names only differ by case. We (I) adopted the convention so we could still build all examples in the same solution.
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] pivotal-jbarrett commented on pull request
#438: GEODE-6211: Add dotnet transaction example
Posted by "pivotal-jbarrett (GitHub)" <gi...@apache.org>.
What does `Cs` mean?
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode-native] pivotal-jbarrett commented on pull request
#438: GEODE-6211: Add dotnet transaction example
Posted by "pivotal-jbarrett (GitHub)" <gi...@apache.org>.
Still don't understand why that is a directory naming issue here. If VS generator has an issue then adjust the project name in the project's CMakeLists.txt.
[ Full content available at: https://github.com/apache/geode-native/pull/438 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org