[dsdp-tm-dev] Re: FTP Listing Parser Extension Point

[javier.montalvoorus@xxxxxxxxxxx]

Hi Martin,

I'm testing at the moment a version
of the FTP Listing Parser using regular expressions and priorities. 
I hope I'll be able to check it in the
CVS, after testing, by the end of today (UTC+1).

The extension for the 'dir command'
is not necessary, as it is part of the FTP Server implementation. 
The FTP client just sends a LIST command
that is translated internally by the server.

Regards,

Javier Montalvo Orús
Development
Tools 
Symbian Software Limited.

"Oberhuber, Martin" <Martin.Oberhuber@xxxxxxxxxxxxx>

14/05/2007 14:13

To	<javier.montalvoorus@xxxxxxxxxxx>
cc	"Target Management developer discussions" <dsdp-tm-dev@xxxxxxxxxxx>
Subject	FTP Listing Parser Extension Point

Hello Javier,

I noticed that you fixed the issues (1) and (2)
I raised for the FTP Listing Parser extension
point.

I'm wondering if you have any plans to also work
on issue (3) -- allowing contributed listing 
parsers to take part in the autodetect?

What I'd like to see is the following optional 
additional attributes for the extension point:
   ftpSystemTypes = ".*VxWorks.*|.*VXWORKS.*"
specifying a regular _expression_ to match against 
the result of the SYST command, such that the 
parser would take part in autodetect only when 
it matches;
   ftpDirCommand = "dir -l"
an optional command string to send if the given
listing parser is active. Perhaps also a third one:
   priority = "100"
giving a priority to know what parser should be
preferred if multiple parsers match the systemTypes
pattern and are thus candidates for autodetect.

If any of these optional attributes is not given,
the current behavior would be the default.

Even if the current implementation doesn't care
for these attributes in the first implementation,
I think I'd like to see these specified.

What do you think? What are your plans?

Thanks,
--
Martin Oberhuber
Wind River Systems, Inc.
Target Management Project Lead, DSDP PMC Member
http://www.eclipse.org/dsdp/tm
-----Original Message-----
From: dsdp-tm-dev-bounces@xxxxxxxxxxx
[mailto:dsdp-tm-dev-bounces@xxxxxxxxxxx] On Behalf Of Oberhuber, Martin
Sent: Friday, May 04, 2007 4:00 PM
To: javier.montalvoorus@xxxxxxxxxxx
Cc: Target Management developer discussions
Subject: [dsdp-tm-dev] FTP Listing Parser Extension Point

Hello Javier,

thanks for moving the FTP Listing Parser extension point
from Services to the Subsystem. Seeing the code, I'm 
actually more convinced than ever that this was the 
right move -- the dropdown and extension point handling 
are now all nicely in one plugin.

There is just few concerns I still have:

1. Avoid early plugin activation for extensions.
2. Translatable label of the FTP Parser extension.

3. Autodetect.
   As I have mentioned before, the AUTO special parser should
   not be restricted to the preconfigured parsers only. User
   contributed parsers should also be considered.
   For doing that, I'd suggest that the extension has an 
   additional attribute "ftpSystemTypes" holding a regular
   _expression_ to be matched against the result of the FTP
   SYST command. If multiple parsers match, they should be
   tried one after the other. An RSEDelegatingFtpListingParser
   could do this work.

See my previous E-Mail comments that I sent you below for reference. 

Thanks,
--
Martin Oberhuber
Wind River Systems, Inc.
Target Management Project Lead, DSDP PMC Member
http://www.eclipse.org/dsdp/tm 

 

________________________________

From: Oberhuber, Martin 
Sent: Friday, April 20, 2007 6:10 PM
To: javier.montalvoorus@xxxxxxxxxxx
Cc: Target Management developer discussions
Subject: New FTP Listing Parser Extension Point


Hello Javier,

two more considerations:


*                
We should avoid having to activate plugins which contribute FTP
parsers, just in order to check whether they are valid. Therefore, the
extension point should have another attribute "systemNames" which
should
contain a regular _expression_ that needs to match the system name
reported by the remote side. If it matches, AUTO would activate the
plugin, instantiate the parser, and try that parser on that system. 

Cheers,
Martin


Oberhuber, Martin schrieb: 

                
Hello Javier,
                
 
                
I looked at your patch and it looks good to me.
                
 
                
I have no time right now to test it with VxWorks, but I'm
confident
                
that it will work fine. Your patch looks sizeable, so please go
ahead
                
and commit it.
                
 
                
What I would like to see going forward, is the ability that the 
                
"AUTO" parser type also tries the various contributed parsers.
                
Right now, when I'm not mistaken, any extender-contributed
                
parser will only be used when explicitly selected by the user.
                
 
                
This is not user-friendly. We should allow extenders to
contribute
                
parsers, but the system should be able to select the contributed
                
parsers automatically wherever possible. In the simplest case, 
                
the parser contribution could have an 
                
   boolean isValidFor(String systemType)
                
method that allows auto-selecting it based on the systemType
                
returned by the remote. In case multiple contributed parsers
                
return TRUE for isValidFor(), they could be tried one after the
                
other while parsing, until the first one is successful.
                
 
                
                
Thanks,
                
--
                
Martin Oberhuber
                
Wind River Systems, Inc.
                
Target Management Project Lead, DSDP PMC Member
                
http://www.eclipse.org/dsdp/tm <http://www.eclipse.org/dsdp/tm>
<http://www.eclipse.org/dsdp/tm>   
                
  


-- 
Martin Oberhuber
Wind River Systems, Inc.
Target Management Project Lead, DSDP PMC Member
http://www.eclipse.org/dsdp/tm
_______________________________________________
dsdp-tm-dev mailing list
dsdp-tm-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/dsdp-tm-dev

---

********************************************************************** Symbian Software Ltd is a company registered in England and Wales with registered number 4190020 and registered office at 2-6 Boundary Row, Southwark, London, SE1 8HP, UK. This message is intended only for use by the named addressee and may contain privileged and/or confidential information. If you are not the named addressee you should not disseminate, copy or take any action in reliance on it. If you have received this message in error please notify postmaster@xxxxxxxxxxx and delete the message and any attachments accompanying it immediately. Neither Symbian nor any of its Affiliates accepts liability for any corruption, interception, amendment, tampering or viruses occurring to this message in transit or for any message sent by its employees which is not in compliance with Symbian corporate policy. **********************************************************************