#36135: gnuplot enhancement to support emacs-app --------------------------+------------------------------------ Reporter: jrh@… | Owner: mojca.miklavec.lists@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch Port: gnuplot | --------------------------+------------------------------------ Comment (by jrh@…): Replying to [comment:2 mojca.miklavec.lists@…]:
Thanks a lot for the patch and diagnosis. I would like to try to avoid setting PATH and fix the configure script instead, and above all, to get the change upstream (the version 4.6.1 is soon going to be released, but the team of developers is usually reluctant to do any last-minute changes). [[BR]] Yeah, the exact nature of my changes didn't strike me as ideal, either, they're mostly due to the order in which I noticed problems. :-) [[BR]][[BR]]
https://sourceforge.net/tracker/?func=detail&aid=3567659&group_id=2055&atid=...
The patch testing for emacs vs. Emacs in lisp/configure is not really
needed since it just tries to patch emacs 20.2 and 20.3. My suggestion for the patch would be [snip]
This doesn't check for {{{Emacs}}}, but if needed, the test could be extended to check for both. However, this would only patch emacs 20.2/20.3, so I wouldn't really try to fix that as it doesn't bring anything. [[BR]] I'd have to try again with your modified patch, but as I recall, without the check for {{{Emacs}}}, I got a build failure. Something like "no rule to build info-look.elc". I think the problem is the {{{INFO_LOOK_ELC =info-look.elc}}} immediately above the binary name check. For Emacs.app we need to make sure that {{{INFO_LOOK_ELC}}} is cleared, as it is in the {{{"$vnum" -ge 2030}}} check. This is probably another bug with the configure script, actually. If I'm interpreting it correctly, this {{{ else info_look="using none" fi }}} should probably be {{{ else info_look="using none" INFO_LOOK_ELC= fi }}} instead. It seems like the contract is intended to be that if {{{INFO_LOOK_ELC}}} is not "", then {{{info-look.el}}} exists. [[BR]] [[BR]] Just curious: do you actually use this emacs extensions? [[BR]] To be honest, I didn't know it existed before discovering the gnuplot {{{emacs}}} variant yesterday. But I use Emacs every day and occasionally have to edit gnuplot scripts, so I intend to start using it!
-- Ticket URL: <https://trac.macports.org/ticket/36135#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS