Bug 194645 - mail/dovecot2-pigeonhole upgrade to 0.4.6
Summary: mail/dovecot2-pigeonhole upgrade to 0.4.6
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: Kurt Jaeger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-28 01:28 UTC by Larry Rosenman
Modified: 2014-11-13 19:00 UTC (History)
3 users (show)

See Also:
ler: maintainer-feedback? (nagy.attila)


Attachments
Upgrade patch (2.17 KB, patch)
2014-10-28 01:28 UTC, Larry Rosenman
no flags Details | Diff
0.4.5 patch (2.91 KB, patch)
2014-10-31 02:02 UTC, Larry Rosenman
no flags Details | Diff
0.4.5 patch -- with the patch file this time (3.84 KB, patch)
2014-10-31 02:05 UTC, Larry Rosenman
no flags Details | Diff
upgrade to 0.4.6 patch (2.91 KB, patch)
2014-11-02 17:59 UTC, Larry Rosenman
no flags Details | Diff
update to 0.4.6/change maintainer to self (3.05 KB, patch)
2014-11-12 17:55 UTC, Larry Rosenman
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Larry Rosenman freebsd_committer freebsd_triage 2014-10-28 01:28:32 UTC
Created attachment 148720 [details]
Upgrade patch

Hello Dovecot users,

It's been about 5 months since the last Pigeonhole release, so it is
high time for a new one. However, the main reason to make a release now
is the fact that Dovecot v2.2.15 has one change that causes problems
with the previous release of Pigeonhole.

Unlike recent earlier releases, this release has a few big new features.
Most notably, I implemented a generic Sieve script storage framework.
This means that it is now relatively easy to implement a storage driver
to retrieve Sieve scripts from a location other than the file system. An
important effect of this new implementation is that these alternative
storage locations can also be used with sieve_before/sieve_after. Also,
storing Sieve scripts there using ManageSieve is also possible, provided
that the driver provides an implementation. I used the new framework to
implement an LDAP storage driver. However, it is read-only, which means
that it doesn't currently provide ManageSieve support.

Several bugs were fixed. The most nasty ones were related to the
doveadm-sieve plugin (which is one of the reasons why this release took
so long). The new script storage driver framework entailed large changes
in this plugin, so many of these problems were addressed implicitly.
Hopefully, all problems are solved now.

Changelog v0.4.4:

* Added support for Japanese mail addresses with dots at non-standard
  places in localpart.
* Changed handling of ENOSPACE into a normal temporary failure and added
  handling of ENOQUOTA as a user error.
* Restructured result execution, so that all actions which involve mail
  storage are always committed before all others.
+ Implemented support for generic Sieve storages. Using alternative
  storages now also possible for sieve_before/sieve_after.
+ Implemented storage driver for retrieving Sieve scripts from LDAP.
  This currently cannot be used with ManageSieve.
+ Implemented sieve_redirect_envelope_from setting, which allows
  configuring the envelope sender of redirected messages.
- Fixed handling of mail storage errors occurring while evaluating the
  input message.
- managesieve-login:
   - Removed bogus ALERT response code returned for AUTHENTICATE
     command.
   - Fixed handling of invalid initial response argument to AUTHENTICATE
     command.
- Fixed handling of stream errors in lexical scanner.
- Fixed handling of SMTP errors. Permanent and temporary errors were
  mixed up.
- Fixed several problems reported by CLang 3.4.
- duplicate extension: Fixed erroneous compile error about conflicting
  tags when `:handle' argument was used last.
- relational extension: Fixed error handling of `:value' match.
- editheader extension: Fixed header unfolding and header iteration.
- mailbox extension: Fixed the `:create' tag, which erroneously
  subscribed an existing folder.
- extprograms plugin: Fixed handling of error codes.
- doveadm-sieve plugin: Fixed several bugs. Synchronization of symbolic
  link in the file storage should now also work properly.

The release is available as follows:

http://pigeonhole.dovecot.org/releases/2.2/dovecot-2.2-pigeonhole-0.4.4.tar.gz
http://pigeonhole.dovecot.org/releases/2.2/dovecot-2.2-pigeonhole-0.4.4.tar.gz.sig

Refer to http://pigeonhole.dovecot.org and the Dovecot v2.x wiki for
more information. Have fun testing this new release and don't hesitate
to notify me when there are any problems.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2014-10-28 01:28:32 UTC
Maintainer CC'd
Comment 2 Larry Rosenman freebsd_committer freebsd_triage 2014-10-31 02:02:37 UTC
Created attachment 148809 [details]
0.4.5 patch

0.4.5 patch
Comment 3 Larry Rosenman freebsd_committer freebsd_triage 2014-10-31 02:04:49 UTC
Author released 0.4.5.

Hello Dovecot users,

Unfortunately, Pigeonhole v0.4.4 had a rather big issue with error
handling and the execution of the implicit keep. This problem presented
itself for example when users exceeded their quota. This did not result
in the proper rejection message. This is why I quickly release a new
version.

Changelog v0.4.5:

+ Added a Pigeonhole version banner to doveconf output. This way, future
  bug reports will also include Pigeonhole version information.
- Fixed handling of implicit keep. Last version erroneously reported
  that implicit keep succeeded after an earlier failure, while it in
  fact had failed. Particularly occurred for mailbox quota errors.
- Fixed segfault occurring on SunOS systems when there is no active
  script.

The release is available as follows:

http://pigeonhole.dovecot.org/releases/2.2/dovecot-2.2-pigeonhole-0.4.5.tar.gz
http://pigeonhole.dovecot.org/releases/2.2/dovecot-2.2-pigeonhole-0.4.5.tar.gz.sig

Refer to http://pigeonhole.dovecot.org and the Dovecot v2.x wiki for
more information. Have fun testing this new release and don't hesitate
to notify me when there are any problems.

Regards,

there is a patch needed as well, included in the patch 

svn add files

after running the patch.
Comment 4 Larry Rosenman freebsd_committer freebsd_triage 2014-10-31 02:05:20 UTC
Created attachment 148810 [details]
0.4.5 patch -- with the patch file this time
Comment 5 John Marino freebsd_committer freebsd_triage 2014-11-01 08:35:03 UTC
To Attila Nagy (maintainer),
You have two consecutive, recent PR timeouts on this port.  If this PR times out as well, I'm resetting the maintainership to ports@

Larry, if you hear nothing on this PR after 2 weeks has passed, then actively ping me and I'll take some action.
Comment 6 Larry Rosenman freebsd_committer freebsd_triage 2014-11-01 16:09:08 UTC
John,
    If this times out, I can take it :)
Comment 7 Larry Rosenman freebsd_committer freebsd_triage 2014-11-02 17:59:23 UTC
Created attachment 148943 [details]
upgrade to 0.4.6 patch

Author integrated the patch in the previous attachment, and that's the only change. 

use THIS patch, and don't add the files/patch-Makefile.am file from the previous one.
Comment 8 John Marino freebsd_committer freebsd_triage 2014-11-12 17:17:22 UTC
Maintainer timed out again.

to the next committer: Please change the maintainer to Larry Rosenman  (ler@lerctr.org)

(Larry, it's best if you update the patch to make the MAINTAINER change yourself and not rely on this comment).  I'm promoting this to the patch-ready pool.
Comment 9 Larry Rosenman freebsd_committer freebsd_triage 2014-11-12 17:55:17 UTC
Created attachment 149331 [details]
update to 0.4.6/change maintainer to self
Comment 10 Kurt Jaeger freebsd_committer freebsd_triage 2014-11-13 18:47:58 UTC
make check-plist worked fine.

doing poudriere build tests right now.
Comment 11 commit-hook freebsd_committer freebsd_triage 2014-11-13 18:59:41 UTC
A commit references this bug:

Author: pi
Date: Thu Nov 13 18:58:43 UTC 2014
New revision: 372535
URL: https://svnweb.freebsd.org/changeset/ports/372535

Log:
  mail/dovecot2-pigeonhole: 0.4.3 -> 0.4.6

  - submitter takes maintainership

  Changes:

  0.4.4:
  - Added support for Japanese mail addresses with dots at non-standard
    places in localpart.
  - Changed handling of ENOSPACE into a normal temporary failure and added
    handling of ENOQUOTA as a user error.
  - Restructured result execution, so that all actions which involve mail
    storage are always committed before all others.
  - Implemented support for generic Sieve storages. Using alternative
    storages now also possible for sieve_before/sieve_after.
  - Implemented storage driver for retrieving Sieve scripts from LDAP.
    This currently cannot be used with ManageSieve.
  - Implemented sieve_redirect_envelope_from setting, which allows
    configuring the envelope sender of redirected messages.
  - Fixed handling of mail storage errors occurring while evaluating the
    input message.
  - managesieve-login:
     - Removed bogus ALERT response code returned for AUTHENTICATE
       command.
     - Fixed handling of invalid initial response argument to AUTHENTICATE
       command.
  - Fixed handling of stream errors in lexical scanner.
  - Fixed handling of SMTP errors. Permanent and temporary errors were
    mixed up.
  - Fixed several problems reported by CLang 3.4.
  - duplicate extension: Fixed erroneous compile error about conflicting
    tags when `:handle' argument was used last.
  - relational extension: Fixed error handling of `:value' match.
  - editheader extension: Fixed header unfolding and header iteration.
  - mailbox extension: Fixed the `:create' tag, which erroneously
    subscribed an existing folder.
  - extprograms plugin: Fixed handling of error codes.
  - doveadm-sieve plugin: Fixed several bugs. Synchronization of symbolic
    link in the file storage should now also work properly.

  0.4.5:
  - Added a Pigeonhole version banner to doveconf output. This way, future
    bug reports will also include Pigeonhole version information.
  - Fixed handling of implicit keep. Last version erroneously reported
    that implicit keep succeeded after an earlier failure, while it in
    fact had failed. Particularly occurred for mailbox quota errors.
  - Fixed segfault occurring on SunOS systems when there is no active
    script.

  0.4.6:
  - ports-related change integrated upstream

  PR:		194645
  Submitted by:	Larry Rosenman <ler@lerctr.org>
  Approved by:	maintainer (timeout)

Changes:
  head/mail/dovecot2-pigeonhole/Makefile
  head/mail/dovecot2-pigeonhole/distinfo
  head/mail/dovecot2-pigeonhole/pkg-plist
Comment 12 Kurt Jaeger freebsd_committer freebsd_triage 2014-11-13 19:00:20 UTC
Committed, thanks.