Page MenuHomeWildfire Games

Show an end of packaging confirmation message.
ClosedPublic

Authored by Stan on Jun 22 2018, 9:59 AM.

Details

Reviewers
vladislavbelov
elexis
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP22042: Add a confirmation message when packaging ends.
Summary

Currently there is no confirmation of the end of the packaging. This is an issue especially on windows, where the cmd is not able to grasp the debug_printf; This patch sends a final message so you don't have to rely on hacks to figure out if the process exited.

Test Plan

Package a mod, and see it prints a final message. For instance using (Linux users beware of \)

pyrogenesis.exe -mod=no-blood-and-gore-mod -archivebuild="..\data\mods\no-blood-and-gore-mod" -archivebuild-compress -archivebuild-output="no-blood-and-gore-mod.2.pyromod"

it will print

Finished packaging "no-blood-and-gore-mod.2.pyromod".

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Stan created this revision.Jun 22 2018, 9:59 AM
Stan edited the test plan for this revision. (Show Details)
Vulcan added a subscriber: Vulcan.Jun 22 2018, 10:01 AM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/differential/663/display/redirect

Stan updated this revision to Diff 6782.Jun 22 2018, 10:59 AM

Fix linux compilation.

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/664/display/redirect

elexis accepted this revision.Jan 8 2019, 12:40 AM
elexis added a subscriber: elexis.

Line itself is as correct as the other ones in this function, because the other log outputs in this function use debug_printf and string8 too.
Line in the right place, because the function contains debug_printfs already for progress output and the caller doesn't.
blablabla
Didn't test.
2019.

This revision is now accepted and ready to land.Jan 8 2019, 12:40 AM
This revision was automatically updated to reflect the committed changes.