Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature] Updated to OpenDDS 3.29.1 #250

Merged
merged 21 commits into from
Sep 1, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/cd_standard.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ concurrency:
cancel-in-progress: true

env:
OpenDdsVersion: "3.28.1"
OpenDdsVersion: "3.29.1"

jobs:
build_windows_native:
Expand Down
12 changes: 8 additions & 4 deletions .github/workflows/ci_standard.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
BuildConfiguration: [Release]

env:
OpenDdsVersion: "3.28.1"
OpenDdsVersion: "3.29.1"

steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -80,12 +80,16 @@ jobs:
if: steps.opendds-libraries-windows.outputs.cache-hit != 'true'
run: ${{ github.workspace }}/Build/OpenDDSharp.Build.ps1 --BuildConfiguration=${{ matrix.BuildConfiguration }} --BuildPlatform=${{ matrix.BuildPlatform }} --OpenDdsVersion=${{ env.OpenDdsVersion }} --IgnoreThirdPartySetup=False --IgnoreThirdPartyBuild=False --VisualStudioVersion=VS2022
working-directory: ${{ github.workspace }}/Build
env:
_NO_DEBUG_HEAP: 1

- name: Build & Test with Cake (Cached)
shell: pwsh
if: steps.opendds-libraries-windows.outputs.cache-hit == 'true'
run: ${{ github.workspace }}/Build/OpenDDSharp.Build.ps1 --BuildConfiguration=${{ matrix.BuildConfiguration }} --BuildPlatform=${{ matrix.BuildPlatform }} --OpenDdsVersion=${{ env.OpenDdsVersion }} --IgnoreThirdPartySetup=True --IgnoreThirdPartyBuild=True --VisualStudioVersion=VS2022
working-directory: ${{ github.workspace }}/Build
env:
_NO_DEBUG_HEAP: 1

- name: Test Report
uses: dorny/test-reporter@v1
Expand Down Expand Up @@ -114,7 +118,7 @@ jobs:
runs-on: ubuntu-20.04

env:
OpenDdsVersion: "3.28.1"
OpenDdsVersion: "3.29.1"
BuildPlatform: "x64"
BuildConfiguration: "Release"
LD_RPATH_FLAGS: "-Wl,-rpath='$ORIGIN'"
Expand Down Expand Up @@ -243,7 +247,7 @@ jobs:
runs-on: macos-13

env:
OpenDdsVersion: "3.28.1"
OpenDdsVersion: "3.29.1"
BuildPlatform: "x64"
BuildConfiguration: "Release"
LD_RPATH_FLAGS: "-Wl,-rpath,@executable_path -Wl,-rpath,@loader_path"
Expand Down Expand Up @@ -372,7 +376,7 @@ jobs:
runs-on: macos-14

env:
OpenDdsVersion: "3.28.1"
OpenDdsVersion: "3.29.1"
BuildPlatform: "ARM64"
BuildConfiguration: "Release"
LD_RPATH_FLAGS: "-Wl,-rpath,@executable_path -Wl,-rpath,@loader_path"
Expand Down
2 changes: 1 addition & 1 deletion Build/OpenDDSharp.Build/BuildContext.cs
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ namespace OpenDDSharp.Build
public class BuildContext : FrostingContext
{
#region Constants
internal const string DEFAULT_OPENDDS_VERSION = "3.28.1";
internal const string DEFAULT_OPENDDS_VERSION = "3.29.1";

private const string DEFAULT_PERL_FOLDER = "C:/Strawberry/perl/bin";

Expand Down
10 changes: 8 additions & 2 deletions Build/OpenDDSharp.Build/Tasks/BuildThirdPartyTask.cs
Original file line number Diff line number Diff line change
Expand Up @@ -57,14 +57,19 @@
platform = PlatformTarget.Win32;
}

var toolPath = @"C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Current\Bin\MSBuild.exe";

Check warning on line 60 in Build/OpenDDSharp.Build/Tasks/BuildThirdPartyTask.cs

View workflow job for this annotation

GitHub Actions / build_macos_arm64

Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)

Check warning on line 60 in Build/OpenDDSharp.Build/Tasks/BuildThirdPartyTask.cs

View workflow job for this annotation

GitHub Actions / build_macos_x64

Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)

Check warning on line 60 in Build/OpenDDSharp.Build/Tasks/BuildThirdPartyTask.cs

View workflow job for this annotation

GitHub Actions / build_linux

Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
if (platform == PlatformTarget.x64)
{
toolPath = @"C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Current\Bin\amd64\MSBuild.exe";
}
if (BuildContext.IsWindows)
{
context.MSBuild(context.OpenDdsSolutionFile, new MSBuildSettings
{
Configuration = context.BuildConfiguration,
PlatformTarget = platform,
Targets = { "Build" },
MaxCpuCount = 1,
MaxCpuCount = 0,
WorkingDirectory = Path.GetFullPath(context.DdsRoot),
EnvironmentVariables =
{
Expand All @@ -74,7 +79,8 @@
{ "MPC_ROOT", Path.GetFullPath(context.MpcRoot).TrimEnd(Path.DirectorySeparatorChar) },
},
ToolVersion = context.VisualStudioVersion,
ToolPath = @"C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Current\Bin\MSBuild.exe",
ToolPath = toolPath,
MSBuildPlatform = platform == PlatformTarget.Win32 ? MSBuildPlatform.x86 : MSBuildPlatform.x64,
});
}
else
Expand Down
2 changes: 1 addition & 1 deletion Build/OpenDDSharp.Build/Tasks/PrepareLinuxLibraries.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ namespace OpenDDSharp.Build.Tasks
[TaskName("PrepareLinuxLibraries")]
public class PrepareLinuxLibraries : FrostingTask<BuildContext>
{
private const string OPENDDS_VERSION = "3.28.1";
private const string OPENDDS_VERSION = "3.29.1";
private const string ACE_VERSION = "7.1.3";
private const string TAO_VERSION = "3.1.3";

Expand Down
44 changes: 22 additions & 22 deletions Build/OpenDDSharp.Build/Tasks/SetupThirdPartyTask.cs
Original file line number Diff line number Diff line change
Expand Up @@ -88,29 +88,27 @@ public override void Run(BuildContext context)
Git(context, $"checkout tags/{_versionTag}");

context.Log.Information("Apply required OpenDDSharp patches to OpenDDS...");
if (!Directory.Exists(BuildContext.PATCHES_FOLDER))
if (Directory.Exists(BuildContext.PATCHES_FOLDER))
{
return;
}
var patches = Directory.EnumerateFiles(BuildContext.PATCHES_FOLDER, "*.patch");
var patchPaths = patches as string[] ?? patches.ToArray();
if (!patchPaths.Any())
{
return;
}
foreach (var patchPath in patchPaths)
{
var patchDirectory = new DirectoryPath(patchPath);
if (BuildContext.IsLinux)
var patches = Directory.EnumerateFiles(BuildContext.PATCHES_FOLDER, "*.patch");
var patchPaths = patches as string[] ?? patches.ToArray();
if (patchPaths.Any())
{
var linuxPath = System.IO.Path.GetFullPath(patchDirectory.FullPath);

context.Log.Information($"Apply {linuxPath} in {context.DdsRoot}...");
Git(context, "apply --whitespace=fix --ignore-space-change --ignore-whitespace " + linuxPath);
}
else
{
Git(context, "apply " + patchDirectory.FullPath);
foreach (var patchPath in patchPaths)
{
var patchDirectory = new DirectoryPath(patchPath);
if (BuildContext.IsLinux)
{
var linuxPath = System.IO.Path.GetFullPath(patchDirectory.FullPath);

context.Log.Information($"Apply {linuxPath} in {context.DdsRoot}...");
Git(context, "apply --whitespace=fix --ignore-space-change --ignore-whitespace " + linuxPath);
}
else
{
Git(context, "apply " + patchDirectory.FullPath);
}
}
}
}

Expand Down Expand Up @@ -149,7 +147,9 @@ public override void Run(BuildContext context)
{
Version = version,
});
var arguments = " /c \"" + vsPath.FullPath + "\\Common7\\Tools\\VsDevCmd.bat\" && " + configurePath + " -v --doc-group3 --no-test";

var vcvar = $"\\VC\\Auxiliary\\Build\\vcvarsall.bat\" {context.BuildPlatform}"; // -vcvars_ver=14.36.32532
var arguments = " /c \"" + vsPath.FullPath + vcvar + " && " + configurePath + " -v --doc-group3 --no-test";
if (context.BuildConfiguration == "Release")
{
arguments += " --no-debug --optimize";
Expand Down
5 changes: 2 additions & 3 deletions Build/OpenDDSharp.Build/Tasks/TestTask.cs
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,9 @@ public override void Run(BuildContext context)
Runtime = context.RunTime,
NoBuild = true,
NoRestore = true,
Verbosity = DotNetVerbosity.Detailed,
Verbosity = DotNetVerbosity.Normal,
Configuration = context.BuildConfiguration,
Loggers = { "trx;LogFileName=test-results.trx", "console;verbosity=detailed" },
Collectors = { "XPlat Code Coverage" },
Loggers = { "trx;LogFileName=test-results.trx", "console;verbosity=normal" },
};

context.DotNetTest(solutionFullPath + "/Tests/OpenDDSharp.UnitTest/OpenDDSharp.UnitTest.csproj", dotnetTestSettings);
Expand Down
2 changes: 1 addition & 1 deletion Documentation/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ publish and subscribe model.
OpenDDS Website: [https://www.opendds.org](https://www.opendds.org)
OpenDDS Repository: [https://github.com/OpenDDS/OpenDDS](https://github.com/OpenDDS/OpenDDS)

OpenDDSharp has been compiled with OpenDDS v3.28.1
OpenDDSharp has been compiled with OpenDDS v3.29.1

| Package | NuGet |
|---------|-------|
Expand Down
5 changes: 1 addition & 4 deletions Native/CMakeListsTemplate.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,14 @@ project(${PROJECT_NAME} CXX)

find_package(OpenDDS REQUIRED)

set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

if(MSVC)
add_compile_options(/bigobj)
add_definitions(-D_CRT_SECURE_NO_WARNINGS)
add_definitions(-D_WINSOCK_DEPRECATED_NO_WARNINGS)
endif()

if(APPLE)
set(CMAKE_CXX_STANDARD 14)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)
elseif(UNIX)
Expand Down
5 changes: 1 addition & 4 deletions Native/OpenDDSWrapper/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@ cmake_minimum_required(VERSION 3.8.2)

project(OpenDDSWrapper CXX)

set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

if (APPLE)
set(CMAKE_CXX_STANDARD 14)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)
elseif (UNIX)
Expand Down
5 changes: 1 addition & 4 deletions Native/OpenDDSharp.IdlGenerator/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,11 +1,8 @@
project(openddsharp_idl CXX)
cmake_minimum_required(VERSION 3.8.2)

set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

if (APPLE)
set(CMAKE_CXX_STANDARD 14)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)
endif ()
Expand Down
70 changes: 0 additions & 70 deletions Patches/RtpsDiscoveryConfig.patch

This file was deleted.

22 changes: 0 additions & 22 deletions Patches/configure.patch

This file was deleted.

2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ OpenDDS Website: [https://www.opendds.org](https://www.opendds.org)
OpenDDS Repository: [https://github.com/OpenDDS/OpenDDS](https://github.com/OpenDDS/OpenDDS)
OpenDDS Documentation: [https://opendds.readthedocs.io](https://opendds.readthedocs.io/en/latest/)

OpenDDSharp has been compiled with OpenDDS v3.28.1
OpenDDSharp has been compiled with OpenDDS v3.29.1

| Package | NuGet |
|----------------------------------------------------------|------------------------------------------------------------------|
Expand Down
16 changes: 14 additions & 2 deletions Tests.runsettings
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,25 @@
<DataCollector friendlyName="XPlat code coverage">
<Configuration>
<Format>cobertura,lcov</Format>
<Include>[OpenDDSharp*]OpenDDSharp.*</Include>
<Exclude>[TestIdl]*,[*]OpenDDSharp.BuildTasks[*],[*]OpenDDSharp.UnitTest[*],[TestSupportProcessCore]*,[*]BenchmarkPerformance[*],[*]OpenDDSharp.Doc[*],[*]OpenDDSharp.Marshaller[*]</Exclude>
<Include>[OpenDDSharp]*</Include>
<Exclude>[TestIdlJson]*,[TestIdlCdr]*,[OpenDDSharp.BuildTasks]*,[OpenDDSharp.Marshaller]*,[OpenDDSharp.Native]*,[OpenDDSharp.Templates]*,[BenchmarkPerformance]*,[OpenDDSharp.UnitTest]*,[TestSupportProcessCore]*,[OpenDDSharp.Doc]*</Exclude>
<ExcludeByAttribute>GeneratedCodeAttribute,ExcludeFromCodeCoverageAttribute</ExcludeByAttribute>
<SingleHit>false</SingleHit>
<UseSourceLink>true</UseSourceLink>
<IncludeTestAssembly>true</IncludeTestAssembly>
<SkipAutoProps>true</SkipAutoProps>
<DeterministicReport>false</DeterministicReport>
<ExcludeAssembliesWithoutSources>MissingAll,MissingAny,None</ExcludeAssembliesWithoutSources>
</Configuration>
</DataCollector>
</DataCollectors>
</DataCollectionRunSettings>
<InProcDataCollectionRunSettings>
<InProcDataCollectors>
<InProcDataCollector assemblyQualifiedName="Coverlet.Collector.DataCollection.CoverletInProcDataCollector, coverlet.collector, Version=6.0.2, Culture=neutral, PublicKeyToken=null"
friendlyName="XPlat Code Coverage"
enabled="True"
codebase="coverlet.collector.dll" />
</InProcDataCollectors>
</InProcDataCollectionRunSettings>
</RunSettings>
4 changes: 2 additions & 2 deletions Tests/TestIdlCdr/TestIdlCdr.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -192,11 +192,11 @@
</Target>
<Target Name="OpenDDSharpCmakeOpenddsOSX64" BeforeTargets="PreBuildEvent" Condition="'$(IsOSX)'=='true' And '$(IsX64)'=='true'" Inputs="@(IdlFiles)" Outputs="@(IdlFiles->'%(RootDir)%(Directory)../$(IntermediateOutputPath)NativeProject/CMakeFiles/CMakeOutput.log')">
<Message Text="Create native project with cmake..." Importance="High" />
<Exec ToolExe="sh" Command="cmake -DCMAKE_CXX_STANDARD=17 -DCMAKE_APPLE_SILICON_PROCESSOR=x86_64 -DCMAKE_PREFIX_PATH=$(DDS_ROOT) -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -H$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject')) -B$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" WorkingDirectory="$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" IgnoreExitCode="false" ContinueOnError="false" />
<Exec ToolExe="sh" Command="cmake -DCMAKE_CXX_STANDARD=14 -DCMAKE_APPLE_SILICON_PROCESSOR=x86_64 -DCMAKE_PREFIX_PATH=$(DDS_ROOT) -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -H$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject')) -B$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" WorkingDirectory="$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" IgnoreExitCode="false" ContinueOnError="false" />
</Target>
<Target Name="OpenDDSharpCmakeOpenddsOSARM64" BeforeTargets="PreBuildEvent" Condition="'$(IsOSX)'=='true' And '$(IsARM64)'=='true'" Inputs="@(IdlFiles)" Outputs="@(IdlFiles->'%(RootDir)%(Directory)../$(IntermediateOutputPath)NativeProject/CMakeFiles/CMakeOutput.log')">
<Message Text="Create native project with cmake..." Importance="High" />
<Exec ToolExe="sh" Command="cmake -DCMAKE_CXX_STANDARD=17 -DCMAKE_APPLE_SILICON_PROCESSOR=arm64 -DCMAKE_PREFIX_PATH=$(DDS_ROOT) -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -H$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject')) -B$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" WorkingDirectory="$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" IgnoreExitCode="false" ContinueOnError="false" />
<Exec ToolExe="sh" Command="cmake -DCMAKE_CXX_STANDARD=14 -DCMAKE_APPLE_SILICON_PROCESSOR=arm64 -DCMAKE_PREFIX_PATH=$(DDS_ROOT) -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -H$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject')) -B$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" WorkingDirectory="$([System.IO.Path]::GetFullPath('$(IntermediateOutputPath)NativeProject'))" IgnoreExitCode="false" ContinueOnError="false" />
</Target>

<!--Build cmake-->
Expand Down
Loading
Loading