-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Dotnet restore fails when ApplicationDisplayVersion has some values to configure Unit Test for MAUI #12859
Comments
@jfversluis - looks like this is a question about something you showed in a video from your channel. Any idea? @danielancines - can you show what exactly you tried to add that didn't work? The repro you shared seems to have a fairly standard CSPROJ file, so I'm not sure if I'm missing something in the repro? |
It looks like the version properties being set cause some extra targets to run thinking it's an application if you have set You can condition these properties so they only build when your app is building as an 'app' for instance:
|
Hi @danielancines. We have added the "s/needs-info" label to this issue, which indicates that we have an open question for you before we can take further action. This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time. |
We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process. |
This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment. If it is closed, feel free to comment when you are able to provide the additional information and we will re-investigate. |
@Redth Unfortunately
|
Following what was asked on issue #6626, I ran using Project System Tools and got some errors. Logs attached. |
Could be related to #14524 |
For now the easiest workaround is to add |
@mattleibow I have the exact same issue, and the "easiest workaround" (or the other posted above) does nothing for me, I get the exact same failures for dotnet restore Edit: After some crosschecking in the other linked issues, I think that the "fix" was just misspelled.
instead of
@jonathanpeppers and you even mention that this is the property that really matters, not ApplicationVersion ?! Edit2: After playing around with it from a dotnet new template, it seems it just purely depends on what was edited last. |
Description
Following links like https://www.youtube.com/watch?v=C9vIDLQwc7M&t=449s to configure xUnit Tests on my Maui App, I'm having problems to add net7.0 as a TargetFrameworks and add a condition to OutputType as Library to able xUnit Project test my classes.
dotnet restore fails when I add
<ApplicationDisplayVersion>1.5.0</ApplicationDisplayVersion>
1.5.0 or 1.5 for the ApplicationDisplayVersion, if the value is 1.0.0 doens't occurs. This is the error:I've been testing this for days and I figure it out that the ApplicationDisplayVersion is the problem, but I don't have ideia why.
Steps to Reproduce
<TargetFrameworks>net7.0;net7.0-android;net7.0-ios;net7.0-maccatalyst</TargetFrameworks>
<ApplicationDisplayVersion>1.0</ApplicationDisplayVersion>
and change the version to 1.5 or 1.5.0dotnet restore
project.assets.json
in obj folder will show the log error above.Link to public reproduction project repository
https://github.com/danielancines/Maui.UnitTestError
Version with bug
7.0 (current)
Last version that worked well
Unknown/Other
Affected platforms
Windows
Affected platform versions
dotnet 7.0.102 - VS Community 2022 17.4.4
Did you find any workaround?
I'm changing the version of my app using a config.json file.
Relevant log output
The text was updated successfully, but these errors were encountered: