[Bug other/60589] Parallel install fails due to multiple cilk.h installs

2014-03-19 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60589

Tobias Burnus  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2014-03-19
Summary|Parallel install fails  |Parallel install fails due
   ||to multiple cilk.h installs
 Ever confirmed|0   |1

--- Comment #2 from Tobias Burnus  ---
(In reply to Yury Gribov from comment #0)
> It looks like libcilkrts Makefile tries to install cilk.h twice in parallel
> (in install-nodist_cilkinclude_HEADERS and install-cilkincludeHEADERS) and
> runs into a conflict.
> The error was probably introduced in r208440.

Hmm, I wonder why I did not see it when I tried it. I once did a find to see
whether "cilk.h" was installed - but the find turned up blank. I now do see
several cilk/ include files - including cilk.h. :-(

Probably the right choice is to simply revert r208440.


> BTW could someone add libcilkrts to list of components?

What do you mean by this? I have a pending - and hopefully more sensible -
patch at http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00492.html where
-fcilkplus implies -lcilkrts [-pthread], but I am not sure whether that's what
you meant.


[Bug other/60589] Parallel install fails due to multiple cilk.h installs

2014-03-19 Thread y.gribov at samsung dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60589

--- Comment #3 from Yury Gribov  ---
(In reply to Tobias Burnus from comment #2)
> > BTW could someone add libcilkrts to list of components?
> 
> What do you mean by this?

No-no, I meant a bugzilla component so that I could fire bugs against Cilk
runtime (like we currently do for libgcc, sanitizers, etc.).


[Bug other/60589] Parallel install fails due to multiple cilk.h installs

2014-03-20 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60589

--- Comment #4 from Tobias Burnus  ---
(In reply to Tobias Burnus from comment #2)
> Probably the right choice is to simply revert r208440.

Unrelated to the revert: GCC currently installs cilk (and also ssp) under both
./lib/gcc/x86_64-unknown-linux-gnu/4.9.0/include and
./lib64/gcc/x86_64-unknown-linux-gnu/4.9.0/include; but it should do so only
under the latter path. Presumably, a "nodist_" is missing.


[Bug other/60589] Parallel install fails due to multiple cilk.h installs

2014-03-20 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60589

--- Comment #5 from Tobias Burnus  ---
Author: burnus
Date: Thu Mar 20 19:50:51 2014
New Revision: 208736

URL: http://gcc.gnu.org/viewcvs?rev=208736&root=gcc&view=rev
Log:
2014-03-20  Tobias Burnus  

PR other/60589

Reverted:
2014-03-09  Tobias Burnus  

* Makefile.am: Install cilk.h.
* Makefile.in: Regenerate.
* aclocal.m4: Regenerate.


Modified:
trunk/libcilkrts/ChangeLog
trunk/libcilkrts/Makefile.am
trunk/libcilkrts/Makefile.in
trunk/libcilkrts/aclocal.m4


[Bug other/60589] Parallel install fails due to multiple cilk.h installs

2014-04-10 Thread burnus at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60589

Tobias Burnus  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Tobias Burnus  ---
Close as FIXED. Comment 5 should have fixed the problem.

[Some other issues were fixed in commit r208847/r208847 (namely: the double
cilk.h and the automatic inclusion of -lcilkrts).]