public inbox for gentoo-catalyst@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-catalyst] catalyst's ccache USE flag seems a bit stupid
@ 2011-06-25  2:07 Matt Turner
  2011-06-25  2:17 ` Sebastian Pipping
  0 siblings, 1 reply; 4+ messages in thread
From: Matt Turner @ 2011-06-25  2:07 UTC (permalink / raw
  To: gentoo-catalyst

In the catalyst ebuild we have

    # Here is where we actually enable ccache
    use ccache && \
        dosed 's:options="autoresume kern:options="autoresume ccache kern:' \
        /etc/catalyst/catalyst.conf
    dosed "s:/usr/lib/catalyst:/usr/$(get_libdir)/catalyst:" \
        /etc/catalyst/catalyst.conf

The ccache USE flag simply adds ccache to RDEPEND and trivially
modifies a .conf file.

Can we assume that people using catalyst are smart enough to realize
that if they turn on ccache in the .conf file that they need to
install ccache?

Matt



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [gentoo-catalyst] catalyst's ccache USE flag seems a bit stupid
  2011-06-25  2:07 [gentoo-catalyst] catalyst's ccache USE flag seems a bit stupid Matt Turner
@ 2011-06-25  2:17 ` Sebastian Pipping
  2011-06-25  3:55   ` Matt Turner
  0 siblings, 1 reply; 4+ messages in thread
From: Sebastian Pipping @ 2011-06-25  2:17 UTC (permalink / raw
  To: gentoo-catalyst

On 06/25/2011 04:07 AM, Matt Turner wrote:
> In the catalyst ebuild we have
> 
>     # Here is where we actually enable ccache
>     use ccache && \
>         dosed 's:options="autoresume kern:options="autoresume ccache kern:' \
>         /etc/catalyst/catalyst.conf
>     dosed "s:/usr/lib/catalyst:/usr/$(get_libdir)/catalyst:" \
>         /etc/catalyst/catalyst.conf
> 
> The ccache USE flag simply adds ccache to RDEPEND and trivially
> modifies a .conf file.

Confirming.


> Can we assume that people using catalyst are smart enough to realize
> that if they turn on ccache in the .conf file that they need to
> install ccache?

I agree we can assume that^^^.

I am not sure what you are suggesting for change, though:

 (A) Removing use flag ccache altogether?

       That could result in the uninstallation of the
       ccache package if it was the sole dependency of catalyst.
       Correct?

 (B) Remove the sed magic only?

       That would remove the comfort of automatic addition
       of option ccache.  Any gains?

 (C) Something else?

Best,




Sebastian



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [gentoo-catalyst] catalyst's ccache USE flag seems a bit stupid
  2011-06-25  2:17 ` Sebastian Pipping
@ 2011-06-25  3:55   ` Matt Turner
  2011-06-25 14:37     ` Sebastian Pipping
  0 siblings, 1 reply; 4+ messages in thread
From: Matt Turner @ 2011-06-25  3:55 UTC (permalink / raw
  To: gentoo-catalyst

On Fri, Jun 24, 2011 at 10:17 PM, Sebastian Pipping <sping@gentoo.org> wrote:
> I am not sure what you are suggesting for change, though:
>
>  (A) Removing use flag ccache altogether?
>
>       That could result in the uninstallation of the
>       ccache package if it was the sole dependency of catalyst.
>       Correct?

Indeed, emerge --prune would remove ccache.

>  (B) Remove the sed magic only?
>
>       That would remove the comfort of automatic addition
>       of option ccache.  Any gains?
>
>  (C) Something else?

I might suggest removing all ccache-related stuff in the ebuild and
adding a few einfo lines to the ebuild suggesting to the user to
emerge ccache (so that it'll be in @world) and how to modify
catalyst.conf to enable it. Do you think that would be OK?

It just seems like the USE flag is the wrong way of doing this.

(I'm so glad to see you working on catalyst. I've got a number of
patches that I need to commit, especially now that I see some other
activity)

Thanks!
Matt



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [gentoo-catalyst] catalyst's ccache USE flag seems a bit stupid
  2011-06-25  3:55   ` Matt Turner
@ 2011-06-25 14:37     ` Sebastian Pipping
  0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Pipping @ 2011-06-25 14:37 UTC (permalink / raw
  To: gentoo-catalyst

On 06/25/2011 05:55 AM, Matt Turner wrote:
> I might suggest removing all ccache-related stuff in the ebuild and
> adding a few einfo lines to the ebuild suggesting to the user to
> emerge ccache (so that it'll be in @world) and how to modify
> catalyst.conf to enable it. Do you think that would be OK?
> 
> It just seems like the USE flag is the wrong way of doing this.

I don't know.  dev-util/metro seems to pull in ccache in a similar way.
 I would love to hear a third voice on this.


> (I'm so glad to see you working on catalyst. I've got a number of
> patches that I need to commit, especially now that I see some other
> activity)

Please go ahead.

Best,



Sebastian



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-06-25 14:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-25  2:07 [gentoo-catalyst] catalyst's ccache USE flag seems a bit stupid Matt Turner
2011-06-25  2:17 ` Sebastian Pipping
2011-06-25  3:55   ` Matt Turner
2011-06-25 14:37     ` Sebastian Pipping

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox