-
Notifications
You must be signed in to change notification settings - Fork 17
Attempt 2: Include mpy-cross '__init__.py' In Packages; Restructure Package Bundling #40
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
Conversation
…ure Package Bundling"" This reverts commit cd1a858.
This reverts commit e010f5a.
This reverts commit 495627b.
… backwards compatibility
…ild-tools into fix_arg_round_2
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.
Thanks for doing this! Two suggestions below.
@@ -48,7 +48,7 @@ def add_file(bundle, src_file, zip_name): | |||
return file_sector_size | |||
|
|||
|
|||
def build_bundle(libs, bundle_version, output_filename, | |||
def build_bundle(libs, bundle_version, output_filename, pkg_folder_prefix, |
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.
Let's rename this to package_folder_prefix
.
@@ -69,8 +69,8 @@ def build_bundle(libs, bundle_version, output_filename, | |||
success = True | |||
for library_path in libs: | |||
try: | |||
build.library(library_path, build_lib_dir, mpy_cross=mpy_cross, | |||
example_bundle=example_bundle) | |||
build.library(library_path, build_lib_dir, pkg_folder_prefix, |
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.
Same.
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.
Thanks for doing this! 🌮
Attempt number 2; original attempt was in #31.
This version defaults
package_folder_prefix
toadafruit_
, so that every .travis.yml doesn't need to be updated.