Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Setting version suffix as non empty for building release package versions #41040

Merged
merged 1 commit into from
Sep 12, 2019
Merged

Setting version suffix as non empty for building release package versions #41040

merged 1 commit into from
Sep 12, 2019

Conversation

Anipik
Copy link

@Anipik Anipik commented Sep 11, 2019

Currently we are getting this error while building packages in release mode

##[error].packages\microsoft.dotnet.build.tasks.packaging\1.0.0-beta.19456.3\build\Packaging.targets(828,5): error : (NETCORE_ENGINEERING_TELEMETRY=Build) No VersionSuffix was set. Ensure it is set before targets in packaging are ran.

Corefx does not depend on version suffix property to check if any of the dependencies are stable or not. We use packageIndex.Json file for that check.
We always require version suffix to be not null.

@mmitche mmitche merged commit 144ebb6 into dotnet:release/3.0 Sep 12, 2019
@Anipik
Copy link
Author

Anipik commented Sep 12, 2019

@mmitche the netstandard change still requires some extra work. i will try to complete that tomorrow.

@mmitche
Copy link
Member

mmitche commented Sep 12, 2019

Okay. How much work? We are on a very tight timeline...we need to be done with it in by noon.

@Anipik
Copy link
Author

Anipik commented Sep 12, 2019

How much work? We are on a very tight timeline...we need to be done with it in by noon

I can get it merged by noon. Its small change, just needs to find a correct place for it.

@Anipik Anipik deleted the fixReleaseBuild branch September 12, 2019 02:01
@karelz karelz added this to the 3.0 milestone Dec 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants