Page MenuHomeFreeBSD

vvv updated to 1.9.0
AbandonedPublic

Authored by dereks_lifeofadishwasher.com on Sep 11 2017, 10:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 12:23 PM
Unknown Object (File)
Sun, Oct 27, 7:10 AM
Unknown Object (File)
Sun, Oct 27, 7:10 AM
Unknown Object (File)
Sun, Oct 27, 6:57 AM
Unknown Object (File)
Oct 16 2024, 3:33 AM
Unknown Object (File)
Oct 2 2024, 1:18 PM
Unknown Object (File)
Sep 23 2024, 8:17 PM
Unknown Object (File)
Sep 18 2024, 10:31 AM
Subscribers

Details

Reviewers
None
Summary

vvv patch set was updated to support mutt 1.9.0

Test Plan
  • confirm nntp applies
  • create new nntp patches (mtime/forcebase64) if offset/fuzz found
  • testport without NNTP option
  • testport with NNTP and two nntp patches

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 11502
Build 11857: arc lint + arc unit

Event Timeline

lwhsu added inline comments.
mail/mutt/Makefile
213

Should we just remove this section?

I believe this review can be closed because of https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=222244

However, the section is there to be uncommented/commented as newer
versions of mutt are released the greeting and VVV might not have been
updated before the port has changed. In some cases mainly minor releases
and most likely bug releases the latest version, normally the previous
release, of the patch can still apply cleanly.

If you look at the current version of mail/mutt Makefile they're both
uncommented with different version numbers.

If we remove the section the port can't be updated until these patches
are released for ${PORTVERSION} otherwise they would need to be pulled
in the mail/mutt/files/. For the greeting patch this might not be so
bad but there's no chance I would attempt to maintain VVV's NNTP patch.
Ideally I would rather just remove it from the port all together based
on the assumption NNTP usage is low at worst and non existent at best.

Got it, was just being curious. Thanks for the explanation. Please feel free to close this review.