Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 434250 - dev-util/ccache: add readme.gentoo.eclass and support for sys-devel/clang and dev-lang/icc
Summary: dev-util/ccache: add readme.gentoo.eclass and support for sys-devel/clang and...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gentoo Toolchain Maintainers
URL:
Whiteboard:
Keywords:
Depends on: 484012 484772 486404
Blocks:
  Show dependency tree
 
Reported: 2012-09-07 20:13 UTC by Martin Väth
Modified: 2013-10-21 21:40 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Add ccache links for clang and clang++ (ccache-clang.patch,637 bytes, patch)
2012-09-07 20:13 UTC, Martin Väth
Details | Diff
gx86 vs sci overlay (t.p,3.37 KB, patch)
2012-12-29 23:15 UTC, Christoph Junghans (RETIRED)
Details | Diff
gx86 vs science (ccache.patch,5.38 KB, patch)
2013-09-05 20:24 UTC, Christoph Junghans (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Väth 2012-09-07 20:13:17 UTC
Created attachment 323164 [details, diff]
Add ccache links for clang and clang++

It would be nice to install symlinks in /usr/lib*/ccache/bin to clang (and clang++): Even if sys-devel/clang is not supported as a system compiler, such symlinks are convenient for local development and testing.

The patch is rather trivial: One just needs to modify the ccache-config script to create these symlinks (patch is attached).

Perhaps it might make sense to introduce a new IUSE=clang as a condition to apply this patch.

In any case, it might be useful to have a pkg_postinst warning to set

CCACHE_CPP2=true

if ccache should be used with clang. (Otherwise many strange warnings arise, causing possibly more problems indirectly.)
Comment 1 SpanKY gentoo-dev 2012-09-08 17:08:50 UTC
they are sorting things out upstream.  i think it might be better to wait for the next release first.
Comment 2 Martin Väth 2012-09-08 19:14:25 UTC
I don't think that upstream has any plans for a directory with symlinks:

The ccache-config script (which generates the links)
exists only in FILESDIR: It is a contribution of gentoo only
and not part of upstream.
Comment 3 SpanKY gentoo-dev 2012-09-10 05:33:51 UTC
you missed my point: the current release of ccache has known bugs when using clang.  we should wait for a version that has at least those known issues fixed before we start *automatically* deploying symlinks for users.
Comment 4 Christoph Junghans (RETIRED) gentoo-dev 2012-12-29 23:15:01 UTC
Created attachment 333716 [details, diff]
gx86 vs sci overlay

Please have a look at:
<http://git.overlays.gentoo.org/gitweb/?p=proj/sci.git;a=tree;f=dev-util/ccache;h=026249923f684c03eb851144fecfea4da11c7914;hb=HEAD>

It includes icc/clang symlinks and prefix support.

We could mask the clang useflag until the next release.
Comment 5 Christoph Junghans (RETIRED) gentoo-dev 2013-07-07 23:28:52 UTC
Ping!
Comment 6 SpanKY gentoo-dev 2013-09-05 07:00:02 UTC
Comment on attachment 333716 [details, diff]
gx86 vs sci overlay

this is hard to review when you've mixed prefix, clang, and icc changes in one.  please split prefix stuff into a different bug.

plus, this bug says it is for "clang".  why is their icc stuff in there at all ?  does it even work ?
Comment 7 Christoph Junghans (RETIRED) gentoo-dev 2013-09-05 20:24:02 UTC
Created attachment 357976 [details, diff]
gx86 vs science

(In reply to SpanKY from comment #6)
> Comment on attachment 333716 [details, diff] [details, diff]
> gx86 vs sci overlay
> 
> this is hard to review when you've mixed prefix, clang, and icc changes in
> one.  please split prefix stuff into a different bug.
Sorry, but if you reply once every year, some other things go on in the mean time. Currently I don't have the time nor the desire to pick things apart.
Maybe Andrew is brave enough to do it....

> plus, this bug says it is for "clang".  why is their icc stuff in there at
> all ?  does it even work ?
Intel has an article about that:
<http://software.intel.com/en-us/articles/using-ccache-to-improve-compilation-speed-with-intel-c-compiler-for-linux>
Comment 8 Pacho Ramos gentoo-dev 2013-09-05 21:04:39 UTC
Regarding readme.gentoo.eclass usage, I usually call "readme.gentoo_print_elog" in pkg_postinst when I need to have a phase defined in ebuild
Comment 9 SpanKY gentoo-dev 2013-09-06 01:21:23 UTC
(In reply to Christoph Junghans from comment #7)

that's not really a valid reason.  if you want read all my unread bugs, then feel free.  you got a response because of the ping and it caught my eye.

i'll review things once they're split out.  that said, that DOC_CONTENTS line looks god awful and also wrong.  ROOT is not valid outside of pkg_* context.
Comment 10 SpanKY gentoo-dev 2013-09-29 01:00:03 UTC
Christoph Junghans has done all the footwork to get things merged.  this bug doesn't serve any purpose now.
Comment 11 Christoph Junghans (RETIRED) gentoo-dev 2013-09-29 01:13:00 UTC
(In reply to SpanKY from comment #10)
> Christoph Junghans has done all the footwork to get things merged.  this bug
> doesn't serve any purpose now.
The readme.gentoo.eclass is still under review in bug #486404
Comment 12 Christoph Junghans (RETIRED) gentoo-dev 2013-10-21 21:40:39 UTC
Everything fixed.