Bug 219321 - lang/php71: add mail header patch
Summary: lang/php71: add mail header patch
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: Torsten Zuehlsdorff
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-16 09:20 UTC by Karsten Schmidt
Modified: 2017-11-23 11:40 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (tz)

Attachments
MAILHEAD option (1.50 KB, patch)
2017-05-16 09:20 UTC, Karsten Schmidt 		no flags Details | Diff
MAILHEAD option (1.50 KB, patch)
2017-05-16 09:23 UTC, Karsten Schmidt 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Karsten Schmidt 2017-05-16 09:20:27 UTC 
Created attachment 182623 [details]
MAILHEAD option

Add the mail header patch to php7x like php5x
Comment 1 Karsten Schmidt 2017-05-16 09:23:11 UTC 
Created attachment 182624 [details]
MAILHEAD option

Attached the wrong patch first time around, this one should work.
Comment 2 commit-hook freebsd_committer freebsd_triage 2017-11-23 11:39:14 UTC 
A commit references this bug:

Author: tz
Date: Thu Nov 23 11:38:59 UTC 2017
New revision: 454753
URL: https://svnweb.freebsd.org/changeset/ports/454753

Log:
  lang/php70 and lang/php71: Add mail header patch

  While here to some improvements:
  - Remove CONFLICTS of no longer existing PHP 5.5
  - Add CONFLICTS to PHP 7.2
  - Remove IPV6_DESC in order to use default one

  PR:           219321
  Submitted by: Karsten Schmidt <bsd@guggemand.dk>

Changes:
  head/lang/php70/Makefile
  head/lang/php70/distinfo
  head/lang/php71/Makefile
  head/lang/php71/distinfo
Comment 3 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2017-11-23 11:40:26 UTC 
Hello Karsten,

i just committed your patch. I also ported it back to lang/php70 so we now have a consistent state of options.

Greetings,
Torsten