Opened 7 years ago
Closed 3 years ago
#54944 closed defect (fixed)
inkscape @0.92.2: library dependency aspell prevents build of hunspell-en_US and hunspell-de_DE due to conflict with ispell
Reported by: | maehne (Torsten Maehne) | Owned by: | dbevans (David B. Evans) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | Schamschula (Marius Schamschula), udbraumann | |
Port: | inkscape inkscape-devel |
Description
The build of hunspell-en_US
and hunspell-de_DE
requires the port ispell
to be installed. Unfortunately, inkscape
forces the installation of port aspell
by specifying it as a library dependency. However, this direct dependency on aspell
doesn't seem to be necessary because according to the inkscape wiki, the spell checking is done through gtkspell
. This is already reflected in the inkscape
port through the library dependency on gtkspell2
. gtkspell2
itself does use enchant
for unified access to different spell checkers (applespell
, aspell
, ispell
, myspell
). Is there another reason to force installation of aspell
or can the dependency simply dropped?
Attachments (1)
Change History (14)
comment:1 Changed 7 years ago by mf2k (Frank Schima)
Cc: | dbevans removed |
---|---|
Keywords: | aspell gtkspell2 enchant aspell hunspell removed |
Owner: | set to dbevans |
Port: | inkscape-devel added |
Status: | new → assigned |
comment:2 Changed 7 years ago by Schamschula (Marius Schamschula)
comment:3 Changed 7 years ago by Schamschula (Marius Schamschula)
Cc: | Schamschula added |
---|
comment:4 Changed 7 years ago by raimue (Rainer Müller)
Inkscape links with aspell, so the dependency is in fact correct:
$ otool -L /opt/local/bin/inkscape |grep aspell /opt/local/lib/libaspell.15.dylib (compatibility version 17.0.0, current version 17.5.0)
For reference, the reasons for this dependency were discussed previously in ticket:51287#comment:18, which also contains more details on spellchecking in inkscape.
comment:5 Changed 7 years ago by maehne (Torsten Maehne)
Thanks a lot for your feedback to my bug report and pointing out why aspell is currently needed for the inkscape port! I hope that the issue will be cleaned up in the upstream implementation some day in the not too far future. Thanks also for the workaround of deactivating aspell for installing hunspell-de_DE. I did that, but found it inconvenient enough to report this issue.
comment:6 Changed 7 years ago by udbraumann
Cc: | udbraumann added |
---|
comment:7 follow-up: 8 Changed 7 years ago by udbraumann
I wonder why you think that hunspell-en_US
is requiring ispell
. In fact it requires aspell
. However, I noticed some serious trouble on my 10.5.8 PPC system while upgrading hunspell-dict-en_US @2006-02-07_0
to hunspell-en_US @2017.08.24_1
:
... :info:build prep :info:build creating en_US.dic :info:build Can't locate object method "newdir" via package "File::Temp" at ../speller/munch-list line 47, <F> line 1. :info:build Can't locate object method "newdir" via package "File::Temp" at ../speller/munch-list line 47, <F> line 1. :info:build Command failed: cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_textproc_hunspell-en/hunspell-en_US/work/scowl-2017.08.24/speller" && ./make-hunspell-dict -all :info:build Exit code: 255 :error:build Failed to build hunspell-en_US: command execution failed :debug:build Error code: CHILDSTATUS 16647 255 ...
To be honest, I have no idea the "newdir"
method is missing. I attach the respective main.log.
comment:8 follow-up: 9 Changed 7 years ago by Schamschula (Marius Schamschula)
Replying to udbraumann:
I wonder why you think that
hunspell-en_US
is requiringispell
. In fact it requiresaspell
. However, I noticed some serious trouble on my 10.5.8 PPC system while upgradinghunspell-dict-en_US @2006-02-07_0
tohunspell-en_US @2017.08.24_1
:... :info:build prep :info:build creating en_US.dic :info:build Can't locate object method "newdir" via package "File::Temp" at ../speller/munch-list line 47, <F> line 1. :info:build Can't locate object method "newdir" via package "File::Temp" at ../speller/munch-list line 47, <F> line 1. :info:build Command failed: cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_textproc_hunspell-en/hunspell-en_US/work/scowl-2017.08.24/speller" && ./make-hunspell-dict -all :info:build Exit code: 255 :error:build Failed to build hunspell-en_US: command execution failed :debug:build Error code: CHILDSTATUS 16647 255 ...To be honest, I have no idea the
"newdir"
method is missing. I attach the respective main.log.
This should be a separate ticket, as it does not pertain to inkscape
.
The issue in this ticket is that the hunspell-en_US
dictionary requires aspell
to be built, but inkscape
requires ispell
at runtime. aspell
conflicts with ispell
.
comment:9 Changed 7 years ago by udbraumann
Replying to Schamschula:
This should be a separate ticket, as it does not pertain to
inkscape
.The issue in this ticket is that the
hunspell-en_US
dictionary requiresaspell
to be built, butinkscape
requiresispell
at runtime.aspell
conflicts withispell
.
Well, this also means that the ticket title is wrong, or at least is misleading, as inscape
's library dependency aspell
does not prevent building of hunspell-en_US
, but only of hunspell-de_DE
(and other hunspell-de_*
)
I would open a new ticket if I would know some maintainer of hunspell-en_US
(or hunspell-en_*
).
comment:10 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Cc: | majoc-at-astro added |
---|
comment:11 Changed 7 years ago by majoc-at-astro (majoc-at-astro)
Cc: | majoc-at-astro removed |
---|
comment:12 Changed 3 years ago by Christopher Nielsen <mascguy@…>
comment:13 Changed 3 years ago by evanmiller (Evan Miller)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
This is an issue I ran into when first porting
hunspell-de_DE
. Unfortunately, even thoughaspell
also supplies anispell
binary, it is missing capabilities required to buildhunspell-de_DE
. I just temporarily deactivatedaspell
and activatedispell
.ispell
is only needed during installation, soaspell
can be reactivated afterward.