PHP plugin doesn't support parallel make

[Andreas Pakulat]

On 30.07.10 13:55:17, Alexander Shaduri wrote:
> 
> Hello,
> 
> On Fri, 30 Jul 2010 08:56:31 +0200
> Andreas Pakulat wrote:
> 
> > I've just looked again at your error message...
> > 
> > > 5 conflicts total: 1 FIRST/FOLLOW conflicts, 4 FIRST/FIRST conflicts.
> > 
> > This means something got generated.
> > 
> > > [  3%] Building CXX object parser/CMakeFiles/kdev4phpparser.dir/kdev4phpparser_automoc.o
> > > [  3%] Built target php-parser_automoc
> > > [  5%] [  7%] Generating phpast.h, phpparser.h, phpparser.cpp, phpvisitor.h, phpvisitor.cpp, phpdefaultvisitor.h, phpdefaultvisitor.cpp, phpdebugvisitor.h, phptokentext.h
> > > Building CXX object parser/CMakeFiles/kdev4phpparser.dir/phplexer.o
> > > In file included from /0S/0tmp/DL-CVS/kdevelop/php/parser/phplexer.cpp:23:
> > > /0S/0tmp/DL-CVS/kdevelop/php/000/parser/phpparser.h:7:20: error: phpast.h: No such file or directory
> > 
> > But this suggests that it never happened. So maybe your problem is
> > actually the 2 targets that both generate the files and that these are
> > generated into the same dir (as far as I can see). Can you try disabling
> > the add_custom_target() line in parser/CMakeLists.txt, does that change
> > anything?
> 
> After commenting that line I cannot reproduce the error anymore (tried it
> about 10 times). Just for the test I changed it back and got the error on
> the third run. So I guess that's what it is.

Thanks for doing that test. As the custom target is not necessary anymore
since php depends on kdevelop-pg-qt during buildtime now I'll remove it (at
least from master, not sure about the 1.0 branch). 

If someone wants to create a simple example that reproduces this (should be
enough to have a custom-command that produces a header file with a
function, add the custom-target thing and a real target that uses the
function) that could be posted as bug with kitware (or at least on the
cmake list).

Andreas

-- 
You're working under a slight handicap.  You happen to be human.