publish: getsentry/[email protected]

[getsentry-release]

Requested by: @jamescrosswell

Merge target: (default)

Quick links:

• View changes
• View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).

Targets

• nuget
• github
• registry

Targets marked with a checkbox have already been executed. Administrators can manually tick a checkbox to force craft to skip it.

[github-actions]

Publishing: run#11991358537

[github-actions]

Failed to publish. (run logs)

Bad branch? You can delete with ease and start over.

[bruno-garcia]

STDOUT: dotnet: Pushing Sentry.Bindings.Android.5.0.0-alpha.0.nupkg to 'https://www.nuget.org/api/v2/package'...
dotnet:   PUT https://www.nuget.org/api/v2/package/
dotnet:   BadRequest https://www.nuget.org/api/v2/package/ 1343ms
dotnet: error: Response status code does not indicate success: 400 (The package is invalid and cannot be published. The package entry for 'Sentry.Bindings.Android.targets' has a  double forward slashes in path, which is causing a file name collision during extraction.).
dotnet: 

That's odd

[bruno-garcia]

Don't think it's reentrant (nuget publish that is)

[github-actions]

Publishing: run#11991487490

[github-actions]

Failed to publish. (run logs)

Bad branch? You can delete with ease and start over.

[bruno-garcia]

dotnet: To skip already published packages, use the option --skip-duplicate

Yeah we'd need to provide this to make this re-entrant.

• feat: dotnet publish re-entrant craft#571

But fails in the UI too:

The file they say has "double forward slashes in path":

I mean it has the xmlns with http://.

<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <ItemGroup>
    <ProguardConfiguration Include="$(MSBuildThisFileDirectory)..\..\proguard.cfg" />
  </ItemGroup>
</Project>

This has been there since the first version:
https://github.com/getsentry/sentry-dotnet/pull/2450/files#diff-a00f24179d4fb2ec1e497e77c12422251f15c3c2e62c9c5155eff83f57bc606f

[bruno-garcia]

Raised here:

• "double forward slashes in path" when uploading package NuGet/Home#13957

[jamescrosswell]

STDOUT: dotnet: Pushing Sentry.Bindings.Android.5.0.0-alpha.0.nupkg to 'https://www.nuget.org/api/v2/package'...
dotnet: PUT https://www.nuget.org/api/v2/package/
dotnet: BadRequest https://www.nuget.org/api/v2/package/ 1343ms
dotnet: error: Response status code does not indicate success: 400 (The package is invalid and cannot be published. The package entry for 'Sentry.Bindings.Android.targets' has a double forward slashes in path, which is causing a file name collision during extraction.).
dotnet:

This is the diff between the v5.0.0 release and the v4.13.0 release:

We did have a short conversation about the change on line 15 when it was made.

The error message is odd though. You might get a double forward slash from the old include (not the new one) if both $TargetFramework and $TargetPlatformVersion were blank. I can't see how that would be possible though.

[bruno-garcia]

We'll make a new release with a fixed version