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

update autocrap & Makefile as in libvmod-example - fixes make check ... #14

Open
wants to merge 2 commits into
base: 5.0
Choose a base branch
from

Conversation

nigoroll
Copy link
Contributor

... for non-default PREFIX installs of varnish

I hope we'll be using @dridi 's new m4s soon, but I just need this to work now :)

... for non-default PREFIX installs of varnish
@aondio
Copy link
Owner

aondio commented Oct 13, 2016

Hi @nigoroll,
I'm testing those changes and I get this error:

"configure.ac:5: error: possibly undefined macro: AC_MSG_ERROR
If this token and others are legitimate, please use m4_pattern_allow.
See the Autoconf documentation.
"
Do you know what it could be?

@nigoroll
Copy link
Contributor Author

outdated autoconf

@aondio
Copy link
Owner

aondio commented Oct 14, 2016

Hi @nils,
I'm running Autoconf 2.69
I'll have to investigate this.

Thanks for you pull request, I'll merge it once everything builds smoothly :)

For req.method != POST, x-meth was unset, and setting bereq.method to the
empty value is now an error
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