-
-
Notifications
You must be signed in to change notification settings - Fork 640
Refactor build scripting to support Python 3 only builds, pinned build dependency versions, optional system installed build dependencies #416
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
8490024
add support for pipenv Python dependency version pinning
chrissimpkins 384f3fa
copies build-ttf.sh as build-ttf-system.sh to use as basis for script…
chrissimpkins fe6e6ba
[build-pipenv.sh] added new shell script that supports builds of venv…
chrissimpkins e3c0e97
updated script documentation
chrissimpkins 1700e78
[build-pipenv.sh] modified if blocks in shell script
chrissimpkins 74d359e
[build-pipenv.sh] add Python interpreter version to environment report
chrissimpkins 0eb8a96
[build-ttf.sh] transitioned Python build tool calls to use of pipenv …
chrissimpkins 176a14a
[build-subsets.sh] eliminated build dependency checks, these are now …
chrissimpkins bda185c
[build-subsets.sh] transitioned to use of pipenv virtualenv installs …
chrissimpkins da20882
[Makefile] added pipenv venv setup script to the build process
chrissimpkins 7de5d65
added Python interpreter version definition to virtualenv scripting
chrissimpkins 10c5444
modified build variable name
chrissimpkins aff9f86
removed unnecessary build shell script
chrissimpkins 4d8a9a1
added new sfnt2woff-zopfli local install script
chrissimpkins 953c35b
[build-ttf.sh] updated script to support pinned and system installed …
chrissimpkins 14b1912
[Makefile] updated make targets
chrissimpkins dd71f61
[build-woff.sh] added support for pinned version and system installed…
chrissimpkins 353455d
[build-ttf.sh] fixes incorrect executable path checks with system ins…
chrissimpkins 0da551b
[build-woff.sh] refactored default build approach path test for depen…
chrissimpkins c0f9b0c
added new woff2-compress-build shell script for local builds of woff2…
chrissimpkins f1c3c53
[Makefile] updated make target shell script path for local woff2_comp…
chrissimpkins 8b05246
[build-woff2.sh] added support for pinned dependency builds and rolli…
chrissimpkins 63c6f05
[build-woff2.sh] removed unnecessary variable
chrissimpkins 83eac63
[Makefile] updated shellcheck target deps
chrissimpkins 2b1d273
[build-subsets.sh] updated shell script to support pinned build depen…
chrissimpkins 843e689
new build-archives.sh shell script
chrissimpkins 1028b7e
[Makefile] updated archives make target with new shell script path
chrissimpkins 842a21f
[Makefile] added subsets-system make target
chrissimpkins 4c48555
[Makefile] added subsets-system target to .PHONY
chrissimpkins e20d786
[fix-dsig.py] removed from __future__ imports, code formatting = pass…
chrissimpkins 0a01f04
[fix-fstype.py] source code formatting changes = passes flake8 tests
chrissimpkins 96a35cb
[woff2-compress-build.sh] convert to upstream woff2 at v1.0.2 (includ…
chrissimpkins ee180fb
[build-pipenv.sh] modified command to create the pipenv virtual envir…
chrissimpkins 8b7feda
[Makefile] added documentation and process specific target arrangement
chrissimpkins d6df277
[BUILD.md] updated build documentation with v4.x build tooling changes
chrissimpkins 1dd3664
[BUILD.md] added TOC
chrissimpkins b1ebe12
[BUILD.md] minor text revisions
chrissimpkins bc6a683
[BUILD.md] added documentation for system PATH installed build depend…
chrissimpkins 2a46cad
[woff2-compress-build.sh] modified woff2_compress to use brotli v1.0.3
chrissimpkins File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,6 +14,7 @@ | |
# The woff2 git clone directory. | ||
BUILD="$HOME" | ||
INST="$HOME/woff2" | ||
WOFF2_VERSION_TAG="v1.0.2" | ||
|
||
# woff2 executable path | ||
WOFF2_BIN="$BUILD/woff2/woff2_compress" | ||
|
@@ -29,13 +30,16 @@ echo "#####" | |
echo "git clone woff2 project" | ||
echo "#####" | ||
|
||
# clone the Source Foundry fork of the woff2 repo | ||
# contains fix for OS X build bug - https://github.com/google/woff2/issues/73 | ||
# recursive flag to clone the brotli submodule within the woff2 repo | ||
git clone --recursive https://github.com/source-foundry/woff2.git | ||
# clone the woff2 repository | ||
git clone --recursive https://github.com/google/woff2.git | ||
|
||
cd "$INST" || exit 1 | ||
|
||
# checkout desired version tag | ||
echo " " | ||
echo "Checking out woff2 version tag $WOFF2_VERSION_TAG" | ||
git checkout $WOFF2_VERSION_TAG | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. woff2 is now checked out at the shell script constant defined git version tag from the upstream Google repo instead of pulling from a version pinned downstream branch of the woff2 repository |
||
|
||
echo "#####" | ||
echo "Build woff2" | ||
echo "#####" | ||
|
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
location of new definition of woff2 project version (by version git tag)