[MacPorts] #54573: New port for Fricas (a Computer Algebra System)
MacPorts
noreply at macports.org
Mon Aug 7 05:29:13 UTC 2017
#54573: New port for Fricas (a Computer Algebra System)
-------------------------+-----------------
Reporter: pietvo | Owner:
Type: submission | Status: new
Priority: Normal | Milestone:
Component: ports | Version:
Resolution: | Keywords:
Port: fricas |
-------------------------+-----------------
Comment (by ryandesign):
Thanks. Some comments:
* If the GitHub handle and the obfuscated email address in the
`maintainers` line represent the same person, they should be grouped in
curly braces (`{``}`).
* `supported_archs` is a normal MacPorts "option", and as such the values
are separated by whitespace; there should not be a comma.
* `license` is also a normal "option", so MacPorts thinks this port is
under three licenses, called "Modified", "BSD", and "license", which isn't
what you wanted. If the software's license is sufficiently close to BSD,
particularly with regard to distributability, then just put "BSD",
otherwise just put "Permissive". If desired, you can clarify the license
in a comment line above the `license` line.
* Usually, we want rmd160 and sha256 checksums listed. Additional checksum
types may be added to those if upstream publishes them.
* `configure.cmd` and `build.cmd` should usually only be the actual
command to run. The directory in which they should run should be set
instead in `configure.dir` and `build.dir`. If you need to create the
build directory, do so in a `pre-configure` block. If you need to set
build args like setting the `MAYBE_VIEWPOINT` variable, use `build.args`.
* The x11 variant should not depend on xorg. Nothing should; the xorg port
is a convenience port enabling the user to install all of X at once. Ports
should instead depend on the specific ports providing the specific
libraries needed. You wrote the xorg dependency on the libX11 library.
That library is in fact provided by the xorg-libX11 port, so you should
depend on `port:xorg-libX11`, not `lib:libX11.6:xorg`. Note that you
should not use a `lib:`-style dependency because that would allow software
that wasn't installed with MacPorts to satisfy the dependency, which is
not usually what we want.
* It would be clearer if the code relating to the sbcl variant were
contained within the sbcl variant. This would eliminate the need for each
of the conflicting variants to remove the sbcl dependency and configure
arg.
* Similarly, it would be clearer if the post-patch code relating to the
gmp variant were contained within the gmp variant.
* The tabs should be converted to spaces and the whitespace fixed so that
more of the portfile is neatly aligned into two columns.
--
Ticket URL: <https://trac.macports.org/ticket/54573#comment:1>
MacPorts <https://www.macports.org/>
Ports system for macOS
More information about the macports-tickets
mailing list