[R-pkg-team] Bug#1028130: r-cran-hunspell: please don't use internal en_US and en_GB dictionaries

Andreas Tille andreas at an3as.eu
Tue Jan 10 07:01:31 GMT 2023


Hi Rene,

Am Mon, Jan 09, 2023 at 09:40:17PM +0100 schrieb Rene Engelhard:
> 
> Of course that now happens if you build against sid.
> 
> The private headers now are for 1.7.2 - which as of now is in experimental
> and thus you need to build against 1.7.2 (and probably have a updated
> runtime dependency.)

Ah, sorry for missing this.
 
> Which is what my patch does in addition to the headers only. You just
> applied the header change in https://salsa.debian.org/r-pkg-team/r-cran-hunspell/-/commit/ae8796cc2e2b72372b55cf40ce35a15bf00ae2d8,
> neither the build-dependency nor the runtime dependency. Do that and build
> against hunspell 1.7.2 and it will build.
> 
> 
> The problem per se is that you copy private stuff over.

I'm aware that this is not a nice solution - I considered a bit better
then keeping a full code copy of hunspell which is shipped by upstream.
On the other hand this would saved us some work now.
 
After merging your MR the package builds.  However, I get two failures
when running its test suite:


══ Failed tests ════════════════════════════════════════════════════════════════
── Failure ('test-encodings.R:16'): Dictionaries are found ─────────────────────
hunspell_info("en_US")$wordchars not equal to "’".
1/1 mismatches
x[1]: "0123456789’"
y[1]: "’"
── Failure ('test-encodings.R:17'): Dictionaries are found ─────────────────────
hunspell_info("en_GB")$wordchars not equal to "’".
1/1 mismatches
x[1]: "0123456789’"
y[1]: "’"

[ FAIL 2 | WARN 0 | SKIP 0 | PASS 126 ]
Error: Test failures

Kind regards

      Andreas.

-- 
http://fam-tille.de



More information about the R-pkg-team mailing list