Skip to content
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

Initial modifications to SConstruct to add blackberry #3

Merged
merged 2 commits into from
Mar 22, 2012

Conversation

jpl-mac
Copy link

@jpl-mac jpl-mac commented Mar 16, 2012

CHANGELOG

  • Added blackberry target to the SConstruct file
  • Removed unused variables

Note: as git doesn't seem to track empty dirs, the script will complain
that blackberry dir doesn't exist until it's created

TESTS

  • Verified that the mobile sdk still builds correctly for Android
  • Verified that "scons blackberry=1" Prints the Building for Blackberry
    message and packages and returns
  • Verified that scons without params builds both android and blackberry
    (the full clause)

CHANGELOG
- Added blackberry target to the SConstruct file
- Removed unused variables

Note: as git doesn't seem to track empty dirs, the script will complain
that blackberry dir doesn't exist until it's created

TESTS
- Verified that the mobile sdk still builds correctly for Android
- Verified that "scons blackberry=1" Prints the Building for Blackberry
  message and packages and returns
- Verified that scons without params builds both android and blackberry
  (the full clause)
@alexandergalstyan
Copy link

Looks ok for now!

the script to exit skipping the packaging step.

Changelog:
- Catch the exception and log it to stdout

Test:
- Verified doing a full build still packages the android mobile sdk when
  blackberry folder doesn't exist
@dcampbell-macadamian
Copy link

Approved

jpl-mac pushed a commit that referenced this pull request Mar 22, 2012
Initial modifications to SConstruct to add blackberry
@jpl-mac jpl-mac merged commit 7174929 into Macadamian:blackberry Mar 22, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants