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

Added Agones Category to all UPROPERTY macro #2587

Merged

Conversation

Dinhh1
Copy link
Contributor

@Dinhh1 Dinhh1 commented May 19, 2022

What type of PR is this?

/kind feature

What this PR does / Why we need it:

Unreal currently requires all UPROPERTY to have a Category for Engine Plugins.

This change adds the "Agones" category to all UPROPERTY's exposed in
Classes.h. It allows the AgonesSDK to be added to as an Engine Plugin or
Project Plugin. The motivation behind this was so Custom Online Subsystems
can reference the Agones code properly.

Which issue(s) this PR fixes:

Closes #

Special notes for your reviewer:
Unreal Build Log:
Build started 5/18/2022 9:51:48 AM.
Logging verbosity is set to: Normal.Project "D:\build\UnrealTutorials\AgonesBuild\AgonesBuild.sln" on node 1 (Rebuild target(s)).
ValidateSolutionConfiguration:
Building solution configuration "Development Editor|Win64".
ValidateProjects:
The project "UE4" is not selected for building in solution configuration "Development Editor|Win64".
Project "D:\build\UnrealTutorials\AgonesBuild\AgonesBuild.sln" (1) is building "D:\build\UnrealTutorials\AgonesBuild\Intermediate\ProjectFiles\AgonesBuild.vcxproj" (2) on node 1 (Rebuild target(s)).
Rebuild:
D:\Unreal\UE_4.27\Engine\Build\BatchFiles\Rebuild.bat AgonesBuildEditor Win64 Development -Project="D:\build\UnrealTutorials\AgonesBuild\AgonesBuild.uproject" -WaitMutex -FromMsBuild
Cleaning AgonesBuildEditor and UnrealHeaderTool binaries...
Creating makefile for AgonesBuildEditor (no existing makefile)
Parsing headers for AgonesBuildEditor
Running UnrealHeaderTool "D:\build\UnrealTutorials\AgonesBuild\AgonesBuild.uproject" "D:\build\UnrealTutorials\AgonesBuild\Intermediate\Build\Win64\AgonesBuildEditor\Development\AgonesBuildEditor.uhtmanifest" -LogCmds="loginit warning, logexit warning, logdatabase error" -Unattended -WarningsAsErrors -abslog="C:\Users\hod1\AppData\Local\UnrealBuildTool\Log_UHT.txt" -installed
LogInit: Display: Loading text-based GConfig....
Reflection code generated for AgonesBuildEditor in 4.5467708 seconds
Building AgonesBuildEditor...
Using Visual Studio 2019 14.29.30138 toolchain (C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133) and Windows 10.0.22000.0 SDK (C:\Program Files (x86)\Windows Kits\10).
Building 11 actions with 8 processes...
[1/11] Default.rc2
[2/11] SharedPCH.Engine.ShadowErrors.cpp
[3/11] AgonesBuild.cpp
[4/11] AgonesBuild.init.gen.cpp
[5/11] AgonesBuildGameMode.cpp
[6/11] AgonesBuildGameMode.gen.cpp
[7/11] AgonesBuildCharacter.gen.cpp
[8/11] AgonesBuildCharacter.cpp
[9/11] UE4Editor-AgonesBuild.lib
Creating library D:\build\UnrealTutorials\AgonesBuild\Intermediate\Build\Win64\UE4Editor\Development\AgonesBuild\UE4Editor-AgonesBuild.lib and object D:\build\UnrealTutorials\AgonesBuild\Intermediate\Build\Win64\UE4Editor\Development\AgonesBuild\UE4Editor-AgonesBuild.exp
[10/11] UE4Editor-AgonesBuild.dll
Creating library D:\build\UnrealTutorials\AgonesBuild\Intermediate\Build\Win64\UE4Editor\Development\AgonesBuild\UE4Editor-AgonesBuild.suppressed.lib and object D:\build\UnrealTutorials\AgonesBuild\Intermediate\Build\Win64\UE4Editor\Development\AgonesBuild\UE4Editor-AgonesBuild.suppressed.exp
[11/11] AgonesBuildEditor.target
Total time in Parallel executor: 28.41 seconds
Total execution time: 35.71 seconds
Done Building Project "D:\build\UnrealTutorials\AgonesBuild\Intermediate\ProjectFiles\AgonesBuild.vcxproj" (Rebuild target(s)).
Done Building Project "D:\build\UnrealTutorials\AgonesBuild\AgonesBuild.sln" (Rebuild target(s)).

Build succeeded.
0 Warning(s)
0 Error(s)

Unreal currently requires all UPROPERTY to have a Category.

This change adds the "Agones" category to all UPROPERTY's exposed in
Classes.h. It allows the AgonesSDK to be added to as an Engine Plugin or
Project Plugin. The motivation behind this was so Custom Online Subsystems
can reference the Agones code properly.
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 6b6e82a8-104f-4c60-9df3-04faed8f7ae9

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/2587/head:pr_2587 && git checkout pr_2587
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.24.0-8b8cfed-amd64

@markmandel markmandel added kind/feature New features for Agones area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc labels May 19, 2022
@markmandel
Copy link
Member

Calling on our usual Unreal people - @highlyunavailable @domgreen -- thoughts?

@highlyunavailable
Copy link
Contributor

Good and correct. 👍 from me.

@google-oss-prow google-oss-prow bot added the lgtm label May 19, 2022
@roberthbailey roberthbailey merged commit c3dba65 into googleforgames:main May 19, 2022
@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Dinhh1, roberthbailey

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@domgreen
Copy link
Contributor

@SaitejaTamma SaitejaTamma added this to the 1.24.0 milestone Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved area/user-experience Pertaining to developers trying to use Agones, e.g. SDK, installation, etc kind/feature New features for Agones lgtm size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants