Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338120 - Backport to 3.2.x Failed to copy the content of a tar file
Summary: Backport to 3.2.x Failed to copy the content of a tar file
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.2.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.2+   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords:
Depends on: 337612
Blocks:
  Show dependency tree
 
Reported: 2011-02-24 11:57 EST by David McKnight CLA
Modified: 2011-03-01 12:15 EST (History)
2 users (show)

See Also:
xuanchen: review+

Attachments
backport patch (3.62 KB, patch)
2011-02-24 12:12 EST, David McKnight CLA 		no flags Details | Diff
updated backport fix (4.23 KB, patch)
2011-02-28 17:30 EST, David McKnight CLA 		no flags Details | Diff
updated backport (4.60 KB, patch)
2011-03-01 10:53 EST, David McKnight CLA 		no flags Details | Diff
Show Obsolete (2) 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 David McKnight CLA 2011-02-24 11:57:49 EST 
This bug is for the backport of the fix for bug 337612 to the R3_2_maintenance stream.
Comment 1 David McKnight CLA 2011-02-24 12:12:40 EST 
Created attachment 189720 [details]
backport patch
Comment 2 David McKnight CLA 2011-02-24 12:13:06 EST 
Kevin, could you please review this patch?
Comment 3 Samuel Wu CLA 2011-02-28 10:26:59 EST 
Hi Dave,
Is this fix in RSE 3.2.2 build yet? Thanks.
Comment 4 David McKnight CLA 2011-02-28 10:32:59 EST 
(In reply to comment #3)
> Hi Dave,
> Is this fix in RSE 3.2.2 build yet? Thanks.

No, it hasn't been reviewed yet.

Xuan, could you please review this?
Comment 5 David McKnight CLA 2011-02-28 17:30:37 EST 
Created attachment 190006 [details]
updated backport fix
Comment 6 Xuan Chen CLA 2011-03-01 10:01:31 EST 
The fix looks good.  Thanks.
Comment 7 David McKnight CLA 2011-03-01 10:53:52 EST 
Created attachment 190043 [details]
updated backport
Comment 8 David McKnight CLA 2011-03-01 10:55:16 EST 
(In reply to comment #6)
> The fix looks good.  Thanks.

Hi Xuan, thanks for reviewing but could you please review the new patch.  Some changes were made as per the discussion in the original bug.
Comment 9 Xuan Chen CLA 2011-03-01 11:52:34 EST 
Updated patch looks good.
Comment 10 David McKnight CLA 2011-03-01 11:58:33 EST 
Thanks for the review.  I've committed the change to the R3_2_maintenance stream.
Comment 11 Martin Oberhuber CLA 2011-03-01 12:15:58 EST 
Released into 3.2.2+

Note that from now on, when you want something in 3.2.2+ you'll have to release it into the Mapfile yourself (after code review from 1 committer).