mirror of
https://git.yoctoproject.org/poky
synced 2026-04-20 11:28:58 +00:00
devtool: add "rename" subcommand
When you run devtool add on a source tree we attempt to figure out the correct name and version for the recipe. However, despite our best efforts, sometimes the name and/or version we come up with isn't correct, and the only way to remedy that up until now was to reset the recipe, delete the source tree and start again, specifying the name this time. To avoid this slightly painful procedure, add a "rename" subcommand that lets you rename the recipe and/or change the version. (From OE-Core rev: 9303d8055c45a0f6af295d70a6f6a8b9d8d8a7c9) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Paul Eggleton
committed by
Richard Purdie
Richard Purdie
parent
7aebaa4204
commit
43e652f3d1
@@ -852,6 +852,199 @@ def modify(args, config, basepath, workspace):
|
||||
|
||||
return 0
|
||||
|
||||
|
||||
def rename(args, config, basepath, workspace):
|
||||
"""Entry point for the devtool 'rename' subcommand"""
|
||||
import bb
|
||||
import oe.recipeutils
|
||||
|
||||
check_workspace_recipe(workspace, args.recipename)
|
||||
|
||||
if not (args.newname or args.version):
|
||||
raise DevtoolError('You must specify a new name, a version with -V/--version, or both')
|
||||
|
||||
recipefile = workspace[args.recipename]['recipefile']
|
||||
if not recipefile:
|
||||
raise DevtoolError('devtool rename can only be used where the recipe file itself is in the workspace (e.g. after devtool add)')
|
||||
|
||||
if args.newname and args.newname != args.recipename:
|
||||
reason = oe.recipeutils.validate_pn(args.newname)
|
||||
if reason:
|
||||
raise DevtoolError(reason)
|
||||
newname = args.newname
|
||||
else:
|
||||
newname = args.recipename
|
||||
|
||||
append = workspace[args.recipename]['bbappend']
|
||||
appendfn = os.path.splitext(os.path.basename(append))[0]
|
||||
splitfn = appendfn.split('_')
|
||||
if len(splitfn) > 1:
|
||||
origfnver = appendfn.split('_')[1]
|
||||
else:
|
||||
origfnver = ''
|
||||
|
||||
recipefilemd5 = None
|
||||
tinfoil = setup_tinfoil(basepath=basepath, tracking=True)
|
||||
try:
|
||||
rd = parse_recipe(config, tinfoil, args.recipename, True)
|
||||
if not rd:
|
||||
return 1
|
||||
|
||||
bp = rd.getVar('BP', True)
|
||||
bpn = rd.getVar('BPN', True)
|
||||
if newname != args.recipename:
|
||||
localdata = rd.createCopy()
|
||||
localdata.setVar('PN', newname)
|
||||
newbpn = localdata.getVar('BPN', True)
|
||||
else:
|
||||
newbpn = bpn
|
||||
s = rd.getVar('S', False)
|
||||
src_uri = rd.getVar('SRC_URI', False)
|
||||
pv = rd.getVar('PV', True)
|
||||
|
||||
# Correct variable values that refer to the upstream source - these
|
||||
# values must stay the same, so if the name/version are changing then
|
||||
# we need to fix them up
|
||||
new_s = s
|
||||
new_src_uri = src_uri
|
||||
if newbpn != bpn:
|
||||
# ${PN} here is technically almost always incorrect, but people do use it
|
||||
new_s = new_s.replace('${BPN}', bpn)
|
||||
new_s = new_s.replace('${PN}', bpn)
|
||||
new_s = new_s.replace('${BP}', '%s-${PV}' % bpn)
|
||||
new_src_uri = new_src_uri.replace('${BPN}', bpn)
|
||||
new_src_uri = new_src_uri.replace('${PN}', bpn)
|
||||
new_src_uri = new_src_uri.replace('${BP}', '%s-${PV}' % bpn)
|
||||
if args.version and origfnver == pv:
|
||||
new_s = new_s.replace('${PV}', pv)
|
||||
new_s = new_s.replace('${BP}', '${BPN}-%s' % pv)
|
||||
new_src_uri = new_src_uri.replace('${PV}', pv)
|
||||
new_src_uri = new_src_uri.replace('${BP}', '${BPN}-%s' % pv)
|
||||
patchfields = {}
|
||||
if new_s != s:
|
||||
patchfields['S'] = new_s
|
||||
if new_src_uri != src_uri:
|
||||
patchfields['SRC_URI'] = new_src_uri
|
||||
if patchfields:
|
||||
recipefilemd5 = bb.utils.md5_file(recipefile)
|
||||
oe.recipeutils.patch_recipe(rd, recipefile, patchfields)
|
||||
newrecipefilemd5 = bb.utils.md5_file(recipefile)
|
||||
finally:
|
||||
tinfoil.shutdown()
|
||||
|
||||
if args.version:
|
||||
newver = args.version
|
||||
else:
|
||||
newver = origfnver
|
||||
|
||||
if newver:
|
||||
newappend = '%s_%s.bbappend' % (newname, newver)
|
||||
newfile = '%s_%s.bb' % (newname, newver)
|
||||
else:
|
||||
newappend = '%s.bbappend' % newname
|
||||
newfile = '%s.bb' % newname
|
||||
|
||||
oldrecipedir = os.path.dirname(recipefile)
|
||||
newrecipedir = os.path.join(config.workspace_path, 'recipes', newname)
|
||||
if oldrecipedir != newrecipedir:
|
||||
bb.utils.mkdirhier(newrecipedir)
|
||||
|
||||
newappend = os.path.join(os.path.dirname(append), newappend)
|
||||
newfile = os.path.join(newrecipedir, newfile)
|
||||
|
||||
# Rename bbappend
|
||||
logger.info('Renaming %s to %s' % (append, newappend))
|
||||
os.rename(append, newappend)
|
||||
# Rename recipe file
|
||||
logger.info('Renaming %s to %s' % (recipefile, newfile))
|
||||
os.rename(recipefile, newfile)
|
||||
|
||||
# Rename source tree if it's the default path
|
||||
appendmd5 = None
|
||||
if not args.no_srctree:
|
||||
srctree = workspace[args.recipename]['srctree']
|
||||
if os.path.abspath(srctree) == os.path.join(config.workspace_path, 'sources', args.recipename):
|
||||
newsrctree = os.path.join(config.workspace_path, 'sources', newname)
|
||||
logger.info('Renaming %s to %s' % (srctree, newsrctree))
|
||||
shutil.move(srctree, newsrctree)
|
||||
# Correct any references (basically EXTERNALSRC*) in the .bbappend
|
||||
appendmd5 = bb.utils.md5_file(newappend)
|
||||
appendlines = []
|
||||
with open(newappend, 'r') as f:
|
||||
for line in f:
|
||||
appendlines.append(line)
|
||||
with open(newappend, 'w') as f:
|
||||
for line in appendlines:
|
||||
if srctree in line:
|
||||
line = line.replace(srctree, newsrctree)
|
||||
f.write(line)
|
||||
newappendmd5 = bb.utils.md5_file(newappend)
|
||||
|
||||
bpndir = None
|
||||
newbpndir = None
|
||||
if newbpn != bpn:
|
||||
bpndir = os.path.join(oldrecipedir, bpn)
|
||||
if os.path.exists(bpndir):
|
||||
newbpndir = os.path.join(newrecipedir, newbpn)
|
||||
logger.info('Renaming %s to %s' % (bpndir, newbpndir))
|
||||
shutil.move(bpndir, newbpndir)
|
||||
|
||||
bpdir = None
|
||||
newbpdir = None
|
||||
if newver != origfnver or newbpn != bpn:
|
||||
bpdir = os.path.join(oldrecipedir, bp)
|
||||
if os.path.exists(bpdir):
|
||||
newbpdir = os.path.join(newrecipedir, '%s-%s' % (newbpn, newver))
|
||||
logger.info('Renaming %s to %s' % (bpdir, newbpdir))
|
||||
shutil.move(bpdir, newbpdir)
|
||||
|
||||
if oldrecipedir != newrecipedir:
|
||||
# Move any stray files and delete the old recipe directory
|
||||
for entry in os.listdir(oldrecipedir):
|
||||
oldpath = os.path.join(oldrecipedir, entry)
|
||||
newpath = os.path.join(newrecipedir, entry)
|
||||
logger.info('Renaming %s to %s' % (oldpath, newpath))
|
||||
shutil.move(oldpath, newpath)
|
||||
os.rmdir(oldrecipedir)
|
||||
|
||||
# Now take care of entries in .devtool_md5
|
||||
md5entries = []
|
||||
with open(os.path.join(config.workspace_path, '.devtool_md5'), 'r') as f:
|
||||
for line in f:
|
||||
md5entries.append(line)
|
||||
|
||||
if bpndir and newbpndir:
|
||||
relbpndir = os.path.relpath(bpndir, config.workspace_path) + '/'
|
||||
else:
|
||||
relbpndir = None
|
||||
if bpdir and newbpdir:
|
||||
relbpdir = os.path.relpath(bpdir, config.workspace_path) + '/'
|
||||
else:
|
||||
relbpdir = None
|
||||
|
||||
with open(os.path.join(config.workspace_path, '.devtool_md5'), 'w') as f:
|
||||
for entry in md5entries:
|
||||
splitentry = entry.rstrip().split('|')
|
||||
if len(splitentry) > 2:
|
||||
if splitentry[0] == args.recipename:
|
||||
splitentry[0] = newname
|
||||
if splitentry[1] == os.path.relpath(append, config.workspace_path):
|
||||
splitentry[1] = os.path.relpath(newappend, config.workspace_path)
|
||||
if appendmd5 and splitentry[2] == appendmd5:
|
||||
splitentry[2] = newappendmd5
|
||||
elif splitentry[1] == os.path.relpath(recipefile, config.workspace_path):
|
||||
splitentry[1] = os.path.relpath(newfile, config.workspace_path)
|
||||
if recipefilemd5 and splitentry[2] == recipefilemd5:
|
||||
splitentry[2] = newrecipefilemd5
|
||||
elif relbpndir and splitentry[1].startswith(relbpndir):
|
||||
splitentry[1] = os.path.relpath(os.path.join(newbpndir, splitentry[1][len(relbpndir):]), config.workspace_path)
|
||||
elif relbpdir and splitentry[1].startswith(relbpdir):
|
||||
splitentry[1] = os.path.relpath(os.path.join(newbpdir, splitentry[1][len(relbpdir):]), config.workspace_path)
|
||||
entry = '|'.join(splitentry) + '\n'
|
||||
f.write(entry)
|
||||
return 0
|
||||
|
||||
|
||||
def _get_patchset_revs(srctree, recipe_path, initial_rev=None):
|
||||
"""Get initial and update rev of a recipe. These are the start point of the
|
||||
whole patchset and start point for the patches to be re-generated/updated.
|
||||
@@ -1630,6 +1823,15 @@ def register_commands(subparsers, context):
|
||||
parser_sync.add_argument('--keep-temp', action="store_true", help='Keep temporary directory (for debugging)')
|
||||
parser_sync.set_defaults(func=sync)
|
||||
|
||||
parser_rename = subparsers.add_parser('rename', help='Rename a recipe file in the workspace',
|
||||
description='Renames the recipe file for a recipe in the workspace, changing the name or version part or both, ensuring that all references within the workspace are updated at the same time. Only works when the recipe file itself is in the workspace, e.g. after devtool add. Particularly useful when devtool add did not automatically determine the correct name.',
|
||||
group='working', order=10)
|
||||
parser_rename.add_argument('recipename', help='Current name of recipe to rename')
|
||||
parser_rename.add_argument('newname', nargs='?', help='New name for recipe (optional, not needed if you only want to change the version)')
|
||||
parser_rename.add_argument('--version', '-V', help='Change the version (NOTE: this does not change the version fetched by the recipe, just the version in the recipe file name)')
|
||||
parser_rename.add_argument('--no-srctree', '-s', action='store_true', help='Do not rename the source tree directory (if the default source tree path has been used) - keeping the old name may be desirable if there are internal/other external references to this path')
|
||||
parser_rename.set_defaults(func=rename)
|
||||
|
||||
parser_update_recipe = subparsers.add_parser('update-recipe', help='Apply changes from external source tree to recipe',
|
||||
description='Applies changes from external source tree to a recipe (updating/adding/removing patches as necessary, or by updating SRCREV). Note that these changes need to have been committed to the git repository in order to be recognised.',
|
||||
group='working', order=-90)
|
||||
|
||||
Reference in New Issue
Block a user