Skip to content

Commit

Permalink
[buildmgr] Update cpackget: remove pack command from tests and scripts
Browse files Browse the repository at this point in the history
  • Loading branch information
grasci-arm authored Nov 22, 2023
1 parent 15ddb43 commit 5575cbc
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion tools/buildmgr/cbuildgen/installer/create_installer.sh
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ cp -R ../config/. ${distdir}/etc
cp ../../docs/LICENSE.txt ${distdir}

# Get cpackget
cpackget_version="1.0.1"
cpackget_version="1.1.0"
cpackget_base=https://github.com/Open-CMSIS-Pack/cpackget/releases/download/v${cpackget_version}/cpackget_${cpackget_version}
curl --retry 3 -L ${cpackget_base}_linux_amd64.tar.gz -o - | tar xzfO - --wildcards '*cpackget' > ${distdir}/bin/cpackget.lin-amd64
curl --retry 3 -L ${cpackget_base}_windows_amd64.zip -o temp.zip && unzip -p temp.zip '*/cpackget.exe' > ${distdir}/bin/cpackget.exe-amd64 && rm temp.zip
Expand Down
2 changes: 1 addition & 1 deletion tools/buildmgr/cbuildgen/installer/make_deb.sh
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ mkdir -p etc/${PACKAGE_NAME}
mkdir -p etc/profile.d

# Get cpackget
cpackget_version="1.0.1"
cpackget_version="1.1.0"
cpackget_base=https://github.com/Open-CMSIS-Pack/cpackget/releases/download/v${cpackget_version}/cpackget_${cpackget_version}
curl --retry 3 -L ${cpackget_base}_linux_amd64.tar.gz -o - | tar xzfO - --wildcards '*cpackget' > ${input}/bin/cpackget.lin-amd64

Expand Down
6 changes: 3 additions & 3 deletions tools/buildmgr/docs/doxygen/Build/src/CmdLineBuild.txt
Original file line number Diff line number Diff line change
Expand Up @@ -650,7 +650,7 @@ be downloaded and installed.
\b cpackget is called from the \b Bash command line with the following syntax:

\code
cpackget pack add -f <pack-list-file>
cpackget add -f <pack-list-file>
\endcode

<b>Where:</b>
Expand Down Expand Up @@ -705,11 +705,11 @@ The error messages that the cpackget prints are mostly self explanatory. Here ar

\code
# Trying to install a pack without specifying a pack root
$ cpackget pack add -f Simulation.cpinstall
$ cpackget add -f Simulation.cpinstall
E: pack root not found

# Trying to install a pack using an invalid pack url
$ cpackget pack add not-a-pack-path --pack-root packroot
$ cpackget add not-a-pack-path --pack-root packroot
E: bad pack name: pack extension should be "pdsc", "pack" or "zip"
\endcode

Expand Down
2 changes: 1 addition & 1 deletion tools/buildmgr/test/integrationtests/src/CPackGetTests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ void CPackGetTests::RunPackAdd(const TestParam& param, bool env) {
int ret_val;
// "-a" means "agree with embedded license"
// "-f" means "filename with pack list"
RunCPackGet(" pack add -a -f ", param, env, &ret_val);
RunCPackGet(" add -a -f ", param, env, &ret_val);
ASSERT_EQ(param.expect, (ret_val == 0) ? true : false);
}

Expand Down
2 changes: 1 addition & 1 deletion tools/buildmgr/test/scripts/download_packs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,6 @@ echo https://www.keil.com/pack/ARM.CMSIS.5.9.0.pack >> ${packlist}
echo https://www.keil.com/pack/Keil.ARM_Compiler.1.7.2.pack >> ${packlist}
echo http://www.keil.com/pack/Keil.STM32F4xx_DFP.2.14.0.pack >> ${packlist}
echo http://www.keil.com/pack/Keil.MDK-Middleware.7.11.1.pack >> ${packlist}
cpackget pack add -a -f ${packlist}
cpackget add -a -f ${packlist}

exit 0

0 comments on commit 5575cbc

Please sign in to comment.