Bug 230995 - security/ccrypt: Update to 1.11
Summary: security/ccrypt: Update to 1.11
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Fernando Apesteguía
URL: http://ccrypt.sourceforge.net/#news
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-29 14:27 UTC by Dmitri Goutnik
Modified: 2018-09-09 20:56 UTC (History)
3 users (show)

See Also:
dev2: maintainer-feedback+


Attachments
patch file (2.35 KB, patch)
2018-08-29 14:27 UTC, Dmitri Goutnik
dmgk: maintainer-approval? (dev2)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitri Goutnik freebsd_committer freebsd_triage 2018-08-29 14:27:08 UTC
Created attachment 196678 [details]
patch file

- Update 1.10 -> 1.11
- Modernize port

Changelog: http://ccrypt.sourceforge.net/#news

QA:
  poudriere testport: OK (120a, 112a, 104i)
Comment 1 Oliver Heesakkers 2018-09-09 14:11:10 UTC
Looks good to me. Thanks for the patch!

Note: Lately I haven't been able to use FreeBSD as much as I used to or would want to. If you feel like adopting the port, feel free to do so.
Comment 2 Fernando Apesteguía freebsd_committer freebsd_triage 2018-09-09 16:12:43 UTC
Take
Comment 3 Nathan 2018-09-09 17:22:00 UTC
Only part I'd change
 .for i in ccrypt.html cypfaq01.txt
 	${INSTALL_DATA} ${WRKSRC}/doc/$i ${STAGEDIR}${DOCSDIR}
.endfor
I say that as a committer reviewed one of mine some time ago and recommended not to. Guess so it doesn't have to iterate through it so much or something. 

(cd ${WRKSRC}/doc ; ${INSTALL_DATA} ccrypt.html cypfaq01.txt ${STAGEDIR}${DOCSDIR}) is how I'd do it, but that's just me :)
Comment 4 commit-hook freebsd_committer freebsd_triage 2018-09-09 20:54:39 UTC
A commit references this bug:

Author: fernape
Date: Sun Sep  9 20:54:30 UTC 2018
New revision: 479334
URL: https://svnweb.freebsd.org/changeset/ports/479334

Log:
  security/ccrypt: update to 1.11

  * Fixes a number of minor portability issues and minor bugs

  PR:	230995
  Submitted by:	dg@syrec.org
  Approved by:	tcberner (mentor), dev2@heesakkers.info (maintainer)
  Differential Revision:	https://reviews.freebsd.org/D17098

Changes:
  head/security/ccrypt/Makefile
  head/security/ccrypt/distinfo
  head/security/ccrypt/pkg-plist
Comment 5 Fernando Apesteguía freebsd_committer freebsd_triage 2018-09-09 20:56:15 UTC
Committed with minor changes.

Thanks to everyone involved!