* [PATCH hwc] Android: add CleanSpec.mk
@ 2017-11-03 4:45 Chih-Wei Huang
2017-11-03 12:35 ` Rob Herring
0 siblings, 1 reply; 5+ messages in thread
From: Chih-Wei Huang @ 2017-11-03 4:45 UTC (permalink / raw)
To: dri-devel; +Cc: Chih-Wei Huang
The file contains rules that are executed on incremental builds.
Since commit 4f7dc9b6 the library was moved to /vendor so
the old file must be removed.
Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>
---
CleanSpec.mk | 1 +
1 file changed, 1 insertion(+)
create mode 100644 CleanSpec.mk
diff --git a/CleanSpec.mk b/CleanSpec.mk
new file mode 100644
index 0000000..99dcecd
--- /dev/null
+++ b/CleanSpec.mk
@@ -0,0 +1 @@
+$(call add-clean-step, rm -rf $(TARGET_OUT)/lib*/hw/hwcomposer.drm.so)
--
2.13.6
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH hwc] Android: add CleanSpec.mk
2017-11-03 4:45 [PATCH hwc] Android: add CleanSpec.mk Chih-Wei Huang
@ 2017-11-03 12:35 ` Rob Herring
2017-11-06 2:12 ` Chih-Wei Huang
2017-11-06 11:16 ` Robert Foss
0 siblings, 2 replies; 5+ messages in thread
From: Rob Herring @ 2017-11-03 12:35 UTC (permalink / raw)
To: Chih-Wei Huang; +Cc: Chih-Wei Huang, dri-devel
On Thu, Nov 2, 2017 at 11:45 PM, Chih-Wei Huang <cwhuang@android-x86.org> wrote:
> The file contains rules that are executed on incremental builds.
> Since commit 4f7dc9b6 the library was moved to /vendor so
> the old file must be removed.
>
> Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>
> ---
> CleanSpec.mk | 1 +
> 1 file changed, 1 insertion(+)
> create mode 100644 CleanSpec.mk
>
> diff --git a/CleanSpec.mk b/CleanSpec.mk
> new file mode 100644
> index 0000000..99dcecd
> --- /dev/null
> +++ b/CleanSpec.mk
> @@ -0,0 +1 @@
> +$(call add-clean-step, rm -rf $(TARGET_OUT)/lib*/hw/hwcomposer.drm.so)
Seems a bit silly to add forever an explicit file to clean for a
transient problem.
Rob
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH hwc] Android: add CleanSpec.mk
2017-11-03 12:35 ` Rob Herring
@ 2017-11-06 2:12 ` Chih-Wei Huang
2017-11-06 11:16 ` Robert Foss
1 sibling, 0 replies; 5+ messages in thread
From: Chih-Wei Huang @ 2017-11-06 2:12 UTC (permalink / raw)
To: Rob Herring; +Cc: Chih-Wei Huang, dri-devel
2017-11-03 20:35 GMT+08:00 Rob Herring <robh@kernel.org>:
> On Thu, Nov 2, 2017 at 11:45 PM, Chih-Wei Huang <cwhuang@android-x86.org> wrote:
>> The file contains rules that are executed on incremental builds.
>> Since commit 4f7dc9b6 the library was moved to /vendor so
>> the old file must be removed.
>>
>> Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>
>> ---
>> CleanSpec.mk | 1 +
>> 1 file changed, 1 insertion(+)
>> create mode 100644 CleanSpec.mk
>>
>> diff --git a/CleanSpec.mk b/CleanSpec.mk
>> new file mode 100644
>> index 0000000..99dcecd
>> --- /dev/null
>> +++ b/CleanSpec.mk
>> @@ -0,0 +1 @@
>> +$(call add-clean-step, rm -rf $(TARGET_OUT)/lib*/hw/hwcomposer.drm.so)
>
> Seems a bit silly to add forever an explicit file to clean for a
> transient problem.
The problem is people usually won't notice
the libraries have been moved to /vendor.
In an incremental build the old files still
exist in /system that may cause issues.
It's necessary unless you have a better idea
to tell people "hey, you need to do a clean build".
--
Chih-Wei
Android-x86 project
http://www.android-x86.org
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH hwc] Android: add CleanSpec.mk
2017-11-03 12:35 ` Rob Herring
2017-11-06 2:12 ` Chih-Wei Huang
@ 2017-11-06 11:16 ` Robert Foss
2017-11-06 13:38 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Robert Foss @ 2017-11-06 11:16 UTC (permalink / raw)
To: Rob Herring, Chih-Wei Huang; +Cc: Chih-Wei Huang, dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 1023 bytes --]
Hey Rob,
On Fri, 2017-11-03 at 07:35 -0500, Rob Herring wrote:
> On Thu, Nov 2, 2017 at 11:45 PM, Chih-Wei Huang <cwhuang@android-x86.
> org> wrote:
> > The file contains rules that are executed on incremental builds.
> > Since commit 4f7dc9b6 the library was moved to /vendor so
> > the old file must be removed.
> >
> > Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>
> > ---
> > CleanSpec.mk | 1 +
> > 1 file changed, 1 insertion(+)
> > create mode 100644 CleanSpec.mk
> >
> > diff --git a/CleanSpec.mk b/CleanSpec.mk
> > new file mode 100644
> > index 0000000..99dcecd
> > --- /dev/null
> > +++ b/CleanSpec.mk
> > @@ -0,0 +1 @@
> > +$(call add-clean-step, rm -rf
> > $(TARGET_OUT)/lib*/hw/hwcomposer.drm.so)
>
> Seems a bit silly to add forever an explicit file to clean for a
> transient problem.
>
While transient, if maintaining multuple Android versions it may still
stick around for quite some time.
That being said, I don't have much of an opinion about this.
Rob.
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH hwc] Android: add CleanSpec.mk
2017-11-06 11:16 ` Robert Foss
@ 2017-11-06 13:38 ` Rob Herring
0 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2017-11-06 13:38 UTC (permalink / raw)
To: Robert Foss; +Cc: Chih-Wei Huang, dri-devel
On Mon, Nov 6, 2017 at 5:16 AM, Robert Foss <robert.foss@collabora.com> wrote:
> Hey Rob,
>
> On Fri, 2017-11-03 at 07:35 -0500, Rob Herring wrote:
>> On Thu, Nov 2, 2017 at 11:45 PM, Chih-Wei Huang <cwhuang@android-x86.
>> org> wrote:
>> > The file contains rules that are executed on incremental builds.
>> > Since commit 4f7dc9b6 the library was moved to /vendor so
>> > the old file must be removed.
>> >
>> > Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw>
>> > ---
>> > CleanSpec.mk | 1 +
>> > 1 file changed, 1 insertion(+)
>> > create mode 100644 CleanSpec.mk
>> >
>> > diff --git a/CleanSpec.mk b/CleanSpec.mk
>> > new file mode 100644
>> > index 0000000..99dcecd
>> > --- /dev/null
>> > +++ b/CleanSpec.mk
>> > @@ -0,0 +1 @@
>> > +$(call add-clean-step, rm -rf
>> > $(TARGET_OUT)/lib*/hw/hwcomposer.drm.so)
>>
>> Seems a bit silly to add forever an explicit file to clean for a
>> transient problem.
>>
> While transient, if maintaining multuple Android versions it may still
> stick around for quite some time.
That's not the issue. Once you do a clean build, the problem is gone
regardless of version.
Rob
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-11-06 13:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-03 4:45 [PATCH hwc] Android: add CleanSpec.mk Chih-Wei Huang
2017-11-03 12:35 ` Rob Herring
2017-11-06 2:12 ` Chih-Wei Huang
2017-11-06 11:16 ` Robert Foss
2017-11-06 13:38 ` Rob Herring
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.