Skip to content

[main] Fix macOS build around Git installer package#494

Merged
derrickstolee merged 8 commits intomicrosoft:mainfrom
derrickstolee:fix-mac-build
Mar 19, 2021
Merged

[main] Fix macOS build around Git installer package#494
derrickstolee merged 8 commits intomicrosoft:mainfrom
derrickstolee:fix-mac-build

Conversation

@derrickstolee
Copy link
Contributor

This should also fix the microsoft.scalar-release pipeline.

This gets around the multiple sources problem, but has the issue
that every build will download the installers fresh.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
This copies a trick from .github/workflows/continuous-integration.yml
to get the package version from the necessary place.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
@derrickstolee derrickstolee force-pushed the fix-mac-build branch 5 times, most recently from 24b06d1 to 6b660d7 Compare March 19, 2021 15:52
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
For some reason my best attempts to only pick the .pkg failed. Running
the exacxt commands in my terminal worked. The side effect here is that
we will start shipping the .dmg as well as the .pkg.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
derrickstolee added a commit that referenced this pull request Mar 19, 2021
…r package

This should also fix the microsoft.scalar-release pipeline.

See #494 for the version into `main`.

After many attempts, I now have this working with [the `microsoft.scalar-release` build in Azure Pipelines](https://dev.azure.com/mseng/Scalar/_build/results?buildId=14679562&view=results)
@derrickstolee derrickstolee merged commit 295f4bd into microsoft:main Mar 19, 2021
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