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

suggestion: continuous integration & automate maven publishing #38

Merged
merged 4 commits into from
Jan 11, 2018

Conversation

cesine
Copy link
Contributor

@cesine cesine commented Dec 19, 2017

I looked into #35. I was able to get it set up for you using continuous integration so that pushing to master can auto-publish new versions of the library (its also relatively easy to add it as a step in the build.gradle which you can run manually instead).

As part of this, I changed the requirement on pocketsphinx and sphinxbase to be set via an Environment Variable rather than needing to be installed in the same dir as this repo.

How this was tested

screen shot 2017-12-19 at 1 20 54 pm

Resulting aar:

http://dl.bintray.com/cesine/educmupocketsphinx/pocketsphinx-android/5prealpha/

screen shot 2017-12-19 at 1 21 11 pm

Verified build using the demo:

I added travis:

cmusphinx/pocketsphinx-android-demo#47

Then I used the artifacts
cesine/pocketsphinx-android-demo@f78626e

https://travis-ci.org/cesine/pocketsphinx-android-demo/builds/318603056


If you want to use continuous integration with travis, these are the steps you would have do. Getting this set up is complex, and when android build changes, the travis build does get out of date.

Bintray

Travis

After the files are uploading to http://dl.bintray.com/edu.cmu.pocketsphinx/android/pocketsphinx-android/5prealpha/ then change the config to only upload new releases and to run only on the master branch:

  • remove "override": 1
  • remove all_branches: true

@nshmyrev nshmyrev merged commit 08e765b into cmusphinx:master Jan 11, 2018
@nshmyrev
Copy link
Contributor

Let me work on it, thank you!

@cesine cesine deleted the cmusphinx/travis branch January 18, 2018 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants