Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 331109

Summary: [scp] time fields are different in different ls implementations
Product: [Tools] Target Management Reporter: Anna Dushistova <anna.dushistova>
Component: RSEAssignee: Anna Dushistova <anna.dushistova>
Status: RESOLVED FIXED QA Contact: Martin Oberhuber <mober.at+eclipse>
Severity: normal    
Priority: P3 CC: nikita_shulga
Version: 3.3   
Target Milestone: 3.3 M4   
Hardware: PC   
OS: Linux   
Whiteboard:
Bug Depends on: 213438    
Bug Blocks:    
Attachments:
Description Flags
Proposed fix for the problem
none
Proposed fix with updated copyright header anna.dushistova: iplog+

Description Anna Dushistova CLA 2010-11-25 06:13:01 EST
Right now, ls --time-style=long-iso is not supported, and it is default on some systems, for example,
open Suse 11.2.
Comment 1 Nikita Shulga CLA 2010-11-25 09:49:42 EST
Created attachment 183855 [details]
Proposed fix for the problem

Attached patch modifies `ls` output parsing algorithm to detect, whether timestamp is returned in one of the short formats(which usually takes 3 fields)  or in long one (i.e. 'yyyy-MM-dd HH:mm'). 

Legal Message: I, Nikita Shulga, declare that I developed attached code from
scratch, without referencing any 3rd party materials except material licensed
under the EPL. I am authorized by my employer to make this contribution under
the EPL.
Comment 2 Anna Dushistova CLA 2010-11-26 16:35:49 EST
Nikita, I need you to update the copyright on the changed file. Could you please add a line like this to the end:
*Nikita Shulga (Mentor Graphics)-[this bug number] this bug description ?
Thanks!
Comment 3 Nikita Shulga CLA 2010-11-28 23:07:58 EST
Created attachment 184002 [details]
Proposed fix with updated copyright header

Updated copyright header in proposed patch as requested.

Legal Message: I, Nikita Shulga, declare that I developed attached code from
scratch, without referencing any 3rd party materials except material licensed
under the EPL. I am authorized by my employer to make this contribution under
the EPL.
Comment 4 Anna Dushistova CLA 2010-11-29 14:47:17 EST
Patch applied. Thanks, Nikita!