Skip to content

super-slicer: fix build for GCC 14#361477

Closed
FliegendeWurst wants to merge 1 commit intoNixOS:masterfrom
FliegendeWurst:super-slicer-fix
Closed

super-slicer: fix build for GCC 14#361477
FliegendeWurst wants to merge 1 commit intoNixOS:masterfrom
FliegendeWurst:super-slicer-fix

Conversation

@FliegendeWurst
Copy link
Member

@FliegendeWurst FliegendeWurst commented Dec 3, 2024

Regression from #359083, #356812

cc @bendlas

Things done

@github-actions github-actions bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Dec 3, 2024
@ofborg ofborg bot requested review from cab404 and hesiod December 4, 2024 04:00
@bendlas
Copy link
Contributor

bendlas commented Dec 5, 2024

Getting a build error from latest master with this applied:

error: builder for '/nix/store/47j1v340k32jn3v46bjkzqi9k42xdzlr-super-slicer-2.4.58.5.drv' failed with exit code 1;
       last 23 log lines:
       > Running phase: unpackPhase
       > unpacking source archive /nix/store/vfqwfnc0m23mlndi80amcfagyk3dddrn-source
       > source root is source
       > Running phase: patchPhase
       > substituteStream() in derivation super-slicer-2.4.58.5: WARNING: '--replace' is deprecated, use --replace-{fail,warn,quiet}. (file 'src/PrusaSlicer.cpp')
       > substituteStream() in derivation super-slicer-2.4.58.5: WARNING: pattern list\(APPEND\ wxWidgets_LIBRARIES\ libexpat\) doesn't match anything in file 'src/CMakeLists.txt'
       > substituteStream() in derivation super-slicer-2.4.58.5: WARNING: pattern libexpat doesn't match anything in file 'src/libslic3r/CMakeLists.txt'
       > applying patch /nix/store/a1vdclpnypig8j0n419590qjk9vgqdki-superslicer-2.4.58.3-boost-1.79-port-v2.patch
       > patching file src/hints/HintsToPot.cpp
       > patching file src/libslic3r/AppConfig.cpp
       > patching file src/libslic3r/AppConfig.hpp
       > patching file src/libslic3r/LocalesUtils.cpp
       > patching file src/libslic3r/Preset.cpp
       > patching file src/slic3r/GUI/CreateMMUTiledCanvas.cpp
       > patching file src/slic3r/GUI/DesktopIntegrationDialog.cpp
       > patching file src/slic3r/GUI/GUI_App.cpp
       > patching file src/slic3r/GUI/HintNotification.cpp
       > patching file src/slic3r/GUI/Preferences.cpp
       > patching file src/slic3r/GUI/ScriptExecutor.cpp
       > applying patch /nix/store/xgd5yzxl0ln024327ipsz2kdjmdslcwf-meshboolean-const.patch
       > patching file src/libslic3r/MeshBoolean.cpp
       > Hunk #1 FAILED at 147.
       > 1 out of 1 hunk FAILED -- saving rejects to file src/libslic3r/MeshBoolean.cpp.rej
       For full logs, run 'nix log /nix/store/47j1v340k32jn3v46bjkzqi9k42xdzlr-super-slicer-2.4.58.5.drv'.

With an outright revert of 3c6f5c3, we're building.

I'm wondering if we shouldn't split up prusa-slicer and super-slicer - maybe with a common base - when the build is diverging like this.

@FliegendeWurst
Copy link
Member Author

Looks like this patch was already done in the meantime.

@bendlas
Copy link
Contributor

bendlas commented Dec 5, 2024

Are you considering the issue resolved, or are you working on another fix?

@FliegendeWurst
Copy link
Member Author

I am testing again. But I'd think that a conflict on these lines indicates that it was fixed.

@FliegendeWurst
Copy link
Member Author

Actually, I checked on the wrong branch. Seems it is still an issue

@FliegendeWurst
Copy link
Member Author

Opened #362194

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants