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

Ansible.Basic.cs - Fix compile error on PS 7.3.x #79848

Merged
merged 1 commit into from Jan 30, 2023

Conversation

mattclay
Copy link
Member

SUMMARY

Ansible.Basic.cs - Fix compile error on PS 7.3.x

This issue was discovered while trying to upgrade the base test container: #79846

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

lib/ansible/module_utils/csharp/Ansible.Basic.cs

ADDITIONAL INFORMATION

The reported error is:

Line |
 247 |  …             throw [InvalidOperationException]"Failed to compile C# co …
     |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Failed to compile C# code:
error CS1701: Assuming assembly reference 'System.Runtime, Version=6.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' used by 'Newtonsoft.Json' matches identity 'System.Runtime, Version=7.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' of 'System.Runtime', you may need to supply runtime policy

Which comes from:

throw [InvalidOperationException]"Failed to compile C# code:`r`n$($errors -join "`r`n")"

@ansibot ansibot added affects_2.15 bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. windows Windows community labels Jan 30, 2023
@jborean93 jborean93 merged commit 83fe75d into ansible:devel Jan 30, 2023
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label Jan 30, 2023
@mattclay mattclay deleted the fix-powershell branch January 30, 2023 20:21
mattclay added a commit to mattclay/ansible that referenced this pull request Jan 30, 2023
…le#79848)

(cherry picked from commit 83fe75d)

Co-authored-by: Matt Clay <matt@mystile.com>
mattclay added a commit to mattclay/ansible that referenced this pull request Jan 30, 2023
…le#79848)

(cherry picked from commit 83fe75d)

Co-authored-by: Matt Clay <matt@mystile.com>
mattclay added a commit to mattclay/ansible that referenced this pull request Jan 30, 2023
…le#79848)

(cherry picked from commit 83fe75d)

Co-authored-by: Matt Clay <matt@mystile.com>
mattclay added a commit that referenced this pull request Feb 3, 2023
mattclay added a commit that referenced this pull request Feb 3, 2023
mattclay added a commit that referenced this pull request Feb 3, 2023
@ansible ansible locked and limited conversation to collaborators Feb 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.15 bug This issue/PR relates to a bug. windows Windows community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants