Skip to content

Commit 7826ff9

Browse files
committed
Fix boards manager package size issue
Because git ls-files was run in package directory, it didn't list tools directories, so these were not excluded from rsync, causing package to include local copy of tools. Fixed by changing into root directory before doing anything.
1 parent 71347b8 commit 7826ff9

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

package/build_boards_manager_package.sh

+6-5
Original file line numberDiff line numberDiff line change
@@ -8,17 +8,17 @@ if [ -z "$REMOTE_URL" ]; then
88
echo "REMOTE_URL not defined, using default"
99
fi
1010
echo "Remote: $REMOTE_URL"
11-
11+
pushd ..
1212
# Figure out how will the package be called
1313
ver=`git describe --tags --always`
1414
package_name=esp8266-$ver
1515
echo "Version: $ver"
1616
echo "Package name: $package_name"
1717

1818
# Create directory for the package
19-
outdir=versions/$ver/$package_name
20-
srcdir=$PWD/..
21-
rm -rf versions/$ver
19+
outdir=package/versions/$ver/$package_name
20+
srcdir=$PWD
21+
rm -rf package/versions/$ver
2222
mkdir -p $outdir
2323

2424
# Some files should be excluded from the package
@@ -58,7 +58,7 @@ $SED 's/tools.mkspiffs.path={runtime.platform.path}\/tools\/mkspiffs/tools.mkspi
5858
> $outdir/platform.txt
5959

6060
# Zip the package
61-
pushd versions/$ver
61+
pushd package/versions/$ver
6262
echo "Making $package_name.zip"
6363
zip -qr $package_name.zip $package_name
6464
rm -rf $package_name
@@ -80,3 +80,4 @@ jq ".packages[0].platforms[0].version = \"$ver\" | \
8080
> package_esp8266com_index.json
8181

8282
popd
83+
popd

0 commit comments

Comments
 (0)