Bug 200476

Summary: [revive port] sysutils/jailadmin: Jail management system with optional SNMP support
Product: Ports & Packages Reporter: Chris Hutchinson <portmaster>
Component: Individual Port(s)Assignee: Matthias Andree <mandree>
Status: Closed FIXED    
Severity: Affects Only Me CC: mandree, portmaster
Priority: --- Keywords: easy, patch, patch-ready
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
sysutils/jailadmin ports files in shar format
none
QA log for sysutils/jailadmin
none
svn diff for sysutils/jailadmin
none
svn diff for revival of sysutils/jailadmin none

Description Chris Hutchinson 2015-05-26 22:31:11 UTC
Created attachment 157182 [details]
sysutils/jailadmin ports files in shar format

This is an effort to reintroduce sysutils/jailadmin to the
ports tree. I really prefer this to easyjail, as it feels
closer to "traditional" jail management, and comes with
optional SNMNP support.
The original author indicates he's no longer maintaining it.
So I've adopted it. Currently in the pipeline; GTK bindings,
which will provide for config file creation dialog. As well
as a better statistics table display.
Changes;
pretty much the entire ports files, as well as the removal
of the pkg-plist file.

Please find the complete ports file, in shar(1) format,
as well as the requisite QA session log, attached to this
pr(1).

Thanks!

--Chris
Comment 1 Chris Hutchinson 2015-05-26 22:31:58 UTC
Created attachment 157183 [details]
QA log for sysutils/jailadmin
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2015-11-26 08:15:22 UTC
@Chris, It might be worth providing an svn diff for this, and a copy of the (svn copy) command you ran that copied the last revision of the port before it was deleted back into the tree. For more see:

https://www.freebsd.org/doc/en_US.ISO8859-1/articles/committers-guide/ports.html#ports-qa-resurrect
Comment 3 Chris Hutchinson 2015-11-27 19:09:25 UTC
Created attachment 163590 [details]
svn diff for sysutils/jailadmin

Same content as before, but in different format.
This svn diff resurrects sysutils/jailadmin.

Thanks!

--Chris
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2015-11-28 02:35:58 UTC
(In reply to Chris Hutchinson from comment #3)

And the svn copy command you ran to copy it back in? :)
Comment 5 Chris Hutchinson 2015-11-29 21:12:19 UTC
(In reply to Kubilay Kocak from comment #4)
> (In reply to Chris Hutchinson from comment #3)
> 
> And the svn copy command you ran to copy it back in? :)

The information provided in the committers guide, linked to
in <a href="#c2">comment 2</a> is incorrect;
 * repo.freebsd.org does not exist
 * http://people.freebsd.org/~crees/removed_ports/index.xml ,
   does not list the port which I svn co'd out of 2014Q3
so I know it exists.

It appears it is no longer possible to resurrect ports.
shar(1) used to be possible, and always worked. Maybe we
can adopt that procedure, again. As it was more resilient
to failure, and better documented, than what's currently offered.

--Chris
Comment 6 Mark Linimon freebsd_committer freebsd_triage 2015-11-29 22:53:13 UTC
Please try svn.freebsd.org for the repository and report back if it works.  If so, I will fix the stale documentation.

ports/MOVED has:

sysutils/jailadmin||2014-09-01|Not staged

Some detective work with svnweb shows the actual change was:

https://svnweb.freebsd.org/ports?view=revision&revision=366980

and clicking on sysutils/jailadmin in that link takes us to:

https://svnweb.freebsd.org/ports/head/sysutils/jailadmin/?pathrev=366979

from which the port can be reconstituted.
Comment 7 Chris Hutchinson 2015-11-30 04:02:10 UTC
(In reply to Mark Linimon from comment #6)
Thank you kindly for the prompt reply, Mark.

> Please try svn.freebsd.org for the repository and report back if it works. 
Yes. svn.freebsd.org works perfectly for me.

> If so, I will fix the stale documentation.
> 
> ports/MOVED has:
> 
> sysutils/jailadmin||2014-09-01|Not staged
> 
> Some detective work with svnweb shows the actual change was:
> 
> https://svnweb.freebsd.org/ports?view=revision&revision=366980
> 
> and clicking on sysutils/jailadmin in that link takes us to:
> 
> https://svnweb.freebsd.org/ports/head/sysutils/jailadmin/?pathrev=366979
> 
> from which the port can be reconstituted.

Perfect. This will do it.
Thank you again, Mark!

--Chris
Comment 8 Chris Hutchinson 2015-11-30 05:26:48 UTC
Created attachment 163672 [details]
svn diff for revival of sysutils/jailadmin

This, as I understand it, should be what's needed to revive
sysutils/jailadmin. Thanks again, to Mark, for the assistance.

Thanks to all!

--Chris
Comment 9 commit-hook freebsd_committer freebsd_triage 2016-02-05 23:29:46 UTC
A commit references this bug:

Author: mandree
Date: Fri Feb  5 23:28:46 UTC 2016
New revision: 408277
URL: https://svnweb.freebsd.org/changeset/ports/408277

Log:
  Revive and update sysutils/jailadmin port.

  Simplify Makefile a bit for new options framework, remove port from MOVED.

  PR:		200476
  Submitted by:	Chris Hutchinson

Changes:
  head/MOVED
  head/sysutils/Makefile
  head/sysutils/jailadmin/
  head/sysutils/jailadmin/Makefile
  head/sysutils/jailadmin/distinfo
  head/sysutils/jailadmin/pkg-descr
  head/sysutils/jailadmin/pkg-plist
Comment 10 Matthias Andree freebsd_committer freebsd_triage 2016-02-05 23:31:05 UTC
Hi Chris,

thanks for doing the work. 

I have changed a few bits about the Makefile to simplify it, given the extended options frameworks. Please have a look. If you're unhappy with my changes, shout at me and we can fix that up.

I also considered adding a @sample etc/jailadmin.conf.sample back to pkg-plist, but refrained from that because that creates demo servers that may be unfit.

Thanks again!
Comment 11 Chris Hutchinson 2016-02-06 01:56:03 UTC
(In reply to Matthias Andree from comment #10)
> Hi Chris,
> 
> thanks for doing the work. 
> 
> I have changed a few bits about the Makefile to simplify it, given the
> extended options frameworks. Please have a look. If you're unhappy with my
> changes, shout at me and we can fix that up.
> 
> I also considered adding a @sample etc/jailadmin.conf.sample back to
> pkg-plist, but refrained from that because that creates demo servers that
> may be unfit.
> 
> Thanks again!

Just had a look. I have no problem with your changes,
they look great. @sample; I refrained from adding it
for the same reason. :)
I'll try and whip up a couple of suitable examples,
and add them to the next version.

Thanks, Matthias!

--Chris