Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

Add Varnish5 compatibility #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Varnish5 compatibility #10

wants to merge 1 commit into from

Conversation

liyo
Copy link

@liyo liyo commented Aug 9, 2017

This is just a working version to build vmod for Varnish5. It probably can be improved.

@JonathanHuot
Copy link

JonathanHuot commented Aug 29, 2017

Hi @liyo , have you tested with PR #9 ? I did approximatively the same thing copying changes from libvmod-example instead.

However, fastly team does not seem very active on this.

@liyo
Copy link
Author

liyo commented Aug 29, 2017

Hey @JonathanHuot, I tried your PR #9 but I couldn't make it build on my Varnish5 install. The following error is shown when I try to use it:

configure.ac:13: installing 'build-aux/compile'
configure.ac:13: installing 'build-aux/config.guess'
configure.ac:13: installing 'build-aux/config.sub'
configure.ac:9: installing 'build-aux/install-sh'
configure.ac:9: installing 'build-aux/missing'
automake: warnings are treated as errors
/usr/share/automake-1.14/am/ltlibrary.am: warning: 'libvmod_urlcode.la': linking libtool libraries using a non-POSIX
/usr/share/automake-1.14/am/ltlibrary.am: archiver requires 'AM_PROG_AR' in 'configure.ac'
src/Makefile.am:4: while processing Libtool library 'libvmod_urlcode.la'
src/Makefile.am: installing 'build-aux/depcomp'
parallel-tests: installing 'build-aux/test-driver'
ERROR: Service 'varnish' failed to build: The command '/bin/sh -c curl -SLO "https://github.com/thomsonreuters/libvmod-urlcode/archive/master.tar.gz" && mkdir -p /usr/local/varnish-libvmod-urlcode && tar -xzvf "master.tar.gz" -C /usr/local/varnish-libvmod-urlcode --strip-components=1 && cd /usr/local/varnish-libvmod-urlcode && ./autogen.sh && ./configure VARNISHSRC=/usr/local/varnish-cache && make && make install' returned a non-zero code: 1

@JonathanHuot
Copy link

Thanks @liyo, it seems a different automake version was used in my tests. I should have fixed it now. Could you retest it please ?
Thanks in advance!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants