Bring packages formater step on par with 3p installation:
- When installing a 3P via use_prebuilt_binary it will pass --installdir=${AUTOBUILD_INSTALL_DIR} to
autobuild. On the other hand packages-formater was not passing this option, this could lead to subtle
errors when AUTOBUILD_INSTALL_DIR is different to what autobuild.xml stipulates.
master
parent
603ad5a4f2
commit
cca7e12e1b
|
|
@ -16,5 +16,5 @@ add_custom_command(OUTPUT packages-info.txt
|
|||
COMMAND ${PYTHON_EXECUTABLE}
|
||||
${CMAKE_SOURCE_DIR}/cmake/run_build_test.py -DAUTOBUILD_ADDRSIZE=${ADDRESS_SIZE} -DAUTOBUILD=${AUTOBUILD_EXECUTABLE}
|
||||
${PYTHON_EXECUTABLE}
|
||||
${CMAKE_SOURCE_DIR}/../scripts/packages-formatter.py "${VIEWER_CHANNEL}" "${VIEWER_SHORT_VERSION}.${VIEWER_VERSION_REVISION}" > packages-info.txt
|
||||
${CMAKE_SOURCE_DIR}/../scripts/packages-formatter.py "${VIEWER_CHANNEL}" "${VIEWER_SHORT_VERSION}.${VIEWER_VERSION_REVISION}" "${AUTOBUILD_INSTALL_DIR}" > packages-info.txt
|
||||
)
|
||||
|
|
|
|||
|
|
@ -34,6 +34,7 @@ import argparse
|
|||
parser = argparse.ArgumentParser(description='Format dependency version and copyright information for the viewer About box content')
|
||||
parser.add_argument('channel', help='viewer channel name')
|
||||
parser.add_argument('version', help='viewer version number')
|
||||
parser.add_argument('install_dir', help="install dir of packages")
|
||||
args = parser.parse_args()
|
||||
|
||||
_autobuild=os.getenv('AUTOBUILD', 'autobuild')
|
||||
|
|
@ -74,8 +75,8 @@ def add_info(key, pkg, lines):
|
|||
else:
|
||||
dups[key].add(pkg)
|
||||
|
||||
versions=autobuild('install', '--versions')
|
||||
copyrights=autobuild('install', '--copyrights')
|
||||
versions=autobuild('install', '--versions', '--install-dir', args.install_dir)
|
||||
copyrights=autobuild('install', '--copyrights', '--install-dir', args.install_dir)
|
||||
viewer_copyright = copyrights.readline() # first line is the copyright for the viewer itself
|
||||
|
||||
# Two different autobuild outputs, but we treat them essentially the same way:
|
||||
|
|
|
|||
Loading…
Reference in New Issue