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 perl to travis #5778

Merged
merged 6 commits into from Apr 16, 2019
Merged

add perl to travis #5778

merged 6 commits into from Apr 16, 2019

Conversation

AndyColson
Copy link

A start just to see if it'll build and test. Only one test and one example so far. If this goes ok, then I'll add more.

@geographika
Copy link
Member

@AndyColson - looks great so far. You can see build and test output at https://travis-ci.org/mapserver/mapserver/jobs/515505933

@sdlime
Copy link
Member

sdlime commented Apr 9, 2019

Great addition... Should this be completed and merged as part of beta2?

@AndyColson
Copy link
Author

I have two small swig changes I'd like to make:

  1. enable msIO_getAndStripStdoutBufferMimeHeaders for everything. (why is it python only?)
  2. clean up compiler warning in image.i (and also return null on error, rather than broken object)

Would that be ok to check in? If I check it in, and you don't like it, I dunno how to un-check-out.

@AndyColson
Copy link
Author

Ahh, bummer, my merge seems to have picked up the version change in CMakeLists.txt. How do I undo that?

@geographika
Copy link
Member

@AndyColson - is this pull request ready to merge to a new 7.4-beta release?

@AndyColson
Copy link
Author

Yes.

@geographika geographika merged commit 423d434 into MapServer:branch-7-4 Apr 16, 2019
@geographika
Copy link
Member

@AndyColson - thanks very much for setting this up

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.

None yet

3 participants