-
Notifications
You must be signed in to change notification settings - Fork 0
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
Handle Web Site project #2
Comments
That needs more investigation: not exclude web site project but instead track it's references to other projects. |
Would it be possible for you to provide or tell me how to create such a project? (I do not have a copy of one). How would one be created "normally" (IE within Visual Studio)? I do see some references in unit tests like this one here: https://github.com/microsoft/msbuild/blob/e70a3159d64f9ed6ec3b60253ef863fa883a99b1/src/Deprecated/Engine/Shared/UnitTests/SolutionParser_Tests.cs One possible solution (if I would ignore the entire project) would be to do this: internal static IEnumerable<string> GetProjectsFromSolution(SolutionFile solution)
{
HashSet<SolutionProjectType> excludedSolutionProjectTypes =
new HashSet<SolutionProjectType>
{
SolutionProjectType.SolutionFolder,
SolutionProjectType.WebProject,
};
return
solution
.ProjectsInOrder
.Where(project => !excludedSolutionProjectTypes.Contains(project.ProjectType))
.Select(project => Path.GetFullPath(project.AbsolutePath));
} But your next comment seems to indicate that it might be nice to chase down those dependencies (which I agree with). |
ASP.NET web site project is a folder, not project file, project type Guid is E24C65DC-7377-472B-9ABA-BC803B73C61A.
VisualStudioSolutionUpdater fails handling web site solution file with a message like:
Bad Solution
C:\Dev\BlahBlah\Webs\MyWebSite.sln
ErrorAccess to the path 'C:\Dev\BlahBlah\Webs\MyWebSite' is denied.
To fix this, in SolutionUtilities.GetProjectsFromSolution() ignore project type SolutionProjectType.WebProject the same way as you do with SolutionProjectType.SolutionFolder.
The text was updated successfully, but these errors were encountered: