Skip to content

Commit ec98797

Browse files
committed
Enable IsAotCompatible
1 parent 2f9b40f commit ec98797

30 files changed

+534
-128
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -132,13 +132,9 @@ jobs:
132132
exit 1
133133
134134
- name: Test (${{ matrix.docker.name }})
135-
run: >-
136-
dotnet test
137-
--configuration Release
138-
--framework ${{ matrix.dotnet.tfm }}
139-
--no-restore
140-
--no-build
141-
--logger console
135+
run: |
136+
./test/Docker.DotNet.Tests/bin/Release/${{ matrix.dotnet.tfm }}/linux-x64/publish/Docker.DotNet.Tests
137+
./test/Docker.DotNet.TestsV2/bin/Release/${{ matrix.dotnet.tfm }}/linux-x64/publish/Docker.DotNet.TestsV2
142138
env:
143139
DOCKER_HOST: ${{ matrix.docker.docker_host }}
144140
DOCKER_TLS_VERIFY: ${{ matrix.docker.tls_verify }}

Directory.Packages.props

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
<PackageVersion Include="System.Text.Json" Version="8.0.5" />
1212
<PackageVersion Include="BouncyCastle.Cryptography" Version="2.5.1" />
1313
<PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
14-
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
15-
<PackageVersion Include="xunit.runner.visualstudio" Version="2.8.2" />
16-
<PackageVersion Include="xunit" Version="2.9.2" />
14+
<PackageVersion Include="xunit.v3.mtp-v2" Version="4.0.0-pre.33" />
15+
<PackageVersion Include="xunit.v3.aot.mtp-v2" Version="4.0.0-pre.33" />
1716
</ItemGroup>
1817
</Project>

global.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"test": {
3+
"runner": "Microsoft.Testing.Platform"
4+
}
5+
}

src/Directory.Build.props

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020

2121
<PropertyGroup>
2222
<Nullable>enable</Nullable>
23+
<IsAotCompatible>true</IsAotCompatible>
2324
</PropertyGroup>
2425

2526
<PropertyGroup Condition="'$(GITHUB_ACTIONS)' == 'true'">

src/Docker.DotNet.NPipe/DockerHandlerFactory.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,10 @@ public ResolvedTransport CreateHandler(NPipeTransportOptions transportOptions, C
4545

4646
var dockerStream = new DockerPipeStream(clientStream);
4747

48-
#if NETSTANDARD
49-
var namedPipeConnectTimeout = (int)transportOptions.ConnectTimeout.TotalMilliseconds;
50-
#else
48+
#if NET
5149
var namedPipeConnectTimeout = transportOptions.ConnectTimeout;
50+
#else
51+
var namedPipeConnectTimeout = (int)transportOptions.ConnectTimeout.TotalMilliseconds;
5252
#endif
5353

5454
await clientStream.ConnectAsync(namedPipeConnectTimeout, cancellationToken)

src/Docker.DotNet.NativeHttp/DockerHandlerFactory.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public ResolvedTransport CreateHandler(NativeHttpTransportOptions transportOptio
2929
var scheme = clientOptions.AuthProvider.TlsEnabled ? Uri.UriSchemeHttps : Uri.UriSchemeHttp;
3030
var uri = new UriBuilder(clientOptions.Endpoint) { Scheme = scheme }.Uri;
3131

32-
#if NET6_0_OR_GREATER
32+
#if NET
3333
var handler = new SocketsHttpHandler
3434
{
3535
MaxConnectionsPerServer = MaxConnectionsPerServer,

src/Docker.DotNet.NativeHttp/NativeHttpTransportOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ namespace Docker.DotNet.NativeHttp;
55
/// </summary>
66
public sealed record NativeHttpTransportOptions
77
{
8-
#if NET6_0_OR_GREATER
8+
#if NET
99
/// <summary>
1010
/// Gets a callback that configures the created <see cref="SocketsHttpHandler"/> instance.
1111
/// </summary>

src/Docker.DotNet.X509/CertificateCredentials.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ public CertificateCredentials(X509Certificate2? certificate)
1515

1616
public HttpMessageHandler ConfigureHandler(HttpMessageHandler handler)
1717
{
18-
#if NET6_0_OR_GREATER
18+
#if NET
1919
if (handler is SocketsHttpHandler socketsHandler)
2020
{
2121
if (_certificate != null)

src/Docker.DotNet.X509/DockerTlsCertificates.cs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ public static X509Certificate2 LoadCertificateFromPemFiles(string certPemPath, s
6464
}
6565

6666
return certificate;
67-
#elif NET6_0_OR_GREATER
67+
#elif NET
6868
var certificate = X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
6969

7070
if (OperatingSystem.IsWindows())
@@ -75,10 +75,8 @@ public static X509Certificate2 LoadCertificateFromPemFiles(string certPemPath, s
7575
}
7676

7777
return certificate;
78-
#elif NETSTANDARD
79-
return Polyfills.X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
8078
#else
81-
return X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
79+
return Polyfills.X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
8280
#endif
8381
}
8482

@@ -141,7 +139,7 @@ public static RemoteCertificateValidationCallback CreateCertificateAuthorityVali
141139
using var chain = new X509Chain();
142140
chain.ChainPolicy.RevocationMode = X509RevocationMode.NoCheck;
143141

144-
#if NET5_0_OR_GREATER
142+
#if NET
145143
chain.ChainPolicy.TrustMode = X509ChainTrustMode.CustomRootTrust;
146144
chain.ChainPolicy.CustomTrustStore.Add(certificateAuthorityCertificate);
147145
return chain.Build(serverCertificate2);

src/Docker.DotNet/DockerClient.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -370,7 +370,7 @@ private async Task<HttpResponseMessage> PrivateMakeRequestAsync(
370370

371371
if (Timeout.InfiniteTimeSpan == timeout)
372372
{
373-
#if NET6_0_OR_GREATER
373+
#if NET
374374
return await _client.SendAsync(request, completionOption, cancellationToken)
375375
.ConfigureAwait(false);
376376
#else

0 commit comments

Comments
 (0)