[Bug 225708] Merge Review: dovecot

2014-03-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #22 from Michal Hlavinka --- Is there anything left or can we close this review? -- You are receiving this mail because: You are always notified about changes to this product and component _

[Bug 225708] Merge Review: dovecot

2014-03-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #23 from Dan HorĂ¡k --- (In reply to Michal Hlavinka from comment #22) > Is there anything left or can we close this review? There is one little thing - you should switch from the hardcoded gz extension in the man pages in %files to

[Bug 225708] Merge Review: dovecot

2014-03-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #24 from Michal Hlavinka --- I don't think such change is worth it, but changed anyway. -- You are receiving this mail because: You are always notified about changes to this product and component __

[Bug 225708] Merge Review: dovecot

2014-03-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #25 from Paul Howarth --- A few more things I noticed: * Adding BR: xz-devel would enable support for xz-compressed mailboxes * %defattr has been redundant since rpm 4.4 (EL-5 doesn't need it...) * /var/run/dovecot is marked as %

[Bug 225708] Merge Review: dovecot

2014-03-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #26 from Michal Hlavinka --- > A few more things I noticed: > * Adding BR: xz-devel would enable support for xz-compressed mailboxes added > * %defattr has been redundant since rpm 4.4 (EL-5 doesn't need it...) removed > * /var/

[Bug 225708] Merge Review: dovecot

2014-03-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #27 from Paul Howarth --- (In reply to Michal Hlavinka from comment #26) > > * /var/run/dovecot is marked as %ghost but its implicitly-included subdir > > /var/run/dovecot/empty is not, which doesn't really make sense > > /var/ru

[Bug 225708] Merge Review: dovecot

2014-03-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #28 from Michal Hlavinka --- Seem I forgot to push the changes. Fixed, it's in repository now. -- You are receiving this mail because: You are always notified about changes to this product and component ___

[Bug 225708] Merge Review: dovecot

2012-01-04 Thread bugzilla
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225708 Michal Hlavinka changed: What|Removed |Added ---

[Bug 225708] Merge Review: dovecot

2012-02-23 Thread bugzilla
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225708 Paul Howarth changed: What|Removed |Added --

[Bug 225708] Merge Review: dovecot

2012-02-24 Thread bugzilla
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #21 from Michal Hlavinka 2012-02-24 07:33:55 EST --- (In reply to comment #20) > Some comments from comparing the b

[Bug 225708] Merge Review: dovecot

2010-06-10 Thread bugzilla
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #18 from Michal Hlavinka 2010-06-10 11:32:41 EDT --- no objections from me, it's another two months, I think you ca

[Bug 225708] Merge Review: dovecot

2010-04-09 Thread bugzilla
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225708 --- Comment #17 from Paul Howarth 2010-04-09 07:11:27 EDT --- No response from Jef for 5 months; anyone object to me taking ove