mordak / playbook-dev-tools

gcc for the Blackberry Playbook, and maybe some other stuff..
54 stars 31 forks source link

Running build.sh process stops with sed error on bison-related compilation #3

Closed jamespatrickwhite closed 11 years ago

jamespatrickwhite commented 11 years ago

When using with bbndk 2.1.0 on Ubuntu 12.10 Linux:

./build.sh
...
(much compilation output)
...
Making install in etc
make[2]: Entering directory `/home/pb/playbook-dev-tools/work/bison-2.5.1/etc'
make[3]: Entering directory `/home/pb/playbook-dev-tools/work/bison-2.5.1/etc'
make[3]: Nothing to be done for `install-exec-am'.
make[3]: Nothing to be done for `install-data-am'.
make[3]: Leaving directory `/home/pb/playbook-dev-tools/work/bison-2.5.1/etc'
make[2]: Leaving directory `/home/pb/playbook-dev-tools/work/bison-2.5.1/etc'
make[2]: Entering directory `/home/pb/playbook-dev-tools/work/bison-2.5.1'
make[3]: Entering directory `/home/pb/playbook-dev-tools/work/bison-2.5.1'
make[3]: Nothing to be done for `install-exec-am'.
 /bin/mkdir -p '/home/pb/playbook-dev-tools/pbhome/share/aclocal'
 /usr/bin/install -c -m 644 m4/bison-i18n.m4 '/home/pb/playbook-dev-tools/pbhome/share/aclocal'
make[3]: Leaving directory `/home/pb/playbook-dev-tools/work/bison-2.5.1'
make[2]: Leaving directory `/home/pb/playbook-dev-tools/work/bison-2.5.1'
make[1]: Leaving directory `/home/pb/playbook-dev-tools/work/bison-2.5.1'
sed: can't read s/'.*bison'/'bison'/: No such file or directory

The bison executable appears in the pbhome/bin/ directory, and runs if I move it manually to the PlayBook. Unfortunately, I'm not really familiar with sed to understand what it's looking for.

jamespatrickwhite commented 11 years ago

Re-running the build.sh script seems to continue passed the sed error.

mordak commented 11 years ago

This is a difference betweeen BSD sed and GNU sed. All we're doing is fixing yacc (which is just a sh wrapper around bison) so that it doesn't contain the absolute path to the bison executable (which points to something on your build system, not on the playbook). I think I have a fix and will push it as soon as I am done a clean build testing my fix for the conf dir issue. :-)

mordak commented 11 years ago

I have pushed a fix that should work for both BSD and GNU sed. Thanks!