Skip to content

Unity(JUMBO) build support#307

Merged
jspelletier merged 21 commits intoubisoft:mainfrom
Say-Y:SharpmakePR
Nov 24, 2023
Merged

Unity(JUMBO) build support#307
jspelletier merged 21 commits intoubisoft:mainfrom
Say-Y:SharpmakePR

Conversation

@Say-Y
Copy link
Copy Markdown
Contributor

@Say-Y Say-Y commented Nov 1, 2023

I'm familiar with Jumbo build, so I think it would be nice to have this feature integrated.

#305
Apply modifications

@jspelletier
Copy link
Copy Markdown
Collaborator

Could modify one of the existing sample to add jumbo build support in one of the configuration at least?

Say-Y and others added 2 commits November 3, 2023 22:58
Co-authored-by: Jean-Sébastien Pelletier <jean-sebastien.pelletier@ubisoft.com>
Copy link
Copy Markdown
Contributor Author

@Say-Y Say-Y left a comment

Choose a reason for hiding this comment

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

Unity -> Jumbo

- Add JumboBuild Sample
Copy link
Copy Markdown
Contributor Author

@Say-Y Say-Y left a comment

Choose a reason for hiding this comment

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

  • Add JumboBuild Sample based on HelloWorld Sample
  • Add JumboBuild options

Please let me know if there is anything that needs to be corrected.

@sylvainfortin
Copy link
Copy Markdown
Collaborator

Hi @Say-Y. Since you are adding a new sample, could you add it to SamplesDef.json. Once added to this file, it will be automatically be added to CI along other samples. See https://github.com/ubisoft/Sharpmake/blob/main/docs/Samples.md.

@Say-Y
Copy link
Copy Markdown
Contributor Author

Say-Y commented Nov 6, 2023

Hi @jspelletier , I added a few header and cpp files. (test1.h, test1.cpp, test2.h, test2.cpp)

Hi @sylvainfortin , 'JumboBuild' was added by referring to the 'HelloWorld' part of the 'SamplesDef.json' file. also removed JumboBuild's 'reference' folder.
And I don't need to worry about 'UpdateSamplesOutput.bat'?

@jspelletier
Copy link
Copy Markdown
Collaborator

Hi @jspelletier , I added a few header and cpp files. (test1.h, test1.cpp, test2.h, test2.cpp)

Hi @sylvainfortin , 'JumboBuild' was added by referring to the 'HelloWorld' part of the 'SamplesDef.json' file. also removed JumboBuild's 'reference' folder. And I don't need to worry about 'UpdateSamplesOutput.bat'?

UpdateSamplesOutput.bat is used to generate the reference files

@Say-Y
Copy link
Copy Markdown
Contributor Author

Say-Y commented Nov 6, 2023

Hi @jspelletier , I added a few header and cpp files. (test1.h, test1.cpp, test2.h, test2.cpp)
Hi @sylvainfortin , 'JumboBuild' was added by referring to the 'HelloWorld' part of the 'SamplesDef.json' file. also removed JumboBuild's 'reference' folder. And I don't need to worry about 'UpdateSamplesOutput.bat'?

UpdateSamplesOutput.bat is used to generate the reference files

Should I also edit that file?

if not "%ERRORLEVEL_BACKUP%" == "0" goto error
call :UpdateRef samples JumboBuild                  JumboBuild.sharpmake.cs                    reference         JumboBuild

Say-Y and others added 4 commits November 11, 2023 06:27
Co-authored-by: Jean-Sébastien Pelletier <jean-sebastien.pelletier@ubisoft.com>
Co-authored-by: Jean-Sébastien Pelletier <jean-sebastien.pelletier@ubisoft.com>
@Say-Y
Copy link
Copy Markdown
Contributor Author

Say-Y commented Nov 10, 2023

I have applied your request.
thank you

@jspelletier
Copy link
Copy Markdown
Collaborator

jspelletier commented Nov 17, 2023

I ran the CI.
CI failed because you need to run the .bat to create the reference files. They need to be added to your PR.

@Say-Y
Copy link
Copy Markdown
Contributor Author

Say-Y commented Nov 17, 2023

I have confirmed that UpdateSamplesOutput.bat fails without the following settings

AddTargets(new Target(
        Platform.win32 | Platform.win64,
        DevEnv.vs2019,
        Optimization.Debug | Optimization.Release
));

After the change, I confirmed that regression_test.py passed.

@jspelletier jspelletier merged commit 5958ff6 into ubisoft:main Nov 24, 2023
@Say-Y Say-Y mentioned this pull request Nov 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants