Skip to content

Conversation

ArturWyszomirski
Copy link
Contributor

It is necessary to explicitly declare using submodels in solution. Otherwise builder goes with the error MSB3202: The project file was not found.

@jfversluis
Copy link
Owner

I don't really understand why this would be necessary? This is not using any submodules? So I don't see why this is needed? Builds on this repo have been running fine without it?

@ArturWyszomirski
Copy link
Contributor Author

Yes, sorry for not being clear. I just bumped to that problem when tried to build a solution that consist submodules and thought it would be good to share solution with others and make the pipelines more resilient. But now it turned out that even more is needed to be changed when building NuGet that uses submodules, so this PR should be further updated.

However I totally understand if you don't want to over-complicate this template, so no hard feelings if you just drop my PR :)

@ArturWyszomirski
Copy link
Contributor Author

Just to give a glimpse what are we dealing with I have updated the PR with things that need to be done in order to create a NuGet when using submodules.

As I wrote before, I totally understand if you decide not to pull this up as it might reduce readability of your template.

And just to make story complete - I developed this solution because I wanted to incorporate Plugin.Maui.Audio with silence detection (which is still waiting for merge) into Plugin.Maui.Chat.

Maybe I am missing something and there was a simpler way to do it?

@jfversluis
Copy link
Owner

Hm yeah I appreciate the effort and adding this! I don't think this will be something a lot of people will run into?

Can we maybe instead somehow add this as documentation maybe to the wiki or readme? What do you think?

@ArturWyszomirski
Copy link
Contributor Author

Maybe putting code in separate branch and adding an annotation about submodules branch in readme?

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