-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 28/02/11 15:40, Stefan Hellermann wrote: > Am 28.02.2011 15:23, schrieb Stefan Hellermann: >> Am 28.02.2011 09:08, schrieb Samuli Seppänen: >>> Hi Stefan, >>> >>> I think you should rebase your patch against the "beta2.2" branch. No >>> development is done in the "allmerged" branch, it only used to aggregate >>> code from other branches. >>> >> The commit which introduces this build-failure is in the bugfix2.1 >> branch. Could you apply it there and merge the patch to allmerged? >> >> Otherwise the weekly snapshots linked on >> https://community.openvpn.net/openvpn/wiki/TesterDocumentation >> do not build in the !ENABLE_PLUGIN case. >> >> The router distribution Openwrt is using these. >> > > plugin.h: update prototype of plugin_call dummy in !ENABLE_PLUGIN case [v2] > > Commit 2db5a0ac3e053857d97e468de53e70a605f54561 adds two arguments to > plugin_call(...), but missed the !ENABLE_PLUGIN case. With > !ENABLE_PLUGIN, plugin_call(...) is only a dummy, so add these two > parameters there too. > > --- > plugin.h | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > > There is a small copy-paste failure in the patch, the correct one below. > This one is now really run-time tested. > > Signed-off-by: Stefan Hellermann <ste...@the2masters.de>
ACK! Thanks a lot for this fix. Good catch! Applied to bugfix2.1 and merged into allmerged. commit 4c4b8cedfa98e8892a53eadd154836f8fa8cea7a Author: Stefan Hellermann <ste...@the2masters.de> List-Post: openvpn-devel@lists.sourceforge.net Date: Sun Feb 27 22:15:44 2011 +0100 Signed-off-by: Stefan Hellermann <ste...@the2masters.de> Acked-by: David Sommerseth <dav...@redhat.com> Signed-off-by: David Sommerseth <dav...@redhat.com> You asked earlier if this patch was good, and I see you managed to add the "Signed-off-by:" line (git commit -s). So this basically looks good to me! kind regards, David Sommerseth -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/ iEYEARECAAYFAk1rwdEACgkQDC186MBRfrq+9gCdGgk78cFkcXo+Oc1yEFi7Y5l0 SDkAni0Oa4lDyvK/9OrIwOAIeaEWFpVT =RoHL -----END PGP SIGNATURE-----