| Summary: | ASTERISK-06840: Do not require zaptel by default | ||
| Reporter: | Andrey S Pankov (casper) | Labels: | |
| Date Opened: | 2006-04-24 19:22:35 | Date Closed: | 2006-04-26 19:11:55 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Core/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ||
| Description: | It seems like there is still a problem I described to Russell. If I have libtonezone installed, but no linux/zaptel.h or zaptel.h the build fails forcing me to install zaptel-devel. I understand that removing libtonezone will solve the problem, but IMO the default autoconf behavior should be to disable feature if headers are not found for installed library. It concerns not only zaptel, but other libraries as well using AST_EXT_LIB | ||
| Comments: | By: Andrey S Pankov (casper) 2006-04-24 19:24:00 BTW, it would be nice to create "configure/build issues" category in the bugtracker. By: Russell Bryant (russell) 2006-04-24 23:51:33 No. The build does not fail. The configure script fails informing you of the situation. It says you either need to install the devel package, or run configure again with --without-zaptel. Is this what you are seeing? That's what is supposed to happen ... By: Andrey S Pankov (casper) 2006-04-25 01:26:17 --without-tonezone more litterally... By: Andrey S Pankov (casper) 2006-04-25 01:29:33 If we speak about no differences in the old and the new build process it should not ask to install something, it should just not include the support for things not found during configure stage. By: Russell Bryant (russell) 2006-04-25 12:23:33 The configure script will now not include support for a library if the library is found, but the associated headers are not found. The only time it will error out is if --with-lib is explicitly specified. By: Andrey S Pankov (casper) 2006-04-26 18:41:54 If library is found, but headers are not configure defines e.g. HAVE_ZAPTEL... :( It would be reasonable to check for presence of e.g. zaptel.h or linux/zaptel.h in AC_CHECK_HEADERS and have HAVE_ZAPTEL_H, HAVE_LINUX_ZAPTEL_H to define HAVE_ZAPTEL or not. By: Russell Bryant (russell) 2006-04-26 19:11:44 This should be fixed in rev 22703 | ||