Bug 201717

Summary: sysutils/screen example screenrc contains incorrect command
Product: Ports & Packages Reporter: David Maxwell <davidmaxwell>
Component: Individual Port(s)Assignee: Cy Schubert <cy>
Status: Closed FIXED    
Severity: Affects Some People    
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   

Description David Maxwell 2015-07-20 14:54:35 UTC 
sysutils/screen/files/screenrc.sample, which gets installed in /usr/local/etc
includes the following (last) line:

logfile flush

The logfile directive in screen has two forms, one which takes a filename argument (the file name to log to when ^aH logging is turned on), and one which takes a secondary keyword flush and a time in seconds of how often to flush.

By including flush without the secondary argument, the sample rc file sets the logfile name for all screens to $HOME/flush (lacking a '%n' in the argument, it won't create separate files for each screen either).

This is probably not what was intended.
Comment 1 commit-hook freebsd_committer freebsd_triage 2015-07-22 03:14:43 UTC 
A commit references this bug:

Author: cy
Date: Wed Jul 22 03:14:32 UTC 2015
New revision: 392658
URL: https://svnweb.freebsd.org/changeset/ports/392658

Log:
  Remove incorrect screenrc command.

  PR:		201717

Changes:
  head/sysutils/screen/Makefile
  head/sysutils/screen/files/screenrc.sample
Comment 2 Cy Schubert freebsd_committer freebsd_triage 2015-07-22 03:16:19 UTC 
Removed the offending command. Fixed.
Comment 3 commit-hook freebsd_committer freebsd_triage 2015-09-08 16:45:32 UTC 
A commit references this bug:

Author: feld
Date: Tue Sep  8 16:45:01 UTC 2015
New revision: 396410
URL: https://svnweb.freebsd.org/changeset/ports/396410

Log:
  MFH: r392658

  Remove incorrect screenrc command.

  PR:		201717
  Approved by:	ports-secteam (with hat)

Changes:
_U  branches/2015Q3/
  branches/2015Q3/sysutils/screen/Makefile
  branches/2015Q3/sysutils/screen/files/screenrc.sample