On 2/20/19 5:37 PM, Alex Bennée wrote: > Paolo Bonzini <pbonz...@redhat.com> writes: > >> It looks like something in netmap has changed and compilation fails: >> >> install -D libnetmap.a //usr/local/lib/libnetmap.a >> install: cannot stat libnetmap.a: No such file or directory >> >> Add an explicit "make" step to fix it. > > I took an alternate approach as tracking another projects master seems > like a bad idea: > > tests/docker: peg netmap code to a specific version > > Tracking head is always going to be at the whims of the upstream. > Let's use a defined release so things don't magically change under us.
Oh now I see your reply... > Signed-off-by: Alex Bennée <alex.ben...@linaro.org> > > 1 file changed, 1 insertion(+) > tests/docker/dockerfiles/debian-amd64.docker | 1 + > > modified tests/docker/dockerfiles/debian-amd64.docker > @@ -33,6 +33,7 @@ RUN DEBIAN_FRONTEND=noninteractive eatmydata \ > apt-get install -y --no-install-recommends \ > linux-headers-amd64 > RUN git clone https://github.com/luigirizzo/netmap.git /usr/src/netmap > +RUN cd /usr/src/netmap && git checkout v11.3 ... and I notice I forgot to change directory in my previous suggestion. Why do you take v11.3 and not v11.4? I agree v11.3 was closer to what I tested when I introduced this dependency, but I'd rather use the latest release. Regards, Phil. > RUN cd /usr/src/netmap/LINUX && ./configure --no-drivers --no-apps > --kernel-dir=$(ls -d /usr/src/linux-headers-*-amd64) && make install > ENV QEMU_CONFIGURE_OPTS --enable-netmap > > > <snip> > > -- > Alex Bennée >
