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

configure: configure-ac updates #1427 v5 #3794

Closed

Conversation

jmtaylor90
Copy link
Contributor

Make sure these boxes are signed before submitting your Pull Request -- thank you.

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/1427

Describe changes:

  • removed 'exit 1' as requested in v4 review
  • updated nss library/includes checks
  • updates nspr library/includes checks
  • added appveyor/travis builds

PRScript output (if applicable):

Signed-off-by: jason taylor <jtfas90@gmail.com>
Signed-off-by: jason taylor <jtfas90@gmail.com>
* added nss and nspr requirements for appveyor build
* added nss and nspr requirements for travis builds
* added travis build without nss and nspr

Signed-off-by: jason taylor <jtfas90@gmail.com>
@jmtaylor90 jmtaylor90 requested a review from a team as a code owner April 11, 2019 13:41
@victorjulien victorjulien mentioned this pull request Apr 19, 2019
@victorjulien
Copy link
Member

Merged in #3810, thanks Jason!

@jmtaylor90 jmtaylor90 deleted the configure-ac-updates-1427-v5 branch April 29, 2019 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants