Skip to content

Add support for relative project-paths in build.ps1 - #3885

Merged
vatsan-madhavan merged 2 commits into
masterfrom
dev/vatsan/allow-relative-project-paths
Sep 6, 2019
Merged

Add support for relative project-paths in build.ps1#3885
vatsan-madhavan merged 2 commits into
masterfrom
dev/vatsan/allow-relative-project-paths

Conversation

@vatsan-madhavan

Copy link
Copy Markdown
Member

eng\common\build.ps1's -projects argument requires full-path's to projects.

This change adds support for relative paths to supported as well.

Before:
eng\common\build.ps1 -build -projects .\src\Project1.csproj // error
eng\common\build.ps1 -build -projects C:\repos\somerepo\src\Project1.csproj // ok

After:
eng\common\build.ps1 -build -projects .\src\Project1.csproj // new - ok
eng\common\build.ps1 -build -projects C:\repos\somerepo\src\Project1.csproj // ok

@vatsan-madhavan
vatsan-madhavan requested a review from tmat September 6, 2019 19:01

@chcosta chcosta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor feedback. Thanks for making this change, it fixes an annoyance that I've just lived with and stopped thinking about.

Comment thread eng/common/build.ps1 Outdated
@vatsan-madhavan
vatsan-madhavan merged commit 2d39324 into master Sep 6, 2019
@vatsan-madhavan
vatsan-madhavan deleted the dev/vatsan/allow-relative-project-paths branch September 6, 2019 20:50
@tmat

tmat commented Sep 7, 2019

Copy link
Copy Markdown
Member

Shouldn't eng/common/build.sh be updated as well?

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.

4 participants