Firefox 53 scheduled to arrive in 2017-04-18 is going to depend on Hunspell 1.5.0. Note, some headers are no longer packaged: https://github.com/hunspell/hunspell/commit/99675e791d12
Created attachment 177400 [details] v0 poudriere bulk -t is green for 93i386, 93amd64, 101i386, 103amd64, 110i386. The following consumers are yet to be tested: devel/codeblocks devel/lokalize editors/codelite editors/focuswriter editors/focuswriter-qt4 editors/libreoffice editors/libreoffice4 editors/openoffice-4 editors/openoffice-devel editors/tea editors/texmaker editors/texstudio editors/textroom editors/texworks german/hunspell korean/hunspell mail/thunderbird multimedia/aegisub net-im/licq-qt-gui print/lyx print/scribus textproc/enchant textproc/goldendict textproc/kf5-sonnet textproc/mythes textproc/sigil textproc/te-hunspell www/firefox www/firefox-esr www/libxul www/qupzilla-qt4 www/qupzilla-qt5 www/seamonkey x11/leechcraft
Created attachment 177405 [details] v0.1 - s/libhunspell-1.3/libhunspell-1.5/ in LIB_DEPENDS - Bump PORTREVISION in consumers
A commit references this bug: Author: jbeich Date: Sat Nov 26 08:09:44 UTC 2016 New revision: 427132 URL: https://svnweb.freebsd.org/changeset/ports/427132 Log: textproc/enchant: unbreak with hunspell 1.5 myspell_checker.cpp:430:8: error: cannot initialize a variable of type 'char *' with an rvalue of type 'const char *' char *enc = myspell->get_dic_encoding(); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ https://github.com/hunspell/hunspell/commit/971f8a9cf52e PR: 214837 Approved by: portmgr blanket Changes: head/textproc/enchant/files/patch-src_myspell_myspell__checker.cpp
A commit references this bug: Author: jbeich Date: Sat Nov 26 12:30:04 UTC 2016 New revision: 427160 URL: https://svnweb.freebsd.org/changeset/ports/427160 Log: x11/leechcraft: unbreak with hunspell 1.5 -- Could NOT find HUNSPELL (missing: HUNSPELL_LIBRARIES) [...] CMake Error: The following variables are used in this project, but they are set to NOTFOUND. Please set them or make sure they are set and tested correctly in the CMake files: HUNSPELL_LIBRARIES (ADVANCED) linked by target "leechcraft_rosenthal" in directory src/plugins/rosenthal PR: 214837 Approved by: portmgr blanket Changes: head/x11/leechcraft/files/patch-plugins_rosenthal_cmake_FindHunspell.cmake
A commit references this bug: Author: jbeich Date: Sat Nov 26 12:30:31 UTC 2016 New revision: 427161 URL: https://svnweb.freebsd.org/changeset/ports/427161 Log: editors/codelite: unbreak with hunspell 1.5 ** ** NOTICE ** Could not locate libhunspell. SpellChecker plugin will not be build ** NOTICE ** Try installing libhunspell-dev package ** [...] ===> Building package for codelite-9.2_2 pkg-static: Unable to access file ${STAGEDIR}${PREFIX}/lib/codelite/SpellCheck.so: No such file or directory pkg-static: DEVELOPER_MODE: Plist error, missing file: lib/codelite/SpellCheck.so pkg-static: Unable to access file ${STAGEDIR}${PREFIX}/share/codelite/dics/en_GB.aff: No such file or directory pkg-static: DEVELOPER_MODE: Plist error, missing file: share/codelite/dics/en_GB.aff pkg-static: Unable to access file ${STAGEDIR}${PREFIX}/share/codelite/dics/en_GB.dic: No such file or directory pkg-static: DEVELOPER_MODE: Plist error, missing file: share/codelite/dics/en_GB.dic pkg-static: Unable to access file ${STAGEDIR}${PREFIX}/share/codelite/dics/en_US.aff: No such file or directory pkg-static: DEVELOPER_MODE: Plist error, missing file: share/codelite/dics/en_US.aff pkg-static: Unable to access file ${STAGEDIR}${PREFIX}/share/codelite/dics/en_US.dic: No such file or directory pkg-static: DEVELOPER_MODE: Plist error, missing file: share/codelite/dics/en_US.dic *** Error code 1 PR: 214837 Approved by: portmgr blanket Changes: head/editors/codelite/files/ head/editors/codelite/files/patch-SpellChecker_CMakeLists.txt
A commit references this bug: Author: jbeich Date: Sat Nov 26 12:30:54 UTC 2016 New revision: 427162 URL: https://svnweb.freebsd.org/changeset/ports/427162 Log: editors/tea: unbreak with hunspell 1.5 spellchecker.cpp:339:12: error: assigning to 'char *' from incompatible type 'const char *' encoding = speller->get_dic_encoding(); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ https://github.com/hunspell/hunspell/commit/971f8a9cf52e PR: 214837 Approved by: portmgr blanket Changes: head/editors/tea/files/ head/editors/tea/files/patch-spellchecker.h
A commit references this bug: Author: jbeich Date: Sat Nov 26 12:31:16 UTC 2016 New revision: 427163 URL: https://svnweb.freebsd.org/changeset/ports/427163 Log: print/scribus: unbreak with hunspell 1.5 -- Could NOT find HUNSPELL (missing: HUNSPELL_LIBRARIES) HunSpell or its developer libraries NOT found - Disabling support for spell checking [...] ===> Checking for items in pkg-plist which are not in STAGEDIR Error: Missing: lib/scribus/plugins/libhunspellplugin.so scribus/plugins/tools/hunspellcheck/hunspelldict.cpp:22:9: error: cannot initialize a variable of type 'char *' with an rvalue of type 'const char *' char* dictEncoding = m_hunspell->get_dic_encoding(); ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ https://github.com/hunspell/hunspell/commit/971f8a9cf52e PR: 214837 Approved by: portmgr blanket Changes: head/print/scribus/files/patch-cmake_modules_FindHUNSPELL.cmake head/print/scribus/files/patch-scribus_plugins_tools_hunspellcheck_hunspelldict.cpp
A commit references this bug: Author: jbeich Date: Sat Nov 26 12:31:38 UTC 2016 New revision: 427164 URL: https://svnweb.freebsd.org/changeset/ports/427164 Log: textproc/kf5-sonnet: unbreak with hunspell 1.5 -- Could NOT find HUNSPELL (missing: HUNSPELL_LIBRARIES) [...] -- The following OPTIONAL packages have not been found: * HUNSPELL , Spell checking support via Hunspell , <http://hunspell.sourceforge.net/> [...] ===> Checking for items in pkg-plist which are not in STAGEDIR Error: Missing: %%QT_PLUGINDIR%%/kf5/sonnet/hunspell.so PR: 214837 Approved by: portmgr blanket Changes: head/textproc/kf5-sonnet/files/ head/textproc/kf5-sonnet/files/patch-cmake_FindHUNSPELL.cmake
A commit references this bug: Author: jbeich Date: Sat Nov 26 12:32:02 UTC 2016 New revision: 427165 URL: https://svnweb.freebsd.org/changeset/ports/427165 Log: x11/kdelibs4: unbreak consumers with hunspell 1.5 $ make -C devel/lokalize [...] -- Could NOT find HUNSPELL (missing: HUNSPELL_LIBRARIES) CMake Error at /usr/local/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:148 (message): Could NOT find HUNSPELL (missing: HUNSPELL_LIBRARIES) Call Stack (most recent call first): /usr/local/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:388 (_FPHSA_FAILURE_MESSAGE) /usr/local/share/apps/cmake/modules/FindHUNSPELL.cmake:22 (FIND_PACKAGE_HANDLE_STANDARD_ARGS) src/CMakeLists.txt:4 (find_package) PR: 214837 Approved by: portmgr blanket Changes: head/x11/kdelibs4/Makefile head/x11/kdelibs4/files/patch-cmake_modules_FindHUNSPELL.cmake
My QA per comment 1 is complete. Trivial bustage is fixed, the rest has blocking bugs. Once very few or none remain an exp-run would be good for a finishing touch e.g., if more consumers appear by that time.
Created attachment 177429 [details] v0.2 (rebased against ports r427185) - Drop unnecessary changes e.g., USES=autoreconf
Created attachment 177458 [details] v0.3 (rebased against ports r427270) - Update to hunspell 1.5.1 which fixes compatibility with 1.4 API
1.5.2 was released shortly after 1.5.1 with more API compatibility fixes -- the specific one that was addressed may fix editors/texmaker, but I'm fixing that port in a different way.
Created attachment 177472 [details] v0.4 (rebased against ports r427292) - Dropped namespace pollution from csutil.hxx - Skip to v1.5.2 to keep `const` at least for now
poudriere bulk -tj 103amd64 is green for every direct consumer here. Can you check more? ### /usr/local/etc/poudriere.d/PR214837-make.conf ### OPTIONS_SET += HUNSPELL editors_tea_UNSET += ASPELL x11_leechcraft_SET += ROSENTHAL
Created attachment 177479 [details] v0.4.1 (rebased against ports r427295) Update to 1.5.3 but add back `const` for get_dic_encoding(). Skipping more than one version ends up confusing.
Exp-run looks fine
A commit references this bug: Author: jbeich Date: Sat Dec 3 22:24:40 UTC 2016 New revision: 427743 URL: https://svnweb.freebsd.org/changeset/ports/427743 Log: textproc/hunspell: update to 1.5.4 Changes: https://github.com/hunspell/hunspell/blob/v1.5.4/ChangeLog PR: 214837 Exp-run by: antoine (for 1.5.3) Approved by: office (bapt) Changes: head/Mk/bsd.gecko.mk head/devel/codeblocks/Makefile head/devel/lokalize/Makefile head/editors/codelite/Makefile head/editors/focuswriter/Makefile head/editors/libreoffice/Makefile head/editors/libreoffice/Makefile.common head/editors/libreoffice4/Makefile head/editors/openoffice-4/Makefile head/editors/openoffice-devel/Makefile head/editors/tea/Makefile head/editors/texmaker/Makefile head/editors/texstudio/Makefile head/editors/textroom/Makefile head/editors/texworks/Makefile head/german/hunspell/Makefile head/korean/hunspell/Makefile head/mail/thunderbird/Makefile head/multimedia/aegisub/Makefile head/net-im/licq-qt-gui/Makefile head/print/lyx/Makefile head/print/scribus/Makefile head/textproc/enchant/Makefile head/textproc/goldendict/Makefile head/textproc/hunspell/Makefile head/textproc/hunspell/distinfo head/textproc/hunspell/files/patch-add-const head/textproc/hunspell/files/patch-configure head/textproc/hunspell/files/patch-src_tools_hunspell.cxx head/textproc/hunspell/pkg-descr head/textproc/hunspell/pkg-plist head/textproc/kf5-sonnet/Makefile head/textproc/mythes/Makefile head/textproc/sigil/Makefile head/textproc/te-hunspell/Makefile head/www/firefox/Makefile head/www/firefox-esr/Makefile head/www/libxul/Makefile head/www/qupzilla-qt4/Makefile head/www/qupzilla-qt5/Makefile head/www/seamonkey/Makefile head/x11/leechcraft/Makefile