Re: PRServer's problem

From: Martin Jansa <martin.jansa@gmail.com>
To: Robert Yang <liezhi.yang@windriver.com>
Cc: oe-core <openembedded-core@lists.openembedded.org>
Subject: Re: PRServer's problem
Date: Wed, 18 May 2016 12:15:56 +0200	[thread overview]
Message-ID: <20160518101556.GB2579@jama> (raw)
In-Reply-To: <573C365D.3070407@windriver.com>

[-- Attachment #1: Type: text/plain, Size: 3611 bytes --]

On Wed, May 18, 2016 at 05:31:09PM +0800, Robert Yang wrote:
> 
> 
> On 05/18/2016 05:20 PM, Martin Jansa wrote:
> > On Wed, May 18, 2016 at 04:03:58PM +0800, Robert Yang wrote:
> >> Hi Martin,
> >>
> >> On 05/18/2016 03:39 PM, Martin Jansa wrote:
> >>> See:
> >>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=5970
> >>>
> >>> Just using recipe checksum wont work, because the main reason for PR bumps is to
> >>> automatically upgrade the packages when one of the dependencies changes .so
> >>> version, which you won't detect from recipe checksum of the app which is just
> >>> using the library.
> >>
> >> For the development branch like master, yes, that would happen. But for
> >> the stable release like jethro and krogoth, it is unlikely that would
> >> happen, and if if does, the user can manually bump the impacted recipe's
> >> PR to fix the problem. The current problem is that when *all* recipes'
> >> PR are bumped, there is no way to fix the problem.
> >
> > You can still stop using PR service and start doing manual PR bumps, but
> 
> We can't stop PR service and start doing manual PR bumps since we need keep
> update to date with upstream, the changes from upstream don't do the manual
> PR, and I don't think that we have to if they can be done automatically.
> 
> > it's quite annoying if you need to bump a lot of recipes you don't
> > control :).
> 
> What's your opinion about only consider RDEPENDS for PR service's checksum,
> please ?

I would like to have separate handler as described in that bug, option
c).

Not only because of unnecessary EXTENDPRAUTO bumps, but also unnecessary rebuilds.

> >>> On Wed, May 18, 2016 at 8:09 AM, Robert Yang <liezhi.yang@windriver.com
> >>> <mailto:liezhi.yang@windriver.com>> wrote:
> >>>
> >>>      The PRServer bumps PR according to do_package's task hash, that
> >>>      causes it bumps *all* packages' PR when recipes like pseudo-native
> >>>      and rpm-native is changed. It is a very bad user experience when we
> >>>      run "smart/opkg upgrade" on running target, for example, when we apply
> >>>      a CVE patch to pseudo-native or rpm-native, or do some slight changes
> >>>      in their do_compile, "smart/opkg upgrade" will download/install *all*
> >>>      the packages since all of the packages' PR are bumped.
> >>>
> >>>      Here are some rough suggestions to fix this problem, and please feel
> >>>      free to give your suggestions.
> >>>      1) Do not use do_package's task for bumping PR, the easiest way
> >>>          is simulate manually bump PR -- only bump PR when the recipe
> >>>          itself's checksum is changed.
> >>>
> >>>      2) Add a new task for PRServer, redefine its task hash for bumping
> >>>          PR, for example, this task hash only considers RDEPENDS (no
> >>>          DEPENDS), and drop any native dependencies.
> >>>
> >>>      I prefer the first way, and an alternative way maybe add a var so that
> >>>      the user can configure it:
> >>>      PR_CHECKSUM = "${BB_TASKHASH}" (current way)
> >>>      Or
> >>>      PR_CHECKSUM = "<recipe checksum>"
> >>>
> >>>      --
> >>>      Thanks
> >>>
> >>>      Robert
> >>>      --
> >>>      _______________________________________________
> >>>      Openembedded-core mailing list
> >>>      Openembedded-core@lists.openembedded.org
> >>>      <mailto:Openembedded-core@lists.openembedded.org>
> >>>      http://lists.openembedded.org/mailman/listinfo/openembedded-core
> >>>
> >>>
> >

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]