Skip to content

Commit 2edd82f

Browse files
schielesupermerill
authored andcommitted
scintilla is not needed
Why are we jumping through hoops for something we don't need? There was special handling for the scintilla integration in wxWidgets despite that fact that we actually don't even need it in the code. This removes this useless dependency and the corresponding special handling of the library. It also adapts the counting in the build scripts since we removed one step.
1 parent 05d6f11 commit 2edd82f

File tree

5 files changed

+20
-44
lines changed

5 files changed

+20
-44
lines changed

BuildLinux.sh

Lines changed: 6 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ then
203203
then
204204
mkdir deps/build
205205
fi
206-
echo -e "[1/9] Configuring dependencies ...\n"
206+
echo -e "[1/8] Configuring dependencies ...\n"
207207
BUILD_ARGS=""
208208
if [[ -n "$FOUND_GTK3_DEV" ]]
209209
then
@@ -228,32 +228,16 @@ then
228228
cmake .. $BUILD_ARGS
229229
echo -e "\n ... done\n"
230230

231-
echo -e "\n[2/9] Building dependencies...\n"
231+
echo -e "\n[2/8] Building dependencies...\n"
232232
# make deps
233233
make -j$NCORES
234-
echo -e "\n ... done\n"
235-
236-
# rename wxscintilla
237-
echo "[3/9] Renaming wxscintilla library..."
238-
pushd destdir/usr/local/lib > /dev/null
239-
if [[ -z "$FOUND_GTK3_DEV" ]]
240-
then
241-
cp libwxscintilla-3.2.a libwx_gtk2u_scintilla-3.2.a
242-
else
243-
cp libwxscintilla-3.2.a libwx_gtk3u_scintilla-3.2.a
244-
fi
245-
echo "> ls destdir/usr/local/lib"
246-
ls -al .
247-
echo "> ls ROOT/deps/build/destdir/usr/local/lib"
248-
ls -al $ROOT/deps/build/destdir/usr/local/lib
249-
popd > /dev/null
250234
popd > /dev/null
251235
echo -e "\n ... done\n"
252236
fi
253237

254238
if [[ -n "$BUILD_CLEANDEPEND" ]]
255239
then
256-
echo -e "[4/9] Cleaning dependencies...\n"
240+
echo -e "[3/8] Cleaning dependencies...\n"
257241
pushd deps/build > /dev/null
258242
rm -fr dep_*
259243
popd > /dev/null
@@ -262,7 +246,7 @@ fi
262246

263247
if [[ -n "$BUILD_SLIC3R" ]]
264248
then
265-
echo -e "[5/9] Configuring SuperSlicer ...\n"
249+
echo -e "[4/8] Configuring SuperSlicer ...\n"
266250
if [[ -n $BUILD_WIPE ]]
267251
then
268252
echo -n "wiping build directory ..."
@@ -317,12 +301,12 @@ then
317301
cmake .. -DCMAKE_PREFIX_PATH="$PWD/../deps/build/destdir/usr/local" -DSLIC3R_STATIC=1 ${BUILD_ARGS}
318302
echo " ... done"
319303
# make SuperSlicer
320-
echo -e "\n[6/9] Building SuperSlicer ...\n"
304+
echo -e "\n[5/8] Building SuperSlicer ...\n"
321305
make -j$NCORES Slic3r
322306
make -j$NCORES OCCTWrapper
323307
echo -e "\n ... done"
324308

325-
echo -e "\n[7/9] Generating language files ...\n"
309+
echo -e "\n[6/8] Generating language files ...\n"
326310
#make .mo
327311
if [[ -n "$UPDATE_POTFILE" ]]
328312
then

BuildMacOS.sh

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@ brew --prefix zstd
207207
export $BUILD_ARCH
208208
export LIBRARY_PATH=$LIBRARY_PATH:$(brew --prefix zstd)/lib/
209209

210-
echo -n "[1/9] Updating submodules..."
210+
echo -n "[1/8] Updating submodules..."
211211
{
212212
# update submodule profiles
213213
pushd resources/profiles
@@ -219,12 +219,12 @@ echo "done"
219219

220220
if [[ -n "$VERSION_DATE" ]]
221221
then
222-
echo -n "[2/9] Changing date in version ... "
222+
echo -n "[2/8] Changing date in version ... "
223223
# change date in version
224224
sed "s/+UNKNOWN/-$(date '+%F')/" version.inc > version.date.inc
225225
echo "done"
226226
else
227-
echo -n "[2/9] Changing date in version: remove UNKNOWN ... "
227+
echo -n "[2/8] Changing date in version: remove UNKNOWN ... "
228228
sed "s/+UNKNOWN//" version.inc > version.date.inc
229229
echo "done"
230230
fi
@@ -242,7 +242,7 @@ then
242242
then
243243
mkdir deps/build
244244
fi
245-
echo -e " \n[3/9] Configuring dependencies ... \n"
245+
echo -e " \n[3/8] Configuring dependencies ... \n"
246246
BUILD_ARGS=""
247247
if [[ -n "$BUILD_ARCH" ]]
248248
then
@@ -271,7 +271,7 @@ then
271271
exit 1 # terminate and indicate error
272272
fi
273273

274-
echo -e "[4/9] Building dependencies ...\n"
274+
echo -e "[4/8] Building dependencies ...\n"
275275

276276
# make deps
277277
make -j$NCORES
@@ -283,20 +283,12 @@ then
283283
exit 1 # terminate and indicate error
284284
fi
285285

286-
echo -e "[5/9] Renaming wxscintilla library ...\n"
287-
288-
# rename wxscintilla
289-
pushd destdir/usr/local/lib > /dev/null
290-
cp libwxscintilla-3.2.a libwx_osx_cocoau_scintilla-3.2.a
291-
292286
popd > /dev/null
293-
popd > /dev/null
294-
echo -e "\n ... done\n"
295287
fi
296288

297289
if [[ -n "$BUILD_CLEANDEPEND" ]]
298290
then
299-
echo -e "[6/9] Cleaning dependencies...\n"
291+
echo -e "[5/8] Cleaning dependencies...\n"
300292
pushd deps/build
301293
pwd
302294
rm -fr dep_*
@@ -306,7 +298,7 @@ fi
306298

307299
if [[ -n "$BUILD_SLIC3R" ]]
308300
then
309-
echo -e "[5/9] Configuring Slicer ...\n"
301+
echo -e "[4/8] Configuring Slicer ...\n"
310302

311303
if [[ -n $BUILD_WIPE ]]
312304
then
@@ -361,7 +353,7 @@ then
361353
# make Slic3r
362354
if [[ -z "$BUILD_XCODE" ]]
363355
then
364-
echo -e "\n[6/9] Building Slicer ...\n"
356+
echo -e "\n[5/8] Building Slicer ...\n"
365357
make -j$NCORES Slic3r
366358
if [ $? -eq 0 ]
367359
then
@@ -372,7 +364,7 @@ then
372364
fi
373365
fi
374366

375-
echo -e "\n[7/9] Generating language files ...\n"
367+
echo -e "\n[6/8] Generating language files ...\n"
376368
#make .mo
377369
if [[ -n "$UPDATE_POTFILE" ]]
378370
then

src/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ if (SLIC3R_GUI)
5757
if (CMAKE_SYSTEM_NAME STREQUAL "Linux")
5858
set (wxWidgets_CONFIG_OPTIONS "--toolkit=gtk${SLIC3R_GTK}")
5959
endif ()
60-
find_package(wxWidgets 3.2 MODULE REQUIRED COMPONENTS base core adv html gl stc scintilla)
60+
find_package(wxWidgets 3.2 MODULE REQUIRED COMPONENTS base core adv html gl stc)
6161

6262
include(${wxWidgets_USE_FILE})
6363

src/platform/osx/BuildMacOSImage.sh.in

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ while getopts ":ixavhz" opt; do
4242
esac
4343
done
4444

45-
echo -e "\n[8/9] Generating MacOS app..."
45+
echo -e "\n[7/8] Generating MacOS app..."
4646
# {
4747
# update Info.plist
4848
pushd src
@@ -176,7 +176,7 @@ DMG_NAME=@SLIC3R_APP_KEY@${VERSION_NUMBER}-$OS_NAME.dmg
176176

177177
if [[ -n "$BUILD_IMAGE" ]]
178178
then
179-
echo -e "\n[9/9] Creating .tgz and DMG Image for distribution..."
179+
echo -e "\n[8/8] Creating .tgz and DMG Image for distribution..."
180180
{
181181

182182
tar -czvf @SLIC3R_APP_KEY@${VERSION_NUMBER}-$OS_NAME.tgz pack/@SLIC3R_APP_KEY@

src/platform/unix/BuildLinuxImage.sh.in

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ done
4949

5050
if [[ -n "$BUILD_APP" ]]
5151
then
52-
echo -e "\n[8/9] Generating Linux app ..."
52+
echo -e "\n[7/8] Generating Linux app ..."
5353
# create directory and copy into it
5454
if [ -d "package" ]
5555
then
@@ -88,7 +88,7 @@ fi
8888

8989
if [[ -n "$BUILD_IMAGE" ]]
9090
then
91-
echo -e "\n[9/9] Creating .tgz and Appimage for distribution ...\n"
91+
echo -e "\n[8/8] Creating .tgz and Appimage for distribution ...\n"
9292

9393
pushd package > /dev/null
9494
if [[ -z "$FORCE_GTK2" ]]

0 commit comments

Comments
 (0)