-
Notifications
You must be signed in to change notification settings - Fork 5
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
merges master up to the Fast OBS branch. #38
merges master up to the Fast OBS branch. #38
Conversation
…different solution files
.sln file not needed using project file within the iRacingDirector solution
collapses overlay submodules into main repo Closes MerlinCooper#36
Thanks for going through that but are you sure that the project still compiles? Did checkout the master branch and tried to build the sources but had a lot of failures. P.S.: my preferences is to finish up the alpha version before re-doing this messy solution. I even do think about just moving the plugin project into a separate solution. |
Yes. Everything builds fine. The plugin projects are simple enough. I don't think it's worth pulling them out. Not yet at least. |
I'm using VS2017 as well - weird that I did pull a fresh clone and the master branch did no longer compile. |
I think I see why. The Tests project in the iracingSDK repo isn't 4.7.2.
It needs to be updated. I guess it wasn't when it the SDK was updated to
4.7.2.
…On Wed, Jun 10, 2020 at 7:11 PM MerlinCooper ***@***.***> wrote:
I'm using VS2017 as well - weird that I did pull a fresh clone and the
master branch did no longer compile.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#38 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFE7DQCHVJH46JSU246I2LRWAHLXANCNFSM4N2X2Q2A>
.
|
This one doesn't even pass the check by azure pipeline - it's causing 18 errors mostly due to assembl references to the iRacing SDK missing. Was before down to 2 failures because the linked submodules did not have the values for the new keystrokes to pause the replay in |
If I unload the iRacingSDK.Net.Tests project everything builds fine. Azure pipelines build failure are all due that project not being 4.7.2. |
By the way I recommend upgrading the YAML Nuget to version 5.3.1 - some code in the plugins is still using versions before 3.1.1 and that creates vulnerability reports. Do not recommend to upgrade to 8.1.2 at the moment - did that one time and afterwards the app did no longer recognize the start of iRacing (let's keep that for later). P.S..: For the replayscript I'm planing to use JSON anyway because much easier to read if required. At the moment the replaysript is written in YAML and JSON format |
Updated the iRacingSDK with MerlinCooper/iRacingSDK.Net@7f651ee Did a new version of the NuGet 1.0.0.43 as well - although nothing should have changed in this part of the solution. Curiously the project wasn't in the solution file anymore - because at some point of time I removed it due to not compiling anymore after updating to YAML 5.3.1. Did not miss the project so far because it doesn't really do someting - anyway updated the code thus it compiles. |
Had another look on the "assembly reference missing" errors we are facing during building the project. Looks like there is a known bug with .NET 4.7.1 framework which can cause this failures when using package.config files instead of package-references (which was introduced with VS2017). Will try to apply the workaround for the iRacingSDK.NET project and see wheather this help. If it works we might have to migrate the iRacingReplayDirector to packagereference as well. But that would mean that VS2017 or newer is required to compile afterwards. |
Can confirm that migrating to packagereferences did resolve the issue for the iRacingSDK.NET project not being able to compile with azure pipelines sometimes. Will do the same on the master branch for the iRacingReplayDirector. |
I wonder why it's only an issue with the Tests project. Odd. |
Will merge just the changed sources into the master branch and make sure that "master" compiles under all circumstances |
@kareem613 That wasn't easy. After merging everything into the master branch it took me 2 days to get it working / compiling with AzureDev as well. But now everything compiles and is up-to-date to .NET 4.7.2 and uses PackageReferences. |
Ok. Great. Only tough (super tough) because of the submodule removal and .net version mismatched. Looks like the hard stuff is behind us. |
This merge was a bit painful because of the submodule removal. We should be good moving forward from here.
Please merge this before doing more work on the Fast branch to avoid more merge issues.
This won't be a problem moving forward with the submodules taken care of.