Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Generate DotNet nupkgs for ubuntu.14.04-arm and ubuntu.16.04-arm#814

Closed
jyoungyun wants to merge 1 commit into
dotnet:masterfrom
jyoungyun:arm/build
Closed

Generate DotNet nupkgs for ubuntu.14.04-arm and ubuntu.16.04-arm#814
jyoungyun wants to merge 1 commit into
dotnet:masterfrom
jyoungyun:arm/build

Conversation

@jyoungyun
Copy link
Copy Markdown

Add runtime id for ubuntu.14.04-arm and ubuntu.16.04-arm
Currently, pkg/pack.sh does not support cross build.
By adding --rid option, pack.sh supports to generate DotNet nupkgs for arm.

Related issue : #813

Add runtime id for ubuntu.14.04-arm and ubuntu.16.04-arm
Currently, pkg/pack.sh does not support cross build.
By adding --rid option, pack.sh supports to generate DotNet nupkgs for arm.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
@dnfclas
Copy link
Copy Markdown

dnfclas commented Dec 16, 2016

Hi @jyoungyun, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

@jyoungyun
Copy link
Copy Markdown
Author

@hqueue This patch generates DotNetHost nupkgs for ubuntu arm. But some parts of this patch conflict with your PRs. Let's talk about it next Monday.

@gkhanna79
Copy link
Copy Markdown
Member

@jyoungyun @hqueue A bunch of changes in this PR are the same set of changes @hqueue made in #712. Can you please merge the changes into a single PR that validates all packages for the host are being built as expected?

@gkhanna79 gkhanna79 mentioned this pull request Dec 16, 2016
4 tasks
Comment thread pkg/pack.sh
else
__targets_param="TargetsLinux=true"
init_distro_name
if [ -z $__distro_rid ]; then

This comment was marked as spam.

This comment was marked as spam.

@jyoungyun jyoungyun closed this Dec 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants