Opened 15 years ago
Closed 15 years ago
#21527 closed defect (fixed)
bluefish-devel: uses enchant if present
| Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | macports-tickets@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 1.8.0 |
| Keywords: | Cc: | nerdling (Jeremy Lavergne) | |
| Port: | bluefish-devel |
Description
bluefish-devel uses the libraries and headers installed by the enchant port if it is installed. bluefish-devel must be modified to either depend on enchant, or not use enchant even if it is already installed.
Change History (5)
comment:1 Changed 15 years ago by pf_moll@…
comment:2 follow-up: 3 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
I mean only that a port (like bluefish-devel) is not supposed to use software installed by another port (like enchant) unless it declares a dependency on it. So either bluefish-devel must declare a dependency on enchant, or bluefish-devel must not use enchant.
Adding the configure arg --disable-spell-check does not seem to prevent bluefish-devel from attempting to use enchant.
comment:3 follow-up: 4 Changed 15 years ago by pf_moll@…
Replying to ryandesign@…:
I mean only that a port (like bluefish-devel) is not supposed to use software installed by another port (like enchant) unless it declares a dependency on it. So either bluefish-devel must declare a dependency on enchant, or bluefish-devel must not use enchant.
Adding the configure arg --disable-spell-check does not seem to prevent bluefish-devel from attempting to use enchant.
Would it be enough to add a depends-lib to the portfile, or should this be reported upstream?
Off Topic, but I would like to submit a patch to the portfile to change it to 1.3.7 which just came out, what I get from the manual, this is just a new ticket?
comment:4 Changed 15 years ago by pf_moll@…
Ok, just uploaded a diff to update to 1.3.7 and made it dependent on libenchant as well. Also posted a bug upstream about the --disable-spell-check being ignored. This will be fixed in the next release, so then it is possible if wanted to make a variant without spell checking. Hope this is what you meant:-)
comment:5 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | snc@… added |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
Sounds good. The enchant dependency was added in r58465.
Replying to ryandesign@…:
Do you mean it should give an error if it wants to use enchant and it is not installed? To not use it ( from what I understand of it ) you can pass --disable-spell-check to configure. Or do you mean something else?