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

Build Tools .Net Core 3 #42

Merged
merged 10 commits into from Aug 11, 2019
Merged

Build Tools .Net Core 3 #42

merged 10 commits into from Aug 11, 2019

Conversation

HurricanKai
Copy link
Member

Summary of the PR

Updates Build Tools to .Net Core 3
Resolving any Issues

Related issues, Discord discussions, or proposals

See #41 for further information

Further Comments

@HurricanKai HurricanKai changed the title [WIP] Build Tools .Net Core 3 Build Tools .Net Core 3 Aug 11, 2019
@HurricanKai
Copy link
Member Author

Not clear why Azure Pipelines is failing. Seems to be the nuget feed?
Else ready to review.

@HurricanKai
Copy link
Member Author

There we go. Pipelines works.

@Perksey
Copy link
Member

Perksey commented Aug 11, 2019

Have you tried a full run of the BuildTools?

./generator.bat build/binder_scripts/fullrun.json

@HurricanKai
Copy link
Member Author

Yep, I did. Set it up as my Run config in Rider.

Copy link
Member

@Perksey Perksey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome stuff. Approved, but I'll probably do a bit of a clean-up later down the line.

@@ -263,22 +259,44 @@ public static Count ParseCountSignature
return new Count(valueReferenceName);
}

// Some counts are a value reference along with some sort of numerical operation - let's skip forward in
// the string until we hit the first invalid character, and see if the rest are just numbers or math ops
var dataBeforeFirstInvalidCharacter = new string(countData.TakeWhile(char.IsLetterOrDigit).ToArray());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should reinstate these 2 lines as I liked the if statements we had before. Split the parsing logic into its own function, and then do if (SyntaxFacts.IsValidIdentifier(dataBeforeFirstInvalidCharacter) && IsMath(dataAfterFirstInvalidCharater)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a small commit to do this, does that work?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes that works nicely :) open another PR as I've already merged this one :P

@Perksey Perksey merged commit 0b48e4b into dotnet:master Aug 11, 2019
This was referenced Aug 11, 2019
silkdotnet pushed a commit that referenced this pull request Apr 13, 2023
silkdotnet pushed a commit that referenced this pull request Apr 13, 2023
silkdotnet pushed a commit that referenced this pull request Apr 13, 2023
Beyley pushed a commit that referenced this pull request Apr 13, 2023
…SMP Tue Feb 28 19:41:23 UTC 2023 (#1406)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>
silkdotnet pushed a commit that referenced this pull request Apr 13, 2023
silkdotnet pushed a commit that referenced this pull request Apr 13, 2023
Beyley pushed a commit that referenced this pull request Apr 19, 2023
…SMP Tue Feb 28 19:41:23 UTC 2023 (#1406)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>
silkdotnet pushed a commit that referenced this pull request Apr 19, 2023
silkdotnet pushed a commit that referenced this pull request Apr 19, 2023
silkdotnet pushed a commit that referenced this pull request Apr 19, 2023
Beyley pushed a commit that referenced this pull request Apr 21, 2023
…SMP Tue Feb 28 19:41:23 UTC 2023 (#1414)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>
Beyley pushed a commit that referenced this pull request May 2, 2023
…SMP Tue Feb 28 19:41:23 UTC 2023 (#1406)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>
Beyley pushed a commit that referenced this pull request May 2, 2023
…SMP Tue Feb 28 19:41:23 UTC 2023 (#1414)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>
Perksey added a commit that referenced this pull request May 28, 2023
* Add SPIRV-Reflect submodule

* Add SPIRVReflect native project

* Update vkd3d shader compiler build to latest Zig

* Add CI job to build SPIRV-Reflect

* New binaries for SPIRV-Reflect on Linux 5.15.0-1035-azure #42-Ubuntu SMP Tue Feb 28 19:41:23 UTC 2023 (#1406)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* Add SPIRV-Reflect native package to solution file

* Add SPIRV-Reflect bindings

* Split spirv.h into its own project

* BuildTools: Use underscore converted trimming name when enum trimming

This attempts to fix spirv.h enums without needing manual hacks

Currently broken

* Fixed in some ways, broken in others.

* BuildTools: Dont check for duplicate old/new enum names with no-obsolete-enum control descriptor

This fixes the Dim enum from SPIRV and Gamma enum from D2D

* BuildTools: Fix LenientUnderscore with FormatR32ui

SpvImageFormatR32ui now becomes Spv_Image_Format_R32ui instead of Spv_Image_FormatR32ui

* New binaries for SPIRV-Reflect on Linux 5.15.0-1035-azure #42-Ubuntu SMP Tue Feb 28 19:41:23 UTC 2023 (#1414)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* Rename SPIRVReflect to SPIRV.Reflect, change generation prefix

This also excludes a C++ class `ShaderModule` which was eroneously generated
And renames all the structs which conflict with vulkan types after the prefix change

* Rename SPIRV class to Spv

This removes potential problems from intersecting namespace and class names

* SPIRV.Reflect: Reference base SPIRV project

* SPIRV.Reflect: Rename anonymous enums to Constants

* WebGPU: Use new enum names

* Regenerate bindings

* BuildTools: Add prefix overrides for enum trimming

* SPIRV: Specify a prefix override to fix ImageChannelOrder

* Native: Rename Silk.NET.SPIRVReflect.Native

It now reflects the new name of the source package it belongs to

* SPIRV.Reflect.Tests: Add basic InputVariables test

* New binaries for DXVK on Linux 5.15.0-1036-azure #43-Ubuntu SMP Wed Mar 29 16:11:05 UTC 2023 (#1436)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for Assimp on Microsoft Windows 10.0.20348 (#1435)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for GLFW on Microsoft Windows 10.0.20348 (#1434)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for Vulkan Loader on Microsoft Windows 10.0.20348 (#1433)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for SPIRV-Reflect on Linux 5.15.0-1036-azure #43-Ubuntu SMP Wed Mar 29 16:11:05 UTC 2023 (#1432)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* Look Gordon, a whitespace change!

* If i make another commit will it work

* Update WebGPU experiments to new naming

* New binaries for DXVK on Linux 5.15.0-1036-azure #43-Ubuntu SMP Wed Mar 29 16:11:05 UTC 2023 (#1445)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for Assimp on Microsoft Windows 10.0.20348 (#1444)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* Update GLFW binaries (#1443)

* New binaries for GLFW on Microsoft Windows 10.0.20348

* New binaries for GLFW on Darwin 21.6.0 Darwin Kernel Version 21.6.0: Thu Mar  9 20:08:59 PST 2023; root:xnu-8020.240.18.700.8~1/RELEASE_X86_64

---------

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for Vulkan Loader on Microsoft Windows 10.0.20348 (#1442)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* New binaries for SPIRV-Reflect on Linux 5.15.0-1036-azure #43-Ubuntu SMP Wed Mar 29 16:11:05 UTC 2023 (#1441)

Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>

* Add copyright header to SPIRV-Reflect test

* Attempt to bring back the 2.17 enum names as deprecated

* BuildTools changes for bringing back the 2.17 enum names as deprecated

* Regenerate bindings for efd969d

* Account for one more edge case in the deprecated name resurgence

* Pumped Up Kicks but it's not Pumped Up Kicks and I eat a shoe box

* Fix lack of obsoletion message

* Update sln

---------

Co-authored-by: silkdotnet <85832961+silkdotnet@users.noreply.github.com>
Co-authored-by: The Silk.NET Automaton <9011267+dotnet-bot@users.noreply.github.com>
Co-authored-by: Dylan Perks <dylanperks23@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants