Skip to content

Commit

Permalink
Fix build order of BUILD_VMOD_$NAME m4 macro
Browse files Browse the repository at this point in the history
When a VMOD adds a CFLAG with `libvmod_$1_la_CFLAGS` the object name is no
longer `vmod_$name.lo` but `$library-vmod_name.lo` this changes the
build order such that the VCC autogenerated files would not be
guaranteed to compile first causing compilation issues.
  • Loading branch information
Steven Wojcik authored and nigoroll committed Feb 26, 2024
1 parent ce6ea40 commit 1d4950a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions varnish.m4
Expand Up @@ -213,9 +213,9 @@ AC_DEFUN([_VARNISH_VMOD], [
AC_SUBST(m4_toupper(BUILD_VMOD_$1), ["
vmod_$1.lo: vcc_$1_if.c vcc_$1_if.h
\$(libvmod_$1_la_OBJECTS): vcc_$1_if.c vcc_$1_if.h
vmod_$1.lo: \$(nodist_libvmod_$1_la_SOURCES)
\$(libvmod_$1_la_OBJECTS): \$(nodist_libvmod_$1_la_SOURCES)
vcc_$1_if.h vmod_$1.rst vmod_$1.man.rst: vcc_$1_if.c
Expand Down

0 comments on commit 1d4950a

Please sign in to comment.