On 15-Mar-17 8:56 PM, Denys Dmytriyenko wrote:
On Wed, Mar 15, 2017 at 08:10:50AM +0530, Karthik Ramanan wrote:
Denys,
Yes this is for morty.
May be I missed a notification on this, I'm thinking morty=master?
Should I mention morty explicitly?
On 15-Mar-17 3:52 AM, Denys Dmytriyenko wrote:
BTW, can this be done in a separate recipe to add additional app clients,
instead of a patch to Weston? This is not scalable/portable at all...
I take your point.
For now, lets have this patch in this way. There have been no
changes to the application for sometime.
If it undergoes additional changes we will explore ways of combining
them. There is an ongoing effort to consolidate applications so we
will include this one as well.
A bit confused by your statement above...
I was talking about "SoC performance monitoring utilities" added by this patch
being Weston clients. In other words, they are just apps talking to Weston and
hence should be in own separate code bases built by separate recipes. Not as a
patch on top of Weston...
The "SoC performance monitoring utilities" is a weston application. At
the time it
was developed, we reused some of the weston application utilities that
are present
in the weston/clients folder - more specifically window.c and window.h
utilities.
These are not libraries and not exported by Weston.
So the way it exists today, "SoC performance monitoring utilities" will
be a weston
client and that it is the reason why it is applied to the Weston
repository in this
manner.
Are you saying if you don't envision any changes to these SoC monitoring apps,
then you'll keep on carrying them as a Weston patch?
For now, yes that's the approach. This patch is an independent change to
the
weston/clients. So it should apply to any version. But having said that,
we will
try to explore the possibility of separating it to a different
application without
this dependency, but that will have to be for later.
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
