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

Fix target files for Tizen to use TizenTPKFiles #598

Merged
merged 1 commit into from
Aug 6, 2018

Conversation

WonyoungChoi
Copy link
Contributor

@WonyoungChoi WonyoungChoi commented Jul 30, 2018

Current temprary hacks for tizen-x86 RID are still needed because the Tizen 4.0 platform
including .NETCore 2.0 does not understand the tizen-x86 RID.

In this changes, Use TizenTPKFiles defined in Tizen.NET.Sdk instead of Link and CopyToOutputDirectory
to solve following problems.

  • The linked file (runtime/linux-x86/native/foo.so) is displayed in VS Project tree.
  • The linked file is always copied to tpkroot/bin even when RID is specified.
  • The linked file can't be excluded with the file excluding features of Tizen.NET.Sdk.

The files added to TizenTPKFiles will be copied to tpkroot by Tizen.NET.Sdk, see
https://github.com/Samsung/build-task-tizen/blob/master/src/Tizen.NET.Build.Tasks/build/Tizen.NET.Sdk.Packaging.targets

Current temprary hacks for tizen-x86 RID are still needed because the Tizen 4.0 platform
including .NETCore 2.0 does not understand the tizen-x86 RID.

In this changes, Use TizenTPKFiles defined in Tizen.NET.Sdk instead of Link and CopyToOutputDirectory
to solve following problems.
- The linked file (runtime/linux-x86/native/foo.so) is displayed in VS Project tree.
- The linked file is always copied to tpkroot/bin even when RID is specified.
- The linked file can't be excluded using the file excluding features of Tizen.NET.Sdk.
@msftclas
Copy link

msftclas commented Jul 30, 2018

CLA assistant check
All CLA requirements met.

@WonyoungChoi
Copy link
Contributor Author

/cc @vincent-kwon

@mikekinsman
Copy link
Contributor

✅ Validation status: passed

File Status Preview URL Details
binding/HarfBuzzSharp.Tizen/nuget/build/tizen40/HarfBuzzSharp.targets ✅Succeeded
binding/SkiaSharp.Tizen/nuget/build/tizen40/SkiaSharp.targets ✅Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@mattleibow mattleibow added this to the v1.60.3 milestone Aug 6, 2018
@mattleibow mattleibow merged commit 4536194 into mono:master Aug 6, 2018
@mattleibow
Copy link
Contributor

Thanks for this PR! I have merged it in and will be in the v1.60.3 release.

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.

4 participants