Bug 200743

Summary: [MAINTAINER] games/minecraft-server: rc.d improvements
Product: Ports & Packages Reporter: Jonathan Price <freebsd>
Component: Individual Port(s)Assignee: Bartek Rutkowski <robak>
Status: Closed FIXED    
Severity: Affects Only Me CC: robak
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch file
freebsd: maintainer-approval+
Poudriere testport log
freebsd: maintainer-approval+
portlint -A output
freebsd: maintainer-approval+
Patch File
freebsd: maintainer-approval+
portlint -A output
freebsd: maintainer-approval+
Poudriere testport log freebsd: maintainer-approval+

Description Jonathan Price 2015-06-09 20:01:13 UTC
Created attachment 157580 [details]
Patch file

A few improvements have been made to the rc.d script:
- reduced code duplication by adding a function that checks if the server is running, instead of re-writing the check in each other function.
- added "kill" option, which sends a KILL signal to the server.
- Improved the grammar of a few of the messages

The following files have been attached:
- SVN patch
- portlint -A output
- poudriere testport log

The change has been tried with both new and existing installations
Comment 1 Jonathan Price 2015-06-09 20:03:03 UTC
Created attachment 157581 [details]
Poudriere testport log
Comment 2 Jonathan Price 2015-06-09 20:03:38 UTC
Created attachment 157582 [details]
portlint -A output
Comment 3 Jonathan Price 2015-06-10 08:50:37 UTC
Created attachment 157603 [details]
Patch File

Incremeneted PORTREVISION
Comment 4 Jonathan Price 2015-06-10 08:52:13 UTC
Created attachment 157604 [details]
portlint -A output

Revised portlint after modification to patch.
Comment 5 Jonathan Price 2015-06-10 08:53:03 UTC
Created attachment 157605 [details]
Poudriere testport log

Revised Poudriere testport log after revision to Patch file.
Comment 6 commit-hook freebsd_committer freebsd_triage 2015-06-26 16:23:25 UTC
A commit references this bug:

Author: robak
Date: Fri Jun 26 16:22:57 UTC 2015
New revision: 390648
URL: https://svnweb.freebsd.org/changeset/ports/390648

Log:
  games/minecraft-server: improvements to startup script

  PR:		200743
  Submitted by:	Jonathan Price  <freebsd@jonathanprice.org> (maintainer)

Changes:
  head/games/minecraft-server/Makefile
  head/games/minecraft-server/files/minecraft.in
Comment 7 Bartek Rutkowski freebsd_committer freebsd_triage 2015-06-26 16:27:30 UTC
Committed, thanks for your work!