Skip to content

Conversation

bradleyjkemp
Copy link
Contributor

Use relative rather than absolute paths when zipping sdk artifacts to avoid including the entire directory structure in the zip.

Copy link

@improbablepiotr improbablepiotr left a comment

Choose a reason for hiding this comment

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

Looks legit

echo "--- Preparing artefacts for release"
msbuild $REPO_ROOT/apis/apis.csproj /p:Configuration=Release /p:Version=${SDK_VERSION} /t:Clean,Build -verbosity:minimal
zip -r ${ARTEFACT_DIR}/${SDK_VERSION}-net451.zip ${OUTPUT_DIR}/net451
pushd ${OUTPUT_DIR}/net451

Choose a reason for hiding this comment

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

What is the net451?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not entirely sure but it seems to be what version of .Net is being targeted (see aspnet/Announcements#98)

@bradleyjkemp bradleyjkemp merged commit a7c9d9c into master Jan 3, 2019
@bradleyjkemp bradleyjkemp deleted the sdkzip-output branch January 3, 2019 18:29
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