-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Build script updates #2589
Build script updates #2589
Conversation
…o build_script_updates
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.
Looks pretty good. Only a couple minor comments from me.
build.py
Outdated
render_path = 'core/renderers/' + args.render_name | ||
core_search_paths = calcdeps.ExpandDirectories(directories) | ||
for path in core_search_paths: | ||
#If it is the desired renderer |
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.
nit space after # here and below
tests/compile/compile.sh
Outdated
echo "Compiling Blockly..." | ||
COMPILATION_COMMAND="java -jar $COMPILER --js='$BLOCKLY_ROOT/tests/compile/main.js' \ | ||
--js='$BLOCKLY_ROOT/core/**.js' \ | ||
--js='$BLOCKLY_ROOT/tests/compile/temp_core/**.js' \ |
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.
This should reference the tempPath variable -> '$tempPath/**.js'
tests/compile/compile.sh
Outdated
fi | ||
|
||
|
||
tempPath="$BLOCKLY_ROOT/tests/compile/temp_core" |
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.
Why is this under tests? It's more common to see an output or a build directory at the root of the project.
The basics
The details
Resolves
Proposed Changes
Reason for Changes
Test Coverage
Tested on:
Additional Information