Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move SOS to SDK build #15

Merged
merged 2 commits into from
Mar 18, 2019
Merged

Conversation

swaroop-sridhar
Copy link

  • Add RestoreArcade target and some other properties to SOS NETCore project
  • Remove the hack to Skip SoS for arcade builds

* Add RestoreArcade target and some other properties to SOS NETCore project
* Remove the hack to Skip SoS for arcade builds
@swaroop-sridhar
Copy link
Author

The build phase goes through, overall build fails in packaging.

Copy link
Owner

@jashook jashook left a comment

Choose a reason for hiding this comment

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

lgtm other than the redundant rootbindir

<TargetFramework>netcoreapp1.0</TargetFramework>
<TargetFrameworkIdentifier>.NETCoreApp</TargetFrameworkIdentifier>
<PackageTargetFallback>$(PackageTargetFallback);portable-net45+win8</PackageTargetFallback>
<RootBinDir Condition="'$(__RootBinDir)'==''">$(ProjectDir)bin\</RootBinDir>
Copy link
Owner

Choose a reason for hiding this comment

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

This should already be set in dir.props

@jashook jashook merged commit 548ec39 into jashook:arcade_spc_build Mar 18, 2019
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