[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#834331: test code fails and ignored for ibus



Hi,

On Sun, Feb 17, 2019 at 11:02:45AM +0000, Simon McVittie wrote:
> On Sun, 17 Feb 2019 at 15:44:07 +0900, Osamu Aoki wrote:
> > ./runtest: 113: ./runtest: pushd: not found
> 
> pushd is a bash feature (a "bashism"). If runtest needs to use it,
> then it should be a #!/bin/bash script, not a #!/bin/sh script.

Thanks for good catch.

ibus being Developed by RH people under FEDORA etc., #!/bin/sh may be
BASH.  I see.

autogen.sh:#!/bin/sh
debian/ibus.prerm:#!/bin/sh
debian/ibus.postrm:#!/bin/sh
debian/ibus-doc.preinst:#!/bin/sh
debian/ibus.postinst:#!/bin/sh
docs/reference/ibus/Makefile.in:	$(AM_V_GEN)echo "#!/bin/sh -e" > $@; \
gtk-doc.make:	$(AM_V_GEN)echo "#!/bin/sh -e" > $@; \
install-sh:#!/bin/sh
ltmain.sh:#   #!/bin/sh
py-compile:#!/bin/sh
setup/ibus-setup.in:#!/bin/sh
src/tests/runtest:#!/bin/sh

Hmmm... src/tests/runtest:#!/bin/sh is the only causality ...  

Let me patch it to see...

> > NOT FOUNND ibus-memconf. Need configure --enable-memconf
> 
> It looks as though the tests rely on an optional feature that the Debian
> package doesn't enable.

Yah... this is what I was thinking.  debian/rules can be adjusted  ...
I will try to build with --enable-memconf to see whast it produces.

Thanks.

Osamu


Reply to: