Bug 25386 - [patch] viewing attached images doesn't work
Summary: [patch] viewing attached images doesn't work
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: pine
Version: 7.1
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mike A. Harris
QA Contact: David Lawrence
URL:
Whiteboard: Florence RC-1
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2001-01-31 16:38 UTC by Tim Waugh
Modified: 2007-04-18 16:31 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2001-02-03 11:34:50 UTC
Embargoed:


Attachments (Terms of Use)
pine-4.32-tmp.patch (298 bytes, patch)
2001-01-31 18:41 UTC, Tim Waugh
no flags Details | Diff

Description Tim Waugh 2001-01-31 16:38:19 UTC
Viewing attached images in pine doesn't work: ee gets launched, but for
some reason the file it's told to look at has gone by the time it does.

The strace output suggests that the shell is exiting too early, but I
haven't investigated much yet, and was unable to reproduce the problem
outside of pine.

Comment 1 Tim Waugh 2001-01-31 16:49:56 UTC
pine-4.31-2 works fine; pine-4.32-1 doesn't.

Comment 2 Tim Waugh 2001-01-31 18:41:09 UTC
Created attachment 8581 [details]
pine-4.32-tmp.patch

Comment 3 Tim Waugh 2001-01-31 18:44:37 UTC
The above patch is the correct fix, I think

Rationale: Pine is actually executing:

	sh -c '( cmd-read-from-mailcap tmpfile ; rm tmpfile ) &'

The new change in 4.32 was equivalent to this kind of thing:

  fork ( exec )
  wait4
+ unlink (tmpfile)

So tmpfile was disappearing because the (show file; rm file) process was
backgrounded and the shell returned, and in 4.32 it removes the file, needlessly
since the backgrounded task will do that when the viewer exits.


Comment 4 Glen Foster 2001-01-31 21:51:48 UTC
This defect is considered MUST-FIX for Florence Release-Candidate #1

Comment 5 Tim Waugh 2001-02-03 11:34:46 UTC
4.33 has this fix in.

Comment 6 Mike A. Harris 2001-02-06 16:28:43 UTC
Yes, 4.33 has fixed this and will be in rawhide real soon.


Note You need to log in before you can comment on or make changes to this bug.