[MacPorts] #43574: [NEW] honeysnap

MacPorts noreply at macports.org
Mon May 5 00:13:57 PDT 2014


#43574: [NEW] honeysnap
-------------------------+--------------------------------
  Reporter:  jul_bsd@…   |      Owner:  macports-tickets@…
      Type:  submission  |     Status:  new
  Priority:  Normal      |  Milestone:
 Component:  ports       |    Version:
Resolution:              |   Keywords:
      Port:  honeysnap   |
-------------------------+--------------------------------

Comment (by ryandesign@…):

 Thanks, but this does not appear to be ready to commit yet.

  * The indentation is not consistent (PortGroup python line, post-destroot
 block).
  * There are many python variants, which is good, but only the python27
 variant lists its dependencies; seems like each python variant should list
 its dependencies.
  * The port printed the following notes:

 {{{
 --->  Some of the ports you installed have notes:
   honeysnap has the following notes:
     To run honeysnap, you can:
     $
 /opt/local/Library/Frameworks/Python.framework/Versions/2.x/bin/honeysnap
 -c honeynet.cfg myfile.pcap
 }}}

 However that file does not exist:

 {{{
 $
 /opt/local/Library/Frameworks/Python.framework/Versions/2.x/bin/honeysnap
 -c honeynet.cfg myfile.pcap
 -bash:
 /opt/local/Library/Frameworks/Python.framework/Versions/2.x/bin/honeysnap:
 No such file or directory
 }}}

 Here's what does exist:

 {{{
 $ port contents honeysnap | grep /bin/
 /opt/local/Library/Frameworks/Python.framework/Versions/2.7/bin/honeysnap
 /opt/local/Library/Frameworks/Python.framework/Versions/2.7/bin/thresholdflows
 /opt/local/Library/Frameworks/Python.framework/Versions/2.7/bin/xBytesNseconds
 }}}

 The port should probably create a symlink ${prefix}/bin/honeysnap pointing
 to wherever the file got installed, to make it easier for the user to run
 it. If the other two programs are ones a user would want to run, then
 symlinks should be created for them too.

 The variant descriptions describe that they will build e.g. "Python 2.7
 bindings", but this sounds like the software actually requires python
 itself; it's not just bindings. The description should be changed, e.g. to
 "Use Python 2.7".

 I would be inclined to change the order in which the variants are defined,
 the order in which their conflicts are listed, and the order in which the
 variants are checked before setting the default variant, to alphabetical
 order.

-- 
Ticket URL: <https://trac.macports.org/ticket/43574#comment:1>
MacPorts <http://www.macports.org/>
Ports system for OS X


More information about the macports-tickets mailing list