search

LorisYounger / VPet

虚拟桌宠模拟器 一个开源的桌宠软件, 可以内置到任何WPF应用程序
Apache License 2.0
4.64k stars 465 forks source link

PR: sdk风格csproj #116

Closed MJEdwin closed 10 months ago

MJEdwin commented 10 months ago

作者你好，不好意思提前提了pr给你，希望可以review一下(pr: #115) 该pr使用了.net upgrade assistant将原有旧风格的csproj转换为新sdk style csproj，该风格csproj好处很多，我列举一下几点：

  1. 使用新的roslyn编译器而非旧的编译器，项目编译速度有效提升
  2. vs编译无关，旧风格的csproj与vs和msbuild深度绑定，新csproj则与ide无关，可以在vscode上进行开发
  3. csproj文件简洁易懂，内容可读，能有效剔除不需要的功能包，减少包体积，(pr中移除了对System.Data.DataSetExtensions的引用)
  4. 为升级.net core铺平道路，有了sdk风格csproj，可以指定多目标框架编译，直接完成nuget包兼容.net core
  5. 为未来pet跨平台铺平道路
LorisYounger commented 10 months ago

你这新风格支持分开x86/64版本引用不同包吗 我之所以用旧版本，就是为了这个功能

LorisYounger commented 10 months ago

image

MJEdwin commented 10 months ago

嗯，支持的

MJEdwin commented 10 months ago

这里可以通过condition来区分平台 IMG_2504

LorisYounger commented 10 months ago

另外还是有问题, image 虽然说能跑,但是不跑的时候似乎和xaml的连接断开了,看起来挺难受的

LorisYounger commented 10 months ago

另外还是有问题, image 虽然说能跑,但是不跑的时候似乎和xaml的连接断开了,看起来挺难受的

修好了在 https://github.com/LorisYounger/VPet/commit/64277af6abeaeefce7da975d1fe4c021d24c058a

MJEdwin commented 10 months ago

我在指定为x64运行时没有出现过这些错误信息

LorisYounger commented 10 months ago

应该是你给全部指定了x64/x86的原因 只有VPet-Simulator.Windows需要指定x64/x86, 其他的不需要

LorisYounger commented 10 months ago

没事,反正已经修好了