Page MenuHome
Differential D698

Game engine Publishing Upgrade
AbandonedPublic
Actions

Authored by Jorge Bernal (lordloki) on Aug 1 2014, 4:32 AM.

Details

Reviewers
Mitchell Stokes (moguri)
Oren Titane (Genome36)
Group Reviewers
Game Engine
Summary

Upgrading add-on to include in the long run:

  • Refreshed visuals.
  • Reorganized fields by context (initial options, assets, platforms).
  • Working directory using the "project name".
  • Versions tracking and auto incrementation of 1 unit of the revision field for easy bug fixing without worrying of game versions being overwritten.

Old hierarchy:

root - dir
project.blend
platform-archive.tar.gz
platform - dir
blenderplayer
2.71 - dir
python files

New hierarchy:

root - dir
project.blend
project name - dir
platform - dir
version-archive.tar.gz
version - dir
blenderplayer
2.71 - dir
python files

Diff Detail

Event Timeline

Oren Titane (Genome36) updated this revision to Diff 2285.Aug 1 2014, 4:32 AM
Oren Titane (Genome36) retitled this revision from to Game engine Publishing Upgrade.
Oren Titane (Genome36) updated this object.
Oren Titane (Genome36) added a reviewer: Mitchell Stokes (moguri).
Oren Titane (Genome36) added a subscriber: Oren Titane (Genome36).
Oren Titane (Genome36) updated this object.Aug 1 2014, 4:37 AM
Oren Titane (Genome36) updated this object.
Oren Titane (Genome36) updated this revision to Diff 2287.Aug 1 2014, 6:44 AM
This comment was removed by Oren Titane (Genome36).
Oren Titane (Genome36) updated this object.Aug 1 2014, 6:55 AM
Oren Titane (Genome36) updated this object.
Oren Titane (Genome36) updated this revision to Diff 2290.Aug 1 2014, 11:16 PM

Reverted style changes to the strict necessary.

Sorry moguri ... :(

Mitchell Stokes (moguri) requested changes to this revision.Aug 1 2014, 11:55 PM
Mitchell Stokes (moguri) edited edge metadata.

Could you please split this up into separate patches? This makes it easier to push individual parts as they are ready instead of being all or nothing. There are some good simple changes that can go into master quickly, but others will require more discussion. It is also good to separate style and functional changes into different patches.

This revision now requires changes to proceed.Aug 1 2014, 11:55 PM
Oren Titane (Genome36) added a comment.Aug 2 2014, 12:18 AM
This comment was removed by Oren Titane (Genome36).
game_engine_publishing.py
550

Variable works fine when the script is run inside blender but when it is imported as an add-on blender does not recognize the command "bpy.data.filepath".

608

should be changed to "Includes a MD5 checksum of all files packed with the project as JSON format"

Oren Titane (Genome36) updated this revision to Diff 2292.Aug 2 2014, 1:22 AM
Oren Titane (Genome36) updated this object.
Oren Titane (Genome36) edited edge metadata.
  • Moved panel from [ properties > window > render ] to [ properties > window > render_layer ] since when we export the game, the blenderplayer defaults to the scene being used while publishing and because the render_layer is empty while using the Game Engine
Oren Titane (Genome36) updated this revision to Diff 2293.Aug 2 2014, 1:29 AM
Oren Titane (Genome36) edited edge metadata.
  • Removed the first if statement for the "publish default platform" boolean and modified the "add new platform" button to set by default the "default platform" specifications, thus only using one WriteRuntime() instead of two via a for loop.
Oren Titane (Genome36) updated this revision to Diff 2294.Aug 2 2014, 1:33 AM
Oren Titane (Genome36) updated this object.
  • Simplified assets and platforms listing by integrating their boolean on the same label line as their name.
Oren Titane (Genome36) updated this revision to Diff 2295.Aug 2 2014, 1:34 AM
Oren Titane (Genome36) updated this object.
  • Added MD5 checksum generation for all files being created for the platforms under a JSON format.
Oren Titane (Genome36) updated this revision to Diff 2296.Aug 2 2014, 1:39 AM
Oren Titane (Genome36) updated this object.
  • Fixed bug where library folder is not created if it does not exist before downloading platforms.
Oren Titane (Genome36) added inline comments.Aug 2 2014, 1:43 AM
game_engine_publishing.py
316

forgot to delete this line since double clicking on the list lets you change the name property

Oren Titane (Genome36) updated this revision to Diff 2297.Aug 2 2014, 1:47 AM
  • With open("xxx", "xx") as file: closes automatically the file when finished, thus removing the need to close the file using file.close().
Oren Titane (Genome36) added a comment.Aug 2 2014, 1:51 AM

Can't post differentials for the Versions, "Re-skinning", and new "Hierarchy" features since they are dependent on previous differentials.

Oren Titane (Genome36) updated this object.Aug 2 2014, 1:55 AM
Inês Almeida (brita_) added a reviewer: Game Engine.Aug 4 2014, 11:07 AM
Oren Titane (Genome36) updated this revision to Diff 2325.Aug 4 2014, 10:25 PM
  • Changed the assets add class to be able to import multiple files using a file browser thanks to bpy_extras.ioutils.ImportHelper
Mitchell Stokes (moguri) added a comment.Aug 5 2014, 12:49 AM

You should assign each diff to a different Differential revision so you don't continue to override your previous revisions. If some patches are dependent on others, you can designate this with dependencies.

Oren Titane (Genome36) added a comment.Aug 7 2014, 8:25 PM

Ok Moguri, one last thing, can someone discard this revision ? Please.

Mitchell Stokes (moguri) added a comment.Aug 11 2014, 3:03 AM

I think the owner of the revision has to be the one that abandons it. Look for the option in the Action dropdown.

Jorge Bernal (lordloki) commandeered this revision.May 18 2015, 7:31 AM
Jorge Bernal (lordloki) abandoned this revision.
Jorge Bernal (lordloki) added a reviewer: Oren Titane (Genome36).

Revision Contents
Changeset List

Diff 2325

View Options

game_engine_publishing.py

Loading...