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

'System.Text.Json.ReflectionMemberAccessor' could not be found #4032

Open
myokeeh opened this issue May 14, 2021 · 8 comments · May be fixed by #4037
Open

'System.Text.Json.ReflectionMemberAccessor' could not be found #4032

myokeeh opened this issue May 14, 2021 · 8 comments · May be fixed by #4037
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior In-PR 🚀 need more info 📌

Comments

@myokeeh
Copy link

myokeeh commented May 14, 2021

Describe the bug

Projects updated to latest 7.0.2 version now building with this warning:

Embedded manifest in Microsoft.Toolkit.Uwp.UI.Controls.Media: Microsoft.Toolkit.Uwp.UI.Controls.Media.Properties.Microsoft.Toolkit.Uwp.UI.Controls.rd.xml(5): warning : ILTransform : warning ILT0027: Type 'System.Text.Json.ReflectionMemberAccessor' could not be found.

Not seen on version 7.0.0 or 7.0.1

Steps to Reproduce

Steps to reproduce the behavior:

  1. Update to 7.0.2

Environment

NuGet Package(s):  7.0.2

Package Version(s): 

Windows 10 Build Number:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] May 2019 Update (18362)
- [ ] May 2020 Update (19041)
- [x] Insider Build (build number: 19043)

App min and target version:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] May 2019 Update (18362)
- [x] May 2020 Update (19041)
- [x] Insider Build (19043)

Device form factor:
- [x] Desktop
- [ ] Xbox
- [ ] Surface Hub
- [ ] IoT

Visual Studio 
- [ ] 2017 (version: )
- [x] 2019 (version: 16.9.5) 
- [ ] 2019 Preview (version: )

@myokeeh myokeeh added the bug 🐛 An unexpected issue that highlights incorrect behavior label May 14, 2021
@ghost ghost added the needs triage 🔍 label May 14, 2021
@ghost
Copy link

ghost commented May 14, 2021

Hello myokeeh, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

@Kyaa-dost
Copy link
Contributor

@myokeeh can you try recreating your project? Also, please specify the steps you are taking from the beginning.

@michael-hawker
Copy link
Member

@azchohfi I had moved the System.Text.Json version up in our last patch here: f08a351

I didn't notice any issues in the smoke tests or our unit tests around this. Thoughts?

@azchohfi
Copy link
Contributor

That file has changed, and, in theory, to fix the issue we were working around with out rd.xml file:
dotnet/runtime@c73ef49#diff-f5720afdbbfb4d6676f9ae0daad63b3ddd84a818803f00c2566cdc4c9f5a0ac3
Maybe we just don't need that line anymore? Not that it matters much, but the rd.xml files do not seem to match the project name, so it took me a while to figure that this was the media one. I would open a separate issue to rename these files to match their projects.

@myokeeh
Copy link
Author

myokeeh commented May 18, 2021

@Kyaa-dost, do you still need more info given the last two comments?

michael-hawker added a commit that referenced this issue May 18, 2021
@michael-hawker
Copy link
Member

michael-hawker commented May 18, 2021

@myokeeh I've tried removing the line that @azchohfi tested here: #4037

When it builds it should provide a NuGet package you can try by following our preview package steps in our wiki here - would you mind trying that out and seeing if it resolves your issue? (If you can check your project runs in Release as well, that'd be great.)

Edit: Should be 7.0.0-build.1191+253df42f96

@Nirmal4G
Copy link
Contributor

Nirmal4G commented May 19, 2021

@azchohfi

I would open a separate issue to rename these files to match their projects.

I also did hit a different issue that prompted me to check and rename these files to match the library name. The changes are already in my PR #3893.

@michael-hawker michael-hawker linked a pull request May 20, 2021 that will close this issue
8 tasks
@ghost ghost added the In-PR 🚀 label May 20, 2021
michael-hawker added a commit that referenced this issue Aug 12, 2021
michael-hawker added a commit that referenced this issue Sep 13, 2021
@yoshiask
Copy link
Contributor

I just ran into this issue today, even though I updated to 7.0.2 three months ago and successfully built Release packages multiple times since then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior In-PR 🚀 need more info 📌
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants