[MacPorts] #36135: gnuplot enhancement to support emacs-app

[MacPorts]

#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=102055
 >
 > 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