This port contains all you need to play Cave Story. Inspiration for symlinking nx to home directory was taken from Arch Linux PKGBUILD.
Details
Details
- Reviewers
AMDmi3 matthew swills - Commits
- rP449871: new port: games/nxengine
poudriere testport - OK gameplay - OK
Diff Detail
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 11522 Build 11876: arc lint + arc unit
Event Timeline
| games/nxengine/files/nxengine.in | ||
|---|---|---|
| 6 | The construct "$HOME"/.nxengine looks odd to me -- most people would write that as | |
| 10 | Why is this in a subshell? The wrapper script doesn't do anything other than start cd "${HOME}/.nxengine"
exec ./nx "$@" | |
Comment Actions
address matthew's feedback: good point about just exec'ing nx
As for "$HOME"/.nxengine, I prefer this form for efficiency.
There is a whole lot of stuff curly brace expansion does and
in this case we don't use any of it. Likewise, I use double quotes
on just the shortest substring I want double quote expansion on.
If this is strage or hard to read, I'll change it.
Comment Actions
o unhardcode some values having to do with DISTFILES
o be more precise in the LICENSE block about which license applies to what
| games/nxengine/Makefile | ||
|---|---|---|
| 25 | License stuff looks fine. | |
| 37 | I won't reject for this, but I really wish people would do this kind of "patch via sed" less. It makes it easy for this kind of thing to go unnoticed if it happens to be come unnecessary. I've seen a number of these end up still in place ages after they were no longer needed and functionally useless. Also, it makes it almost impossible to submit to upstream or for upstream to even see what changes we're making if they did happen to look for patches. | |
| 40 | I think we want to not mute this? | |
Comment Actions
address swills' feedback:
migrate from "use sed to patch" to patch files for finding <sys/endian.h>
unmute DATADIR creation