[Content.Pipeline] Add new content project system #8830
Conversation
| private static Stream PlatformOpenStream(string safeName) | ||
| { | ||
| return Android.App.Application.Context.Assets.Open(safeName); | ||
| var cachePath = Path.Combine(CacheLocation, safeName); |
There was a problem hiding this comment.
Could this be a potential vector for hacking game content at runtime? Should there be a way to ship a game without this feature turned on? I'm pretty sure allot of people wouldn't want this in a release build.
There was a problem hiding this comment.
Hold your horses for the PR description, will talk about it in there 😄
|
So how do we keep the old pipeline "builder" and the new one from getting mixed up while we're supporting both? Seems like we have |
| using System.IO; | ||
| using System.Net.Http; | ||
| using System.Net.Sockets; |
There was a problem hiding this comment.
Not all console platforms have working networking at the C# level.
We need to put any networking into platform calls. The default implementation for PC/mobile is free to use C# networking APIs. For console it is a different story.
| PlatformInit(); | ||
| ModifiedTimes = LoadModifiedTimes(); |
There was a problem hiding this comment.
Like above... all the network stuff needs to be a platform specific implementation.
|
|
||
| private static Dictionary<string, long> ModifiedTimes { get; set; } | ||
|
|
||
| private static HttpClient Client { get; set; } |
There was a problem hiding this comment.
I'm wondering about the impact of referencing HttpClient, it makes self-contained apps substantially bigger and untrimmable.
WIP