Skip to content

Minor changes to support inclusion of x86 bits in the Nuget packaging pipeline#916

Merged
hariharans29 merged 19 commits intomasterfrom
x86Nuget
Apr 27, 2019
Merged

Minor changes to support inclusion of x86 bits in the Nuget packaging pipeline#916
hariharans29 merged 19 commits intomasterfrom
x86Nuget

Conversation

@hariharans29
Copy link
Member

No description provided.

@hariharans29 hariharans29 requested a review from a team as a code owner April 26, 2019 01:56
@hariharans29 hariharans29 requested a review from jignparm April 26, 2019 02:01
jignparm
jignparm previously approved these changes Apr 26, 2019
<ProjectReference Include="$(OnnxRuntimeCSharpRoot)\src\Microsoft.ML.OnnxRuntime\Microsoft.ML.OnnxRuntime.csproj" />
<PackageReference Include="CommandLineParser" Version="2.4.3" />
<PackageReference Include="Microsoft.ML.Scoring" Version="1.1.0" />
<PackageReference Include="Microsoft.ML.OnnxRuntime" Version="0.3.1" />
Copy link
Contributor

Choose a reason for hiding this comment

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

0.3.1?

Copy link
Contributor

Choose a reason for hiding this comment

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

The perftool project needs to be updated in general (i.e. read from protobuf input data files).

@hariharans29 -since we have a reference to the csproj (below), we should be able to drop this package reference now.

Include="$(OnnxRuntimeCSharpRoot)\src\Microsoft.ML.OnnxRuntime\Microsoft.ML.OnnxRuntime.csproj" />

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh that's right. So let me just drop this line in the csproj. Thanks.

@hariharans29 hariharans29 merged commit 06e0f7e into master Apr 27, 2019
@hariharans29 hariharans29 deleted the x86Nuget branch April 27, 2019 07:41
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.

3 participants