> On Mon, Jan 13, 2003 at 05:44:16PM +0800, Sergei Dolmatov wrote: > > /home/sergei/src/midgard/php4/attachment.c: In function > > `zif_mgd_stat_attachment': > > /home/sergei/src/midgard/php4/attachment.c:496: > `midgard_directory_config' > > undeclared (first use in this function) > > /home/sergei/src/midgard/php4/attachment.c:496: (Each > undeclared identifier > > is reported only once > > /home/sergei/src/midgard/php4/attachment.c:496: for each > function it appears > > in.) > > /home/sergei/src/midgard/php4/attachment.c:496: `dcfg' > undeclared (first use > > in this function) > > I wrote about this problem before... > > php_midgard.h includes <midgard/apache.h>, where it is declared. > > Before this problem was solved by including <midgard/apache.h> in > > attachment.c, but looks like it isn't right solution. > Why not? Looks right to me.
I include config.h instead of php_config.h. config.h includes <midgard/apache.h>, so that compile + make runs without problems. If you only chenge attachment.c, you should have an error during compile preparser.c. We should define HAVE_MIDGARD_PHP_CONFIG_H in (php_)config.h, so we can detect if needed header files are available or not. Regards, Kai --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
