Bug 124462
Summary: | ispell macro in joerc has several problems - fix to use aspell instead [has patch] | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Matthew Miller <mattdm> | ||||
Component: | joe | Assignee: | Lon Hohberger <lhh> | ||||
Status: | CLOSED RAWHIDE | QA Contact: | |||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | Keywords: | Patch | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2004-06-09 21:07:38 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Matthew Miller
2004-05-26 19:35:06 UTC
Created attachment 100603 [details]
patch to use mktemp (and aspell)
Pushed upstream. http://sourceforge.net/mailarchive/forum.php?forum_id=2185&max_rows=25&style=ultimate&viewmonth=200405 I suggested it to the author as well. However, I'm not sure that this one will be accepted upstream, because some places where joe is in use probably actually *do* have ispell. Changing to use aspell is a distro-dependent config. Likewise, not everywhere will have "mktemp -t", but that's the right thing to do for Fedora / Red Hat. And, I'm not sure how long it will be until upstream would accept and release a new version, even if they do. The mktemp issue is a security vulnerability (admitably minor) and could be used as a symlink attack, so I'd like to see that included sooner rather than later. And while doing that, might as well change to using aspell to clean up the .bak file problem. Yeah, a bit of this seems to have gone into 3.1 --it now uses my "SPLTMP" variable, but sets it explicitly to "ispell.tmp". Using mktemp -t is better. I think also that the upstream change has made it so aspell is a buildprereq for the package -- if aspell isn't found, it'll assume ispell. joe-3.1-1.2 should appear in rawhide soon. Cool. Will you patch it to use the mktemp thing? Yes. Cool. Best Editor Ever. :) Thanks! |