> Thx a lot for fixing this~ > > Acked-by: Alex Wang <al...@nicira.com>
thank you, applied. YAMAMOTO Takashi > > > > On Sun, Apr 26, 2015 at 11:30 PM, YAMAMOTO Takashi <yamam...@valinux.co.jp> > wrote: > >> Avoid using a bash construct (=~) in the target. >> >> An alternative would be to make the configure script require >> bash explicitly. (Currently it doesn't and on NetBSD /bin/ksh >> is likely used.) >> >> The code in question was introduced by >> commit b296b82a87326e68773b970284b8e012def0e3ba . >> ("datapath: Check the export of public functions in linux/compat/linux/.") >> >> Signed-off-by: YAMAMOTO Takashi <yamam...@valinux.co.jp> >> --- >> datapath/Makefile.am | 8 +++----- >> 1 file changed, 3 insertions(+), 5 deletions(-) >> >> diff --git a/datapath/Makefile.am b/datapath/Makefile.am >> index 0b0b0bd..a8e9be4 100644 >> --- a/datapath/Makefile.am >> +++ b/datapath/Makefile.am >> @@ -42,19 +42,17 @@ COMPAT_EXPORTS := $(shell $(COMPAT_GET_EXPORTS)) >> check-export-symbol: >> @for fun_ in $(COMPAT_FUNCTIONS); do \ >> if ! grep -- $${fun_} >> $(top_srcdir)/datapath/linux/compat/build-aux/export-check-whitelist > >> /dev/null; then \ >> - if [[ ! $${fun_} =~ ^rpl_* ]] \ >> - && [[ ! $${fun_} =~ ^ovs_* ]]; then \ >> + if ! echo $${fun_} | grep -E '^(rpl|ovs)_'; then \ >> echo "Should prefix $${fun_} with rpl_ or ovs_."; \ >> exit 1; \ >> fi; \ >> fi; \ >> done >> @for fun_ in $(COMPAT_EXPORTS); do \ >> - if [[ ! $${fun_} =~ ^rpl_* ]] \ >> - && [[ ! $${fun_} =~ ^ovs_* ]]; then \ >> + if ! echo $${fun_} | grep -E '^(rpl|ovs)_'; then \ >> echo "Should prefix $${fun_} with rpl_ or ovs_."; \ >> exit 1; \ >> fi; \ >> done >> >> -all-local: check-export-symbol >> \ No newline at end of file >> +all-local: check-export-symbol >> -- >> 2.2.1 >> >> _______________________________________________ >> dev mailing list >> dev@openvswitch.org >> http://openvswitch.org/mailman/listinfo/dev >> > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev