Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
@ 2012-07-09 11:55 Richard Purdie
  2012-07-09 14:16 ` Khem Raj
  0 siblings, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2012-07-09 11:55 UTC (permalink / raw)
  To: openembedded-core

This ensures a rebuild does really build from scratch when ccache is in use.

[YOCTO #2696]

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/classes/ccache.bbclass b/meta/classes/ccache.bbclass
index 5aa2da2..cd06a4b 100644
--- a/meta/classes/ccache.bbclass
+++ b/meta/classes/ccache.bbclass
@@ -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
 
 do_configure[dirs] =+ "${CCACHE_DIR}"
 do_kernel_configme[dirs] =+ "${CCACHE_DIR}"
+
+do_clean[cleandirs] += "${CCACHE_DIR}"





^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
  2012-07-09 11:55 [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task Richard Purdie
@ 2012-07-09 14:16 ` Khem Raj
  2012-07-09 14:25   ` Mark Hatle
  2012-07-09 14:27   ` Richard Purdie
  0 siblings, 2 replies; 7+ messages in thread
From: Khem Raj @ 2012-07-09 14:16 UTC (permalink / raw)
  To: openembedded-core

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 7/9/2012 4:55 AM, Richard Purdie wrote:
> This ensures a rebuild does really build from scratch when ccache
> is in use.
> 
> [YOCTO #2696]
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> 
> --- diff --git a/meta/classes/ccache.bbclass
> b/meta/classes/ccache.bbclass index 5aa2da2..cd06a4b 100644 ---
> a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass @@
> -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
> 
> do_configure[dirs] =+ "${CCACHE_DIR}" do_kernel_configme[dirs] =+
> "${CCACHE_DIR}" + +do_clean[cleandirs] += "${CCACHE_DIR}"
> 

so will this remove all contents of .ccache dir ?
or just the package you are cleanall'ing
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk/657cACgkQuwUzVZGdMxR6/gCeIFHIc01FiI5bCB5jiFaPev0/
VXkAn1V/VQEEfwQop6kt2/kgaei2A5yZ
=gYFM
-----END PGP SIGNATURE-----



^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
  2012-07-09 14:16 ` Khem Raj
@ 2012-07-09 14:25   ` Mark Hatle
  2012-07-09 14:31     ` Richard Purdie
  2012-07-09 14:27   ` Richard Purdie
  1 sibling, 1 reply; 7+ messages in thread
From: Mark Hatle @ 2012-07-09 14:25 UTC (permalink / raw)
  To: openembedded-core

On 7/9/12 9:16 AM, Khem Raj wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 7/9/2012 4:55 AM, Richard Purdie wrote:
>> This ensures a rebuild does really build from scratch when ccache
>> is in use.
>>
>> [YOCTO #2696]
>>
>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> --- diff --git a/meta/classes/ccache.bbclass
>> b/meta/classes/ccache.bbclass index 5aa2da2..cd06a4b 100644 ---
>> a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass @@
>> -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
>>
>> do_configure[dirs] =+ "${CCACHE_DIR}" do_kernel_configme[dirs] =+
>> "${CCACHE_DIR}" + +do_clean[cleandirs] += "${CCACHE_DIR}"
>>
>
> so will this remove all contents of .ccache dir ?
> or just the package you are cleanall'ing

I was wondering the same thing, with the default configuration:

bitbake.conf:# Assign CCACHE_DIR a default value to fix a bug of ccache 3.1.7,
bitbake.conf:# since it would always create CCACHE_DIR/.ccache even if
bitbake.conf:export CCACHE_DIR ??= "${@os.getenv('HOME')}"

--Mark

> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.12 (MingW32)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAk/657cACgkQuwUzVZGdMxR6/gCeIFHIc01FiI5bCB5jiFaPev0/
> VXkAn1V/VQEEfwQop6kt2/kgaei2A5yZ
> =gYFM
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>





^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
  2012-07-09 14:16 ` Khem Raj
  2012-07-09 14:25   ` Mark Hatle
@ 2012-07-09 14:27   ` Richard Purdie
  2012-07-09 14:31     ` Mark Hatle
  1 sibling, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2012-07-09 14:27 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Mon, 2012-07-09 at 07:16 -0700, Khem Raj wrote:
> On 7/9/2012 4:55 AM, Richard Purdie wrote:
> > This ensures a rebuild does really build from scratch when ccache
> > is in use.
> > 
> > [YOCTO #2696]
> > 
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> 
> > --- diff --git a/meta/classes/ccache.bbclass
> > b/meta/classes/ccache.bbclass index 5aa2da2..cd06a4b 100644 ---
> > a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass @@
> > -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
> > 
> > do_configure[dirs] =+ "${CCACHE_DIR}" do_kernel_configme[dirs] =+
> > "${CCACHE_DIR}" + +do_clean[cleandirs] += "${CCACHE_DIR}"
> > 
> 
> so will this remove all contents of .ccache dir ?
> or just the package you are cleanall'ing

Our default CCACHE_DIR defined above this has ${PN} in it so just the
package you are cleaning.

Cheers,

Richard




^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
  2012-07-09 14:27   ` Richard Purdie
@ 2012-07-09 14:31     ` Mark Hatle
  2012-07-09 14:48       ` Richard Purdie
  0 siblings, 1 reply; 7+ messages in thread
From: Mark Hatle @ 2012-07-09 14:31 UTC (permalink / raw)
  To: openembedded-core

On 7/9/12 9:27 AM, Richard Purdie wrote:
> On Mon, 2012-07-09 at 07:16 -0700, Khem Raj wrote:
>> On 7/9/2012 4:55 AM, Richard Purdie wrote:
>>> This ensures a rebuild does really build from scratch when ccache
>>> is in use.
>>>
>>> [YOCTO #2696]
>>>
>>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>>> --- diff --git a/meta/classes/ccache.bbclass
>>> b/meta/classes/ccache.bbclass index 5aa2da2..cd06a4b 100644 ---
>>> a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass @@
>>> -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
>>>
>>> do_configure[dirs] =+ "${CCACHE_DIR}" do_kernel_configme[dirs] =+
>>> "${CCACHE_DIR}" + +do_clean[cleandirs] += "${CCACHE_DIR}"
>>>
>>
>> so will this remove all contents of .ccache dir ?
>> or just the package you are cleanall'ing
>
> Our default CCACHE_DIR defined above this has ${PN} in it so just the
> package you are cleaning.

I'm worried about what happens if someone changes the CCACHE_DIR to a single 
shared location.

--Mark

> Cheers,
>
> Richard
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>





^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
  2012-07-09 14:25   ` Mark Hatle
@ 2012-07-09 14:31     ` Richard Purdie
  0 siblings, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2012-07-09 14:31 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Mon, 2012-07-09 at 09:25 -0500, Mark Hatle wrote:
> On 7/9/12 9:16 AM, Khem Raj wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > On 7/9/2012 4:55 AM, Richard Purdie wrote:
> >> This ensures a rebuild does really build from scratch when ccache
> >> is in use.
> >>
> >> [YOCTO #2696]
> >>
> >> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >> --- diff --git a/meta/classes/ccache.bbclass
> >> b/meta/classes/ccache.bbclass index 5aa2da2..cd06a4b 100644 ---
> >> a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass @@
> >> -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
> >>
> >> do_configure[dirs] =+ "${CCACHE_DIR}" do_kernel_configme[dirs] =+
> >> "${CCACHE_DIR}" + +do_clean[cleandirs] += "${CCACHE_DIR}"
> >>
> >
> > so will this remove all contents of .ccache dir ?
> > or just the package you are cleanall'ing
> 
> I was wondering the same thing, with the default configuration:
> 
> bitbake.conf:# Assign CCACHE_DIR a default value to fix a bug of ccache 3.1.7,
> bitbake.conf:# since it would always create CCACHE_DIR/.ccache even if
> bitbake.conf:export CCACHE_DIR ??= "${@os.getenv('HOME')}"

That line is only in the class file and that class redefines CCACHE_DIR
to include PN.

Cheers,

Richard




^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task
  2012-07-09 14:31     ` Mark Hatle
@ 2012-07-09 14:48       ` Richard Purdie
  0 siblings, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2012-07-09 14:48 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Mon, 2012-07-09 at 09:31 -0500, Mark Hatle wrote:
> On 7/9/12 9:27 AM, Richard Purdie wrote:
> > On Mon, 2012-07-09 at 07:16 -0700, Khem Raj wrote:
> >> On 7/9/2012 4:55 AM, Richard Purdie wrote:
> >>> This ensures a rebuild does really build from scratch when ccache
> >>> is in use.
> >>>
> >>> [YOCTO #2696]
> >>>
> >>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >>> --- diff --git a/meta/classes/ccache.bbclass
> >>> b/meta/classes/ccache.bbclass index 5aa2da2..cd06a4b 100644 ---
> >>> a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass @@
> >>> -4,3 +4,5 @@ CCACHE_DISABLE[unexport] = "1"
> >>>
> >>> do_configure[dirs] =+ "${CCACHE_DIR}" do_kernel_configme[dirs] =+
> >>> "${CCACHE_DIR}" + +do_clean[cleandirs] += "${CCACHE_DIR}"
> >>>
> >>
> >> so will this remove all contents of .ccache dir ?
> >> or just the package you are cleanall'ing
> >
> > Our default CCACHE_DIR defined above this has ${PN} in it so just the
> > package you are cleaning.
> 
> I'm worried about what happens if someone changes the CCACHE_DIR to a single 
> shared location.

It would wipe that shared location upon a clean unless they change the
class...

Cheers,

Richard




^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-07-09 15:00 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-09 11:55 [PATCH] ccache: Ensure CCACHE_DIR is cleaned by do_clean task Richard Purdie
2012-07-09 14:16 ` Khem Raj
2012-07-09 14:25   ` Mark Hatle
2012-07-09 14:31     ` Richard Purdie
2012-07-09 14:27   ` Richard Purdie
2012-07-09 14:31     ` Mark Hatle
2012-07-09 14:48       ` Richard Purdie

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