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

Add libpostal in mason #724

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Add libpostal in mason #724

wants to merge 12 commits into from

Conversation

manoharuss
Copy link
Contributor

No description provided.

@manoharuss manoharuss marked this pull request as ready for review July 23, 2020 20:06
@manoharuss
Copy link
Contributor Author

manoharuss commented Jul 23, 2020

Hi @springmeyer when you have time, could you review this PR for installing libpostal from mason.

build works for me locally on osx with

./mason build libpostal 1.1-alpha

I wasn't able to use ./mason trigger libpostal 1.1-alpha as I didn't not have MASON_TRAVIS_TOKEN variable.

Travis: The expat installation seems return a md5 sha mismatch. I do not completely understand what happens if i directly update the sha there. could you also help with that.
https://travis-ci.org/github/mapbox/mason/builds/711237110

@mattficke
Copy link
Contributor

Travis: The expat installation seems return a md5 sha mismatch. I do not completely understand what happens if i directly update the sha there. could you also help with that.

@manoharuss a mismatched hash like that usually means there was a problem with the download. I retried the build and it succeeded.

I wasn't able to use ./mason trigger libpostal 1.1-alpha as I didn't not have MASON_TRAVIS_TOKEN variable.

You can use the travis gem to fetch this token, there are instructions in the readme: https://github.com/mapbox/mason#trigger

@springmeyer springmeyer mentioned this pull request Sep 3, 2020
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.

2 participants