Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe-pkgdata-util: Make parse_pkgdatafile() support package suffixed vars)
On 08/13/2018 05:14 AM, Peter Kjellerstedt wrote: > *ping* they are in stable/sumo-nmut for testing. - armin >> -Original Message- >> From: openembedded-core-boun...@lists.openembedded.org > core-boun...@lists.openembedded.org> On Behalf Of Peter Kjellerstedt >> Sent: den 7 juni 2018 16:05 >> To: akuster808 >> Cc: openembedded-core@lists.openembedded.org >> Subject: Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe- >> pkgdata-util: Make parse_pkgdatafile() support package suffixed vars) >> >>> -Original Message- >>> From: akuster808 [mailto:akuster...@gmail.com] >>> Sent: den 3 juni 2018 17:13 >>> To: Peter Kjellerstedt ; openembedded- >>> c...@lists.openembedded.org >>> Subject: Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe- >>> pkgdata-util: Make parse_pkgdatafile() support package suffixed vars) >>> >>> On 06/02/2018 12:48 PM, Peter Kjellerstedt wrote: >>>> These two patches, in addition to "oe-pkgdata-util: lookup-recipe, >>>> package-info: Don't finish prematurely" (commit 32c2a2dd) that is >>>> already on master, need to be cherry-picked to the sumo branch as >>>> well. Without them, `oe-pkgdata-util package-info ...` is more or >>>> less broken. >>>> >>>> //Peter >>> Thanks peter. I hope to remember to back port them once they hit >>> master. >>> >>> -armin >> They have hit master now. The commits I want cherry-picked to Sumo >> from master are: >> >> 32c2a2dd oe-pkgdata-util: lookup-recipe, package-info: Don't finish >> prematurely >> 38f82842 oe-pkgdata-util: Make parse_pkgdatafile() support package >> suffixed vars >> cd195bdf oe-pkgdata-util: package-info: Re-add support for the --extra >> option >> >> //Peter >> >> -- >> ___ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe-pkgdata-util: Make parse_pkgdatafile() support package suffixed vars)
*ping* > -Original Message- > From: openembedded-core-boun...@lists.openembedded.org core-boun...@lists.openembedded.org> On Behalf Of Peter Kjellerstedt > Sent: den 7 juni 2018 16:05 > To: akuster808 > Cc: openembedded-core@lists.openembedded.org > Subject: Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe- > pkgdata-util: Make parse_pkgdatafile() support package suffixed vars) > > > -Original Message- > > From: akuster808 [mailto:akuster...@gmail.com] > > Sent: den 3 juni 2018 17:13 > > To: Peter Kjellerstedt ; openembedded- > > c...@lists.openembedded.org > > Subject: Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe- > > pkgdata-util: Make parse_pkgdatafile() support package suffixed vars) > > > > On 06/02/2018 12:48 PM, Peter Kjellerstedt wrote: > > > These two patches, in addition to "oe-pkgdata-util: lookup-recipe, > > > package-info: Don't finish prematurely" (commit 32c2a2dd) that is > > > already on master, need to be cherry-picked to the sumo branch as > > > well. Without them, `oe-pkgdata-util package-info ...` is more or > > > less broken. > > > > > > //Peter > > > > Thanks peter. I hope to remember to back port them once they hit > > master. > > > > -armin > > They have hit master now. The commits I want cherry-picked to Sumo > from master are: > > 32c2a2dd oe-pkgdata-util: lookup-recipe, package-info: Don't finish > prematurely > 38f82842 oe-pkgdata-util: Make parse_pkgdatafile() support package > suffixed vars > cd195bdf oe-pkgdata-util: package-info: Re-add support for the --extra > option > > //Peter > > -- > ___ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe-pkgdata-util: Make parse_pkgdatafile() support package suffixed vars)
> -Original Message- > From: akuster808 [mailto:akuster...@gmail.com] > Sent: den 3 juni 2018 17:13 > To: Peter Kjellerstedt ; openembedded- > c...@lists.openembedded.org > Subject: Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe- > pkgdata-util: Make parse_pkgdatafile() support package suffixed vars) > > On 06/02/2018 12:48 PM, Peter Kjellerstedt wrote: > > These two patches, in addition to "oe-pkgdata-util: lookup-recipe, > > package-info: Don't finish prematurely" (commit 32c2a2dd) that is > > already on master, need to be cherry-picked to the sumo branch as > > well. Without them, `oe-pkgdata-util package-info ...` is more or > > less broken. > > > > //Peter > > Thanks peter. I hope to remember to back port them once they hit > master. > > -armin They have hit master now. The commits I want cherry-picked to Sumo from master are: 32c2a2dd oe-pkgdata-util: lookup-recipe, package-info: Don't finish prematurely 38f82842 oe-pkgdata-util: Make parse_pkgdatafile() support package suffixed vars cd195bdf oe-pkgdata-util: package-info: Re-add support for the --extra option //Peter -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe-pkgdata-util: Make parse_pkgdatafile() support package suffixed vars)
On 06/02/2018 12:48 PM, Peter Kjellerstedt wrote: > These two patches, in addition to "oe-pkgdata-util: lookup-recipe, > package-info: Don't finish prematurely" (commit 32c2a2dd) that is already > on master, need to be cherry-picked to the sumo branch as well. Without > them, `oe-pkgdata-util package-info ...` is more or less broken. > > //Peter Thanks peter. I hope to remember to back port them once they hit master. -armin >> -Original Message- >> From: openembedded-core-boun...@lists.openembedded.org >> [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf Of >> Peter Kjellerstedt >> Sent: den 2 juni 2018 21:31 >> To: openembedded-core@lists.openembedded.org >> Subject: [OE-core] [PATCHv2 1/2] oe-pkgdata-util: Make >> parse_pkgdatafile() support package suffixed vars >> >> Support for variables suffixed with package names, e.g., PKGV_foo, was >> removed in commit 3d2c87c4, which broke support for recipes that set >> other versions on their packages than what is in ${PV}. >> >> Signed-off-by: Peter Kjellerstedt >> --- >> scripts/oe-pkgdata-util | 46 ++--- >> - >> 1 file changed, 18 insertions(+), 28 deletions(-) >> >> diff --git a/scripts/oe-pkgdata-util b/scripts/oe-pkgdata-util >> index aea8a57516..965f473725 100755 >> --- a/scripts/oe-pkgdata-util >> +++ b/scripts/oe-pkgdata-util >> @@ -286,36 +286,26 @@ def lookup_recipe(args): >> >> def package_info(args): >> def parse_pkgdatafile(pkgdatafile): >> +vars = ['PKGV', 'PKGE', 'PKGR', 'PN', 'PV', 'PE', 'PR', >> 'PKGSIZE'] >> with open(pkgdatafile, 'r') as f: >> -pkge = '' >> -pkgr = '' >> -pe = '' >> -pr = '' >> +vals = dict() >> for line in f: >> -if line.startswith('PKGV:'): >> -pkg_version = line.split(':', 1)[1].strip() >> -elif line.startswith('PKGE:'): >> -pkge = line.split(':', 1)[1].strip() >> -elif line.startswith('PKGR:'): >> -pkgr = line.split(':', 1)[1].strip() >> -elif line.startswith('PN:'): >> -recipe = line.split(':', 1)[1].strip() >> -elif line.startswith('PV:'): >> -recipe_version = line.split(':', 1)[1].strip() >> -elif line.startswith('PE:'): >> -pe = line.split(':', 1)[1].strip() >> -elif line.startswith('PR:'): >> -pr = line.split(':', 1)[1].strip() >> -elif line.startswith('PKGSIZE'): >> -pkg_size = line.split(':', 1)[1].strip() >> -if pkge: >> -pkg_version = pkge + ":" + pkg_version >> -if pkgr: >> -pkg_version = pkg_version + "-" + pkgr >> -if pe: >> -recipe_version = pe + ":" + recipe_version >> -if pr: >> -recipe_version = recipe_version + "-" + pr >> +for var in vars: >> +m = re.match(var + '(?:_\S+)?:\s*(.+?)\s*$', line) >> +if m: >> +vals[var] = m.group(1) >> +pkg_version = vals['PKGV'] or '' >> +recipe = vals['PN'] or '' >> +recipe_version = vals['PV'] or '' >> +pkg_size = vals['PKGSIZE'] or '' >> +if 'PKGE' in vals: >> +pkg_version = vals['PKGE'] + ":" + pkg_version >> +if 'PKGR' in vals: >> +pkg_version = pkg_version + "-" + vals['PKGR'] >> +if 'PE' in vals: >> +recipe_version = vals['PE'] + ":" + recipe_version >> +if 'PR' in vals: >> +recipe_version = recipe_version + "-" + vals['PR'] >> print("%s %s %s %s %s" % (pkg, pkg_version, recipe, >> recipe_version, pkg_size)) >> >> # Handle both multiple arguments and multiple values within an arg >> (old syntax) >> -- >> 2.12.0 >> >> -- >> ___ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] Cherry-pick to Sumo (was: [PATCHv2 1/2] oe-pkgdata-util: Make parse_pkgdatafile() support package suffixed vars)
These two patches, in addition to "oe-pkgdata-util: lookup-recipe, package-info: Don't finish prematurely" (commit 32c2a2dd) that is already on master, need to be cherry-picked to the sumo branch as well. Without them, `oe-pkgdata-util package-info ...` is more or less broken. //Peter > -Original Message- > From: openembedded-core-boun...@lists.openembedded.org > [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf Of > Peter Kjellerstedt > Sent: den 2 juni 2018 21:31 > To: openembedded-core@lists.openembedded.org > Subject: [OE-core] [PATCHv2 1/2] oe-pkgdata-util: Make > parse_pkgdatafile() support package suffixed vars > > Support for variables suffixed with package names, e.g., PKGV_foo, was > removed in commit 3d2c87c4, which broke support for recipes that set > other versions on their packages than what is in ${PV}. > > Signed-off-by: Peter Kjellerstedt > --- > scripts/oe-pkgdata-util | 46 ++--- > - > 1 file changed, 18 insertions(+), 28 deletions(-) > > diff --git a/scripts/oe-pkgdata-util b/scripts/oe-pkgdata-util > index aea8a57516..965f473725 100755 > --- a/scripts/oe-pkgdata-util > +++ b/scripts/oe-pkgdata-util > @@ -286,36 +286,26 @@ def lookup_recipe(args): > > def package_info(args): > def parse_pkgdatafile(pkgdatafile): > +vars = ['PKGV', 'PKGE', 'PKGR', 'PN', 'PV', 'PE', 'PR', > 'PKGSIZE'] > with open(pkgdatafile, 'r') as f: > -pkge = '' > -pkgr = '' > -pe = '' > -pr = '' > +vals = dict() > for line in f: > -if line.startswith('PKGV:'): > -pkg_version = line.split(':', 1)[1].strip() > -elif line.startswith('PKGE:'): > -pkge = line.split(':', 1)[1].strip() > -elif line.startswith('PKGR:'): > -pkgr = line.split(':', 1)[1].strip() > -elif line.startswith('PN:'): > -recipe = line.split(':', 1)[1].strip() > -elif line.startswith('PV:'): > -recipe_version = line.split(':', 1)[1].strip() > -elif line.startswith('PE:'): > -pe = line.split(':', 1)[1].strip() > -elif line.startswith('PR:'): > -pr = line.split(':', 1)[1].strip() > -elif line.startswith('PKGSIZE'): > -pkg_size = line.split(':', 1)[1].strip() > -if pkge: > -pkg_version = pkge + ":" + pkg_version > -if pkgr: > -pkg_version = pkg_version + "-" + pkgr > -if pe: > -recipe_version = pe + ":" + recipe_version > -if pr: > -recipe_version = recipe_version + "-" + pr > +for var in vars: > +m = re.match(var + '(?:_\S+)?:\s*(.+?)\s*$', line) > +if m: > +vals[var] = m.group(1) > +pkg_version = vals['PKGV'] or '' > +recipe = vals['PN'] or '' > +recipe_version = vals['PV'] or '' > +pkg_size = vals['PKGSIZE'] or '' > +if 'PKGE' in vals: > +pkg_version = vals['PKGE'] + ":" + pkg_version > +if 'PKGR' in vals: > +pkg_version = pkg_version + "-" + vals['PKGR'] > +if 'PE' in vals: > +recipe_version = vals['PE'] + ":" + recipe_version > +if 'PR' in vals: > +recipe_version = recipe_version + "-" + vals['PR'] > print("%s %s %s %s %s" % (pkg, pkg_version, recipe, > recipe_version, pkg_size)) > > # Handle both multiple arguments and multiple values within an arg > (old syntax) > -- > 2.12.0 > > -- > ___ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core