Bug 221839 - sysutils/qjail [Maintainer update] qjail-5.4
Summary: sysutils/qjail [Maintainer update] qjail-5.4
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Adam Weinberger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-26 19:31 UTC by Joe Barbish
Modified: 2017-09-25 12:17 UTC (History)
1 user (show)

See Also:

Attachments
port make files diff (6.87 KB, text/plain)
2017-08-26 19:31 UTC, Joe Barbish 		no flags Details
port diff (841 bytes, patch)
2017-08-27 00:12 UTC, Joe Barbish 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joe Barbish 2017-08-26 19:31:50 UTC 
Created attachment 185794 [details]
port make files diff

Aaron Sachau sent email about bug he found. The "qjail console jailname" command that has a jail with a fib assigned to it was not working correctly. This error was introduced by changes done to correct coding syntax problems that the 
hs-shellcheck package found and corrected and then published in qjail-5.2. 
Reverted back to how qjail-5.1 did it. Aaron Sachau tested the patch. Fixed now.
Comment 1 Adam Weinberger freebsd_committer freebsd_triage 2017-08-26 20:22:19 UTC 
Hi Joe, can you please regenerate this diff as an actual diff against the current port?
Comment 2 Joe Barbish 2017-08-27 00:12:01 UTC 
Created attachment 185796 [details]
port diff

this should be the correct content
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-08-27 01:14:46 UTC 
A commit references this bug:

Author: adamw
Date: Sun Aug 27 01:14:01 UTC 2017
New revision: 448797
URL: https://svnweb.freebsd.org/changeset/ports/448797

Log:
  Update to 5.4.

  Aaron Sachau sent email about bug he found. The "qjail console jailname"
  command that has a jail with a fib assigned to it was not working correctly.
  This error was introduced by changes done to correct coding syntax problems
  that the hs-shellcheck package found and corrected and then published in
  qjail-5.2. Reverted back to how qjail-5.1 did it. Aaron Sachau tested the
  patch. Fixed now.

  PR:		221839
  Submitted by:	maintainer (Joe Barbish)

Changes:
  head/sysutils/qjail/Makefile
  head/sysutils/qjail/distinfo
Comment 4 Adam Weinberger freebsd_committer freebsd_triage 2017-08-27 01:15:10 UTC 
Committed. Thanks, Joe!
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-09-25 12:17:37 UTC 
A commit references this bug:

Author: adamw
Date: Mon Sep 25 12:17:28 UTC 2017
New revision: 450586
URL: https://svnweb.freebsd.org/changeset/ports/450586

Log:
  MFH: r448797

  Update to 5.4.

  Aaron Sachau sent email about bug he found. The "qjail console jailname"
  command that has a jail with a fib assigned to it was not working correctly.
  This error was introduced by changes done to correct coding syntax problems
  that the hs-shellcheck package found and corrected and then published in
  qjail-5.2. Reverted back to how qjail-5.1 did it. Aaron Sachau tested the
  patch. Fixed now.

  PR:		221839
  Submitted by:	maintainer (Joe Barbish)

  Approved by:	ports-secteam (delphij)

Changes:
_U  branches/2017Q3/
  branches/2017Q3/sysutils/qjail/Makefile
  branches/2017Q3/sysutils/qjail/distinfo