Ubuntu
exim4 package

exiqgrep: error on messages w/o size

Bug #181948 reported by Daniël van Eeden
8
Affects Status Importance Assigned to Milestone
exim4 (Debian)
Fix Released
Unknown
exim4 (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

# exiqgrep -x
Line mismatch: 42h 1JCb3k-0007bY-P1 <email address hidden>

This happens when "exim4 -bp" (mailq) reports lines w/o size info

Revision history for this message
Daniël van Eeden (dveeden) wrote :
Revision history for this message
Daniël van Eeden (dveeden) wrote :
Revision history for this message
Daniël van Eeden (dveeden) wrote :
Revision history for this message
Daniël van Eeden (dveeden) wrote :

The best patch is patch002.

Revision history for this message
Daniël van Eeden (dveeden) wrote :

Version: 4.60-3ubuntu3.1 (Ubuntu 6.06.1 LTS)

Revision history for this message
Daniël van Eeden (dveeden) wrote :
Revision history for this message
Daniel Holbach (dholbach) wrote :

To get your fix included in Ubuntu, try transforming it into a debdiff (http://wiki.ubuntu.com/PackagingGuide/Recipes/Debdiff) and submitting it for review (http://wiki.ubuntu.com/SponsorshipProcess).

Revision history for this message
Daniël van Eeden (dveeden) wrote :

This is patch002 in debdiff format.

Revision history for this message
Daniel Holbach (dholbach) wrote :

Server Team: could you please take a look at it?

Revision history for this message
Daniel Holbach (dholbach) wrote :

Chuck: can you please take a look at it?

Revision history for this message
Bryce Harrington (bryce) wrote :

The patch looks good.

Changed in exim4:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Bryce Harrington (bryce) wrote :

The code in the patch looks fine, but the debdiff does not apply to the exim4 package. exim4 uses a packaging approach I'm not terribly familiar with, so I'd not trust myself to repackage the patch to suit; someone more familiar with the packaging process for exim4 needs to look at this. DHolbach, can you please find someone to sponsor it (or unsub ubuntu-main-sponsors until someone can produce a proper patch)?

Changed in exim4:
assignee: nobody → dholbach
status: Triaged → Incomplete
Daniel Holbach (dholbach)
Changed in exim4:
assignee: dholbach → nobody
Revision history for this message
Daniel Holbach (dholbach) wrote :

The patch still applies cleanly against the unpacked source, but maybe it'd make more sense to forward this change to Debian/Upstream. I'm opening the relevant tasks in LP.

Changed in exim4:
status: Incomplete → Confirmed
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Bryce/Daniel-

I was able to clean up the debdiff, apply it cleanly, and build binary and source packages, which install. I'm going to go ahead and sponsor this.

:-Dustin

Changed in exim4:
status: Confirmed → In Progress
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Attaching the debdiff I used. Note that I also submitted this patch to Debian.

:-Dustin

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

I fat fingered the bug number in the changelog, so this didn't get auto-closed. Sorry!

exim4 (4.69-9ubuntu1) jaunty; urgency=low

  [ Daniel van Eeden <email address hidden> ]
  * debian/patches/71_exiq_grep_error_on_messages_without_size.dpatch:
    Improve handling of broken messages when "exim4 -bp" (mailq) reports lines
    w/o size info, LP: #18194

 -- Dustin Kirkland <email address hidden> Wed, 11 Feb 2009 06:43:52 -0600

Changed in exim4:
status: In Progress → Fix Released
Revision history for this message
Thierry Carrez (ttx) wrote :

I reported it in Debian as:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=528625
For some reason I cannot change the bugwatch since it already exists.

Daniël van Eeden (dveeden)
Changed in exim4 (Debian):
importance: Undecided → Unknown
status: New → Unknown
Bug Watch Updater (bug-watch-updater)
Changed in exim4 (Debian):
status: Unknown → New
Bug Watch Updater (bug-watch-updater)
Changed in exim4 (Debian):
status: New → Confirmed
Bug Watch Updater (bug-watch-updater)
Changed in exim4 (Debian):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.