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

fix warning NETSDK1194 #40

Merged
merged 1 commit into from
Dec 21, 2023
Merged

fix warning NETSDK1194 #40

merged 1 commit into from
Dec 21, 2023

Conversation

sfairat15
Copy link
Contributor

Правка workflow:

#17 [linux/arm64 builder 4/7] RUN dotnet restore ./mysql-data-mover.sln
#17 ...

#20 [linux/amd64 builder 7/7] RUN dotnet publish --no-restore --configuration Release -o output ./mysql-data-mover.sln
#20 0.382 MSBuild version 17.8.3+195e7f5a3 for .NET
#20 0.863 /usr/share/dotnet/sdk/8.0.100/Current/SolutionFile/ImportAfter/Microsoft.NET.Sdk.Solution.targets(36,5): warning NETSDK1194: The "--output" option isn't supported when building a solution. Specifying a solution-level output path results in all projects copying outputs to the same directory, which can lead to inconsistent builds. [/mysql-data-mover.sln]
#20 1.753   Dodo.DataMover -> /src/Dodo.DataMover/bin/Release/net8.0/Dodo.DataMover.dll
#20 1.888   Dodo.DataMover -> /output/
#20 DONE 2.0s

https://learn.microsoft.com/en-us/dotnet/core/compatibility/sdk/7.0/solution-level-output-no-longer-valid#single-project-multiple-targetframeworks

нужно или юзать --property:OutputPath=DESIRED_PATH (чтобы было как раньше)

или

The general recommendation is to perform the action that you previously took without the --output/-o option, and then move the output to the desired location after the command has completed.

@sfairat15 sfairat15 merged commit eeb6dda into master Dec 21, 2023
1 check passed
@sfairat15 sfairat15 deleted the sre-fix-release-workflow branch December 21, 2023 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants