New port: emulators/edumips64: Free, cross-platform, educational, visual MIPS64 CPU simulator
ClosedPublic
Actions

Authored by 0mp on May 2 2019, 3:58 PM.

Details

Reviewers

tobik

Commits

rP500854: New port: emulators/edumips64: Free, cross-platform, educational, visual MIPS64…

Summary

New port: emulators/edumips64: Free, cross-platform, educational, visual MIPS64 CPU simulator

EduMIPS64 is a free (as in free speech) visual and cross-platform MIPS64 CPU
Simulator.

The simulator was written in order to support Computer Architecture students
with a free and cross-platform tool that allows them to run programs written
in the MIPS64 assembly dialect and see how the CPU behaves during the
execution, experimenting with various aspects of low-level programming
that they learn throughout the course.

EduMIPS64 started as a GPL Java port of WinMIPS64, a Windows-only MIPS64
simulator, but independently evolved soon after into a stand-alone
open source application.

WWW: https://www.edumips.org/

Test Plan

portlint
testport on 11.2 i386
manual testing on 13.0 amd64; the port runs just fine

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

0mp created this revision.May 2 2019, 3:58 PM

Herald added a subscriber: mat. · View Herald TranscriptMay 2 2019, 3:59 PM

Harbormaster completed remote builds in B24068: Diff 56969.May 2 2019, 3:59 PM

0mp added a subscriber: Contributor Reviewers (ports).May 2 2019, 4:03 PM

tobik added a subscriber: tobik.May 2 2019, 4:51 PM

tobik added inline comments.

emulators/edumips64/Makefile
8–9 ↗	(On Diff #56969)	Nitpick: The manuals are only installed with DOCS=on, so should be behind DOCS_DISTFILES.
18 ↗	(On Diff #56969)	No point in settings this. JAVA_RUN=jre is the default.
28 ↗	(On Diff #56969)	JAVAJARDIR is already in SUB_LIST by default.
37 ↗	(On Diff #56969)	`${STAGEDIR}${PREFIX}`
41 ↗	(On Diff #56969)	While not all maintainers really do it, the PHB says to use INSTALL_MAN here. https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/install.html The two `${INSTALL_*}` can be combined into one call.
emulators/edumips64/files/pkg-message.in
1–7 ↗	(On Diff #56969)	Is this message really needed? It does not tell me much. That `edumips64` is startable with `edumips64` is kind of expected and there does not appear to be anything special to configure after installation. The port should install a .desktop file (e.g. via DESKTOP_ENTRIES) for users to start it from their desktops. Especially since this appears to have a GUI.