[MacPorts] #39627: new: py-lmfit

MacPorts noreply at macports.org
Tue Jan 21 10:42:54 PST 2014


#39627: new:  py-lmfit
--------------------------+--------------------------------
  Reporter:  jjstickel@…  |      Owner:  macports-tickets@…
      Type:  submission   |     Status:  new
  Priority:  Normal       |  Milestone:
 Component:  ports        |    Version:  2.1.3
Resolution:               |   Keywords:
      Port:  py-lmfit     |
--------------------------+--------------------------------

Comment (by g5pw@…):

 Thanks for the submission!

 Why is your portfile declaring a variant and then just adding a port
 dependency? I'ts better practice to force the setup script to actually use
 the port you list as dependency in a variant. Moreover, it looks like the
 {{{uncertainties}}} package is used just at runtime, so I think users can
 install the necessary package from MacPorts after installing {{{lmfit}}}.
 So, declaring a variant is pointless in my opinion, you can add a note
 about {{{uncertainties}}} if you want, but I wouldn't add a variant since
 the installed files are the same.

 Also, your port is missing a platform keyword. Use the {{{port lint}}}
 command to check the port file in the future.

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


More information about the macports-tickets mailing list