Add build system and continuous integration (Windows, macOS, Linux) #14
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I cannot test this script, and im unwilling to merge without testing. When someone test it we should be able to merge without issues! |
|
Proof of success: https://travis-ci.org/kevinlul/ygopro/builds/547454936 Can also confirm by testing on fresh Ubuntu VM image and following wiki instructions. (This is pretty much Travis though.) Confirmed as working on fresh Windows Subsystem for Linux Ubuntu 18.04 |
kevinlul
changed the title
|
At this point, the Linux scripts and CI are working well. I will focus on the remaining platforms and patch the Linux scripts as needed. Platform scripts will be merged into this branch when they are confirmed working well. |
|
Merged prebuild and deploy system for macOS and updated Travis to match.
Issues:
Possibly remedy for Sierra/High Sierra situation: https://github.com/phracker/MacOSX-SDKs/releases |
Tested on 18.04 Bionic; explanation on wiki fmt no longer needs to be cloned locally; install via apt-get. Moved in premake5.lua so Windows keeps working for now. Added .gitignore for premake5 binary since this script drops it right in the working directory for convenience.
…ng (needs -fms-permissive at least)
- only ignore build/ - all scripts must be run from repo root, see wiki for details - install-ubuntu-apt.sh gets everything you can get from apt - install-ubuntu-src.sh builds fmt and nlohmann-json from source and installs to /usr/local instead of doing some sketchy apt repo shuffling - install-ubuntu-bin.sh downloads premake5 and irrKlang binaries into the working directory
- By default moves the release binaries, set BUILD_CONFIG=debug to choose debug - Only strips the release binary of debug symbols - Does not retrieve FONTS since we can't decide! To do later.
build-essential, cmake, curl are not needed since they are preinstalled TODO: possibly test on other GCC and Clang versions
Corresponding update for ocgcore Current build fails on linking irrlicht: https://pastebin.com/KVL2DTwu
http://irrlicht.sourceforge.net/forum/viewtopic.php?t=36933 Also seen on Koishi's premake
kevinlul
force-pushed
the
prebuild-ubuntu
branch
2 times, most recently
from
26ef111
to
1c67135
Compare
kevinlul
force-pushed
the
prebuild-ubuntu
branch
from
1c67135
to
ae21376
Compare
kevinlul
force-pushed
the
prebuild-ubuntu
branch
2 times, most recently
from
babcb75
to
0182f62
Compare
|
Pending change: revert freetype to system freetype on macOS, test behaviour on Linux and update accordingly. |
|
Also requires edo9300/ygopro-core#48 |
Removed freetype from vcpkg install Added freetype directories to gitignore
|
Changes:
To do when or after merging @edo9300 :
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Tested on 18.04 Bionic; explanation on wiki
fmt no longer needs to be cloned locally; install via apt-get. Moved in premake5.lua so Windows keeps working for now.
Added .gitignore for premake5 binary since this script drops it right in the working directory for convenience.