[Scummvm-devel] compiler can't find config.h in pluto.cpp in Broken Sword 2.5
Willem Jan Palenstijn
wjp at usecode.org
Mon Nov 4 22:10:38 CET 2013
On Mon, Nov 04, 2013 at 08:18:46PM +0000, Willem Jan Palenstijn wrote:
> On Mon, Nov 04, 2013 at 08:10:33PM +0000, Willem Jan Palenstijn wrote:
> > On Mon, Nov 04, 2013 at 06:55:19PM +0100, Johannes Schickel wrote:
> > > On 11/04/2013 02:22 PM, Willem Jan Palenstijn wrote:
> > > > config.h is generated by our configure script.
> > > >
> > > > It's an interesting question why pluto.cpp includes it directly without going
> > > > through common/scummsys.h or using the #ifdef HAVE_CONFIG_H guard, though. How
> > > > does this work when using create_project, which AFAICT doesn't generate a
> > > > config.h?
> > >
> > > I think it simply doesn't work for MSVC etc. which doesn't have a
> > > config.h ;-).
> >
> > That's what I'm afraid of.
> >
> > Strangely the very first thing the very next header includes is scummsys.h...
>
> Aha, that config.h was added as a wrong compilation fix in 6dc3768faaa272a12e3c6e7ce0965905338132ad.
>
> That commit should probably just be reverted.
I've reverted it, and buildbot reports no problems as expected. So config.h is
now gone from pluto.cpp.
-Willem Jan
More information about the Scummvm-devel
mailing list