[ISSUE] Package request: qtwebengine_dictionaries

[ISSUE] Package request: qtwebengine_dictionaries

[Eloitor]

[-- Attachment #1: Type: text/plain, Size: 1067 bytes --]

New issue by Eloitor on void-packages repository

https://github.com/void-linux/void-packages/issues/49030

Description:
### Package name

qtwebengine_dictionaries

### Package homepage

https://doc.qt.io/qt-6.6/qt-add-webengine-dictionary.html

### Description

I'm not sure about this. The use case is to provide spell checking for the dooble Browser.
1. Install and launch the `dooble` browser.
2. Edit -> Settings -> Web -> Spell Checkers

You get a warning that `qtwebengines_dictionary` cannot be accessed. I searched using `xlocate "\.bdic"` and didn't find any package that provided such (only `Grammalecte` for french).

Reading from https://doc.qt.io/qt-6.6/qt-add-webengine-dictionary.html

> A spell checker in Qt Web Engine needs dictionaries in a specific binary format. This CMake command converts dictionaries from the [Hunspell project](https://sourceforge.net/projects/hunspell/). into the bdict binary format.



### Does the requested package meet the package requirements?

System, Compiled

### Is the requested package released?

Yes

Re: Package request: qtwebengine_dictionaries

[chrysos349]

[-- Attachment #1: Type: text/plain, Size: 809 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1976305351

Comment:
>I'm not sure about this.

You missed a rather important bit: all the `*.bdict` dictionaries are generate automatically by `dooble` build system.

Modify the template by adding this to `do_install()`
```sh
vcopy qtwebengine_dictionaries usr/share/dooble
```
then add this patch
```diff
--- a/Source/dooble_settings.cc
+++ b/Source/dooble_settings.cc
@@ -2271,7 +2271,7 @@ void dooble_settings::show_qtwebengine_dictionaries_warning_label(void)
 
   if(bytes.trimmed().isEmpty())
     {
-      bytes = "qtwebengine_dictionaries";
+      bytes = "/usr/share/dooble/qtwebengine_dictionaries";
 
       if(!QFileInfo(bytes).isReadable())
 	{
```

Re: Package request: qtwebengine_dictionaries

[Eloitor]

[-- Attachment #1: Type: text/plain, Size: 184 bytes --]

New comment by Eloitor on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1976308530

Comment:
Wow thank you! I didn't realized that :)

Re: Package request: qtwebengine_dictionaries

[Eloitor]

[-- Attachment #1: Type: text/plain, Size: 411 bytes --]

New comment by Eloitor on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1977177645

Comment:
Looking at https://doc.qt.io/qt-6/qtwebengine-features.html#spellchecker maybe it is better to add the qtwebengine_dictionaries as a separate package and install them to `QT_INSTALL_PREFIX/qtwebengine_dictionaries`...

Is `QT_INSTALL_PREFIX` just `/usr/share/qt6` ?

Re: Package request: qtwebengine_dictionaries

[Eloitor]

[-- Attachment #1: Type: text/plain, Size: 532 bytes --]

New comment by Eloitor on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1977177645

Comment:
Looking at https://doc.qt.io/qt-6/qtwebengine-features.html#spellchecker maybe it is better to add the qtwebengine_dictionaries as a separate package and install them to `QT_INSTALL_PREFIX/qtwebengine_dictionaries`...

Is `QT_INSTALL_PREFIX` just `/usr/share/qt6` ? I think not... I think it is just `/usr`?? But installing the dictionaries in `/usr/qtwebengine_dictionaries` is odd....

Re: Package request: qtwebengine_dictionaries

[sgn]

[-- Attachment #1: Type: text/plain, Size: 204 bytes --]

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1977817284

Comment:
https://groups.google.com/g/linux.debian.bugs.dist/c/dvznQNLhfIQ

Re: Package request: qtwebengine_dictionaries

[Eloitor]

[-- Attachment #1: Type: text/plain, Size: 524 bytes --]

New comment by Eloitor on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978205000

Comment:
```
$ qmake6 -query | grep QT_INSTALL_DATA 
QT_INSTALL_DATA:/usr/share/qt6
```
So the dictionaries could be placed in `/usr/share/qt6/qtwebengine_dictionaries`.

**Do we have any package with qt5 which could potentially use those dictionaries?** If not, we could not care about symlinking them to the qt5 folder.

Should we name the packages like `qtwebengine-dict-en-us`?

Re: Package request: qtwebengine_dictionaries

[chrysos349]

[-- Attachment #1: Type: text/plain, Size: 788 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978305319

Comment:
archlinux has `*.bdic` in `hunspell-<LANG>` packages:
See here - https://gitlab.archlinux.org/archlinux/packaging/packages/hunspell-de/-/blob/main/PKGBUILD?ref_type=heads#L64

```
# Install webengine dictionaries
  install -d "$pkgdir"/usr/share/qt{,6}/qtwebengine_dictionaries/
  for _file in "$pkgdir"/usr/share/hunspell/*.dic; do
  _filename=$(basename $_file)
    /usr/lib/qt6/qwebengine_convert_dict $_file "$pkgdir"/usr/share/qt6/qtwebengine_dictionaries/${_filename/\.dic/\.bdic}
  ln -rs "$pkgdir"/usr/share/qt6/qtwebengine_dictionaries/${_filename/\.dic/\.bdic} "$pkgdir"/usr/share/qt/qtwebengine_dictionaries/
  done
```

Re: Package request: qtwebengine_dictionaries

[chrysos349]

[-- Attachment #1: Type: text/plain, Size: 1371 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978409304

Comment:
In a similar manner `hunspell-<LANG>` templates can be modified, e.g. `hunspell-en_US`:
```diff
diff --git a/srcpkgs/hunspell-en_US/template b/srcpkgs/hunspell-en_US/template
index ce4df5e1a..2d60aa496 100644
--- a/srcpkgs/hunspell-en_US/template
+++ b/srcpkgs/hunspell-en_US/template
@@ -3,7 +3,7 @@ pkgname=hunspell-en_US
 version=2020.12.07
 revision=1
 create_wrksrc=yes
-hostmakedepends="unzip"
+hostmakedepends="unzip qt6-webengine"
 short_desc="American English en_US dictionary for hunspell"
 maintainer="Leah Neukirchen <leah@vuxu.org>"
 license="SCOWL" # not in SPDX
@@ -15,4 +15,15 @@ do_install() {
 	vinstall en_US.aff 644 /usr/share/hunspell
 	vinstall en_US.dic 644 /usr/share/hunspell
 	vlicense README_en_US.txt SCOWL
+	
+	# add qtwebengine_dictionaries
+	vmkdir /usr/share/qt6/qtwebengine_dictionaries
+	vmkdir /usr/share/qt5/qtwebengine_dictionaries
+	for f in ${DESTDIR}/usr/share/hunspell/*.dic; do
+		fa=$(basename $f .dic)
+		/usr/lib/qt6/libexec/qwebengine_convert_dict $f \
+			${DESTDIR}/usr/share/qt6/qtwebengine_dictionaries/${fa}.bdic
+		ln -s /usr/share/qt6/qtwebengine_dictionaries/${fa}.bdic \
+			${DESTDIR}/usr/share/qt5/qtwebengine_dictionaries
+	done
 }
```

Re: Package request: qtwebengine_dictionaries

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 268 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978420884

Comment:
it can't be exactly like that, as that would make it impossible to build on architectures that can't have qt6-webengine

Re: Package request: qtwebengine_dictionaries

[chrysos349]

[-- Attachment #1: Type: text/plain, Size: 197 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978431075

Comment:
you are correct. forgot about those architectures.

Re: Package request: qtwebengine_dictionaries

[chrysos349]

[-- Attachment #1: Type: text/plain, Size: 308 bytes --]

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978431075

Comment:
you are correct. forgot about those architectures.

UPD. What about enabling only when crossbuilding?
```sh
if [ "$CROSS_BUILD" ]; then
        ...
fi
```

Re: Package request: qtwebengine_dictionaries

[sgn]

[-- Attachment #1: Type: text/plain, Size: 197 bytes --]

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978920894

Comment:
Can be built by qt5-webengine (it's only needed in host).

Re: Package request: qtwebengine_dictionaries

[sgn]

[-- Attachment #1: Type: text/plain, Size: 311 bytes --]

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1978926773

Comment:
Another possible approach is `xbps-triggers` that will regenerate `.bdic` files upon installation into `/usr/share/hunspell` and/or `qt5-webengine` and/or `qt6-webengine`.

Re: Package request: qtwebengine_dictionaries

[sgn]

[-- Attachment #1: Type: text/plain, Size: 194 bytes --]

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/issues/49030#issuecomment-1985658881

Comment:
https://github.com/void-linux/void-packages/pull/49189

Re: [ISSUE] [CLOSED] Package request: qtwebengine_dictionaries

[sgn]

[-- Attachment #1: Type: text/plain, Size: 1070 bytes --]

Closed issue by Eloitor on void-packages repository

https://github.com/void-linux/void-packages/issues/49030

Description:
### Package name

qtwebengine_dictionaries

### Package homepage

https://doc.qt.io/qt-6.6/qt-add-webengine-dictionary.html

### Description

I'm not sure about this. The use case is to provide spell checking for the dooble Browser.
1. Install and launch the `dooble` browser.
2. Edit -> Settings -> Web -> Spell Checkers

You get a warning that `qtwebengines_dictionary` cannot be accessed. I searched using `xlocate "\.bdic"` and didn't find any package that provided such (only `Grammalecte` for french).

Reading from https://doc.qt.io/qt-6.6/qt-add-webengine-dictionary.html

> A spell checker in Qt Web Engine needs dictionaries in a specific binary format. This CMake command converts dictionaries from the [Hunspell project](https://sourceforge.net/projects/hunspell/). into the bdict binary format.



### Does the requested package meet the package requirements?

System, Compiled

### Is the requested package released?

Yes