-
-
Notifications
You must be signed in to change notification settings - Fork 3
Add Jenkinsfile, refactor docs, move iOS-project to ios/ #51
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| @Library('pipeline-library') _ | ||
|
|
||
| buildModule { | ||
| // defaults: | ||
| //nodeVersion = '6.11.4' // Must have version set up on Jenkins master before it can be changed | ||
| //sdkVersion = '6.2.2.GA' | ||
| //androidAPILevel = '25' // if changed, must install on build nodes | ||
| } | ||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -62,9 +62,9 @@ def main(args): | |
|
|
||
| if cmd == 'build': | ||
| packages = [] | ||
| if os.path.exists('iphone'): | ||
| create_module('iphone', 'appc ti build -p ios') | ||
| packages.append('iphone') | ||
| if os.path.exists('ios'): | ||
|
Contributor
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. This change is right, but really this file should go away entirely if we're using the appc cli to build now.
Contributor
Author
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. Unfortunatley we cannot at this point, as the appc-cli is not able to package cross-platform modules into one package, so far. I'd love to get rid of all of them. Maybe we could start by removing the platform-specific build.py in ios/. |
||
| create_module('ios', 'appc ti build -p ios') | ||
| packages.append('ios') | ||
|
|
||
| if os.path.exists('android'): | ||
| create_module('android', 'appc ti build -p android') | ||
|
|
@@ -74,8 +74,8 @@ def main(args): | |
| fork('.', packages_cmd, False) | ||
|
|
||
| elif cmd == 'clean': | ||
| if os.path.exists('iphone'): | ||
| clean_build_module('iphone') | ||
| if os.path.exists('ios'): | ||
| clean_build_module('ios') | ||
|
|
||
| if os.path.exists('android'): | ||
| clean_ant_module('android') | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| {"exports":[]} | ||
|
Contributor
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. Doesn't this file get generated by the build? |
||
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.
FYI, these actually are not the default values used. We default to node 4.7.3, sdk 6.0.3.GA, android 23, build-tools 23.0.3, ndk r12b. See https://github.com/appcelerator/pipeline-library/blob/master/vars/buildModule.groovy#L126-L151