Bug 153115 - [maintainer] [patch] shells/scponly Note security concern, cleanups
Summary: [maintainer] [patch] shells/scponly Note security concern, cleanups
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Wen Heping
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-13 04:00 UTC by Robert Farmer
Modified: 2010-12-15 07:20 UTC (History)
0 users

See Also:


Attachments
file.diff (7.54 KB, patch)
2010-12-13 04:00 UTC, Robert Farmer
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Farmer 2010-12-13 04:00:19 UTC
Most important:
-Patch SECURITY doc to include note about bypassing rsync argument checking with popt (from upstream) and tell people to read it

And some minor cleanup:
-Drop long comment describing knobs - it just duplicates OPTIONS
-For SCPONLY_DEFAULT_CHDIR, print a note about setting it. I'm not sure if post-patch is the best place for this, though?
-Drop dead site and just use Sourceforge
-Use the PORTDOCS variable
-Install some useful docs and drop useless one (TODO)
-Drop pre-everything message about defaults changing; that was 5 years ago
-LOCALBASE vs. PREFIX correction
-Add post-install messages to the plist so package users see them too

Fix: Patch attached with submission follows:
Comment 1 Wen Heping freebsd_committer freebsd_triage 2010-12-15 01:32:05 UTC
Responsible Changed
From-To: freebsd-ports-bugs->wen

I'll take it.
Comment 2 dfilter service freebsd_committer freebsd_triage 2010-12-15 07:12:07 UTC
wen         2010-12-15 07:12:02 UTC

  FreeBSD ports repository

  Modified files:
    shells/scponly       Makefile pkg-plist 
  Added files:
    shells/scponly/files patch-SECURITY 
  Log:
  -Patch SECURITY doc to include note about bypassing rsync argument checking with popt (from upstream) and tell people to read it
  -Drop long comment describing knobs - it just duplicates OPTIONS
  -For SCPONLY_DEFAULT_CHDIR, print a note about setting it. I'm not sure if post-patch is the best place for this, though?
  -Drop dead site and just use Sourceforge
  -Use the PORTDOCS variable
  -Install some useful docs and drop useless one (TODO)
  -Drop pre-everything message about defaults changing; that was 5 years ago
  -LOCALBASE vs. PREFIX correction
  -Add post-install messages to the plist so package users see them too
  
  PR:             ports/153115
  Submitted by:   Rob Farmer <rfarmer@predatorlabs.net> (maintainer)
  
  Revision  Changes    Path
  1.36      +15 -77    ports/shells/scponly/Makefile
  1.1       +32 -0     ports/shells/scponly/files/patch-SECURITY (new)
  1.6       +9 -4      ports/shells/scponly/pkg-plist
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Wen Heping freebsd_committer freebsd_triage 2010-12-15 07:14:11 UTC
State Changed
From-To: open->closed

Committed. Thanks!