Bug 213431

Summary: mail/mutt: Upgrade to 1.7.1, use some remote patches, portlint cleanup
Product: Ports & Packages Reporter: Derek Schrock <dereks>
Component: Individual Port(s)Assignee: Adam Weinberger <adamw>
Status: Closed FIXED    
Severity: Affects Some People CC: adamw, dereks, ler
Priority: --- Keywords: patch
Version: Latest   
Hardware: Any   
OS: Any   
Description Flags
1.7.1, portlint and makepatch cleanup
dereks: maintainer-approval+
Updated 1.7.1 patch for subject based patches dereks: maintainer-approval+

Description Derek Schrock 2016-10-13 06:41:34 UTC
Created attachment 175686 [details]
1.7.1, portlint and makepatch cleanup

- Update to 1.7.1
- portlint cleanup (still some warnings that can be explained away)
- Use upsteam maintained patches (date_conditional, deepif, greeting)
- makepatch patch-* patches
Comment 1 VK freebsd_triage 2016-10-13 07:51:49 UTC
Comment on attachment 175686 [details]
1.7.1, portlint and makepatch cleanup

Thanks for the patch. Please set the maintainer-approval+ flag on attachments for ports you're maintainer of, under attachment Details, to help us track this issue better.
Comment 2 Adam Weinberger freebsd_committer 2016-10-13 22:49:48 UTC
Hi Derek,

I appreciate you doing all the hard 'makepatch' work, but are you sure that you want to change from 1-topic-per-patch to 1-file-per-patch? It might make it more work to maintain in the future.
Comment 3 Derek Schrock 2016-10-14 20:20:55 UTC
Yeah, I could see issues with that.  Let me provide another patch that obsoletes the current one.  I'd also like to update the greeting patch vars since it was just updated for 1.7.1.

When creating a subject patch that's acceptable for portlint is it just a concatenated set of patches from makepatch in to a single file?
Comment 4 Adam Weinberger freebsd_committer 2016-10-14 20:33:48 UTC
Yup that's the right idea. Just be aware that they get applied in alphabetical order, so if two patches affect the same file you might need to generate the patches in steps.
Comment 5 Adam Weinberger freebsd_committer 2016-10-14 20:35:55 UTC
Also, keep in mind that portlint makes suggestions, not statements of fact. If you liked the COMMENT the way it was, for example, you don't need to change it just because portlint has a thing against articles.
Comment 6 Derek Schrock 2016-10-17 00:04:05 UTC
Created attachment 175840 [details]
Updated 1.7.1 patch for subject based patches

- Bring back some subject patches
- auto pull dist/patchfiles with makesum
- 1.7.1 greeting patch
Comment 7 Adam Weinberger freebsd_committer 2016-10-17 02:00:22 UTC
Nice work, Derek. This feels like a better path to me.
Comment 8 commit-hook freebsd_committer 2016-10-17 02:40:46 UTC
A commit references this bug:

Author: adamw
Date: Mon Oct 17 02:40:42 UTC 2016
New revision: 424090
URL: https://svnweb.freebsd.org/changeset/ports/424090

  - Update to 1.7.1
  - Move some vars around to make portlint happy
  - Use MUTT_LITE instead of LITE
  - Fetch deepif and date-contitional from remote locations
  - Reword COMMENT
  - Bring back remote patch versions incase mutt version and patch
    versions arn't in sync
  - Remove date_conditional patch pulled in remotely
  - Add two new sidebar samples
  - convert patch-* patches to makepatch patches
  - Greeting patch 1.7.1 now PATCHFILE
  - make makesum will pull all optional dist/patchfiles

  PR:		213431
  Submitted by:	maintainer (dereks lifeofadishwasher com)

Comment 9 Adam Weinberger freebsd_committer 2016-10-17 02:43:14 UTC
Committed. Nice work, Derek! I am sure it was a real pain generating the patches this time, but it'll be much easier in the future.

I modified the patch-Makefile.am, because install was failing as non-root user (the chgrp and chmod 2755 aren't available as non-root). The chgrp and chmod are already handled by the pkg-plist so the simplest thing was to just nuke that block entirely.