bc -l segmentation faults

[dnicholson@…]

Bc segfaults when invoked with the standard math library (-l). See this thread for discussion:

​http://linuxfromscratch.org/pipermail/blfs-support/2006-March/058971.html

The fix already exists in our repo:

​http://www.linuxfromscratch.org/patches/downloads/bc/bc-1.06-fixes-1.patch

Oddly, this patch is from a while back, and I have a working bc on a gcc-3.4 system without this patch. I think the fix should go into the book.

However, that would put bc at 3 patches with a total of 4 lines changed. The same thing can be accomplished with these 4 sed's (I like sed's and don't like patches):

sed -i '/PROTO.*readline/d' bc/scan.l
sed -i '/flex -I8/s/8//' configure
sed -i '/stdlib/a #include <string.h>' lib/number.c
sed -i 's/^  program_counter/static &/' bc/load.c

Any thoughts?

[bdubbs@…]

I prefer seds too. Patches are not as obvious as to what they are changing. It is especially nice when different techniques are used because they provide more educational content.

[Randy McMurchy]

While this is being fixed, it should be considered whether or not to remove the libedit dependency. Though the dependency is valid, installing libedit really screws up the system.

[dnicholson@…]

Fix committed in r5746. Closing.

[dnicholson@…]

Also removed libedit as was discussed on blfs-dev since it causes problems and is no longer maintained.