Difference between revisions of "DSDP/TM/Committer Phone Meeting 2-May-2007"

From Eclipsepedia

< DSDP‎ | TM
Jump to: navigation, search
(New page: {|border=1 cellspacing=0 cellpadding=4 | Meeting Title: | '''TM Committer Phone Meeting''' |- | Date & Time: | Tuesday May 2, 2007 at [http://www.timeanddate.com/worldclock/meetingdet...)
 
Line 27: Line 27:
  
 
{|border=1
 
{|border=1
| Martin  || 100% || API cleanups; || 100%
+
| Martin  || 30% || Platform symbolic links; API cleanups; || 100%
 
|-
 
|-
 
| DaveD  || 100% || Persistence Improvements; UDA || 100%
 
| DaveD  || 100% || Persistence Improvements; UDA || 100%
Line 33: Line 33:
 
| DaveM  || 40% || Bugfixes || 40%
 
| DaveM  || 40% || Bugfixes || 40%
 
|-
 
|-
| Kushal || (?)  || BIDI Encodings || (?)
+
| Kushal || (?)  || BIDI Encodings; UDA || (?)
 
|-
 
|-
 
| Javier  ||  50% || FTP Parser Extension Point - Problems persisting Property Set || 50%
 
| Javier  ||  50% || FTP Parser Extension Point - Problems persisting Property Set || 50%
Line 47: Line 47:
  
 
* '''Plan towards M7''' - goals and deadlines for I-builds
 
* '''Plan towards M7''' - goals and deadlines for I-builds
** I20070426 -- Major breaking API changes (Martin: getSystemType(), getInstance(); UI/Non-UI; Persistence)
 
 
** I20070503 -- Minor API changes - Final Feature Additions - Feature Freeze (Terminal integration; FTP Parser)
 
** I20070503 -- Minor API changes - Final Feature Additions - Feature Freeze (Terminal integration; FTP Parser)
 
** I20070510 -- Stabilization, start [[TM 2.0 M7 Testing]] on May 14th
 
** I20070510 -- Stabilization, start [[TM 2.0 M7 Testing]] on May 14th
Line 57: Line 56:
 
** '''Priority #1: Finish API Changes'''
 
** '''Priority #1: Finish API Changes'''
 
*** '''Javier''': FTP parallel download, allowing to register directory listing parser by extension point
 
*** '''Javier''': FTP parallel download, allowing to register directory listing parser by extension point
**** Please check in changes even if they are partial, so they are part of refactoring
+
**** What about Commons.Net class in the Extension Point API? What about having extension point in the Service?
*** '''DaveD''': Persistence (for UDA); UI/Non-UI
+
*** '''DaveD''': Persistence (for UDA) - independent of IResource?; UI/Non-UI
**** Persistence Provider: try to make the configurable one independent of IResource - and probably wrap it in another one that does know the WorkspaceRoot and thus depends on IResource
+
*** '''DaveM''': [https://bugs.eclipse.org/bugs/show_bug.cgi?id=184322 184322] IProgressMonitor added to file system APIs: need to document whether null is allowed as paramter or not, both for IFileService and IRemoteFileSubSystem!!
*** [https://bugs.eclipse.org/bugs/show_bug.cgi?id=182221 bug 182221]: Should IRemoteFileSubSystem handle exceptions (by showing dialogs)? What about ISubSystem in general?
+
**** Should we have a common guideline of having the progress monitor first, or last?
**** '''Decision:''' Pragmatically, start converting simple methods in IRemoteFileSubSystem; but not convert ones that look too difficult - bring it up again next week
+
*** [https://bugs.eclipse.org/bugs/show_bug.cgi?id=182221 bug 182221]: IRemoteFileSubSystem should forward exceptions rather than handling them with dialogs - Decision was to pragmatically start converting simple methods - who should do it?
 +
*** '''Martin''': [https://bugs.eclipse.org/bugs/show_bug.cgi?id=177523 177523] getInstance; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=175680 175680] cleanup ISystemRegistry - probably bring more into non-UI; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=173042 173042] Refresh; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=172650 172650] Capabilities; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=176461 176461] expand-to-level; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=170910 170910] Integrate Terminalview; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=185098 185098] constants for system types; [https://bugs.eclipse.org/bugs/show_bug.cgi?id=185097 185097] make getCoreRegistry() simpler
 +
*** '''Michael''': Small API improvements for Terminal (with xyzmodem, [https://bugs.eclipse.org/bugs/show_bug.cgi?id=165893 165893]
 
** '''Priority #2: Import/Export and User Actions'''
 
** '''Priority #2: Import/Export and User Actions'''
*** DaveM: Import/Export should be complete
+
*** Import/Export should be complete - did anybody except DaveM test it? What about User Docu?
 
*** '''AI DaveD, Kushal''' persistence with User Actions; Dave working on API prerequisites now;  
 
*** '''AI DaveD, Kushal''' persistence with User Actions; Dave working on API prerequisites now;  
 
** '''Priority #3: Bugs'''
 
** '''Priority #3: Bugs'''
 
*** Bugzilla: [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_severity=blocker&bug_severity=critical&bug_severity=major&cmdtype=doit Severity Major], [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&cmdtype=doit&field0-0-0=priority&type0-0-0=regexp&value0-0-0=P%5B12%5D&field0-0-1=bug_severity&type0-0-1=regexp&value0-0-1=blocker%7Ccritical%7Cmajor hi-priority]
 
*** Bugzilla: [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_severity=blocker&bug_severity=critical&bug_severity=major&cmdtype=doit Severity Major], [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&cmdtype=doit&field0-0-0=priority&type0-0-0=regexp&value0-0-0=P%5B12%5D&field0-0-1=bug_severity&type0-0-1=regexp&value0-0-1=blocker%7Ccritical%7Cmajor hi-priority]
 +
**** [https://bugs.eclipse.org/bugs/show_bug.cgi?id=167620 167620] Select and Delete in the Main thread
 
*** Encoding Issues: Kushal single encoding control for all subsystems?
 
*** Encoding Issues: Kushal single encoding control for all subsystems?
*** BIDI bugs: Will we adopt [[ICU4J]]? (Note according to [[RCP FAQ]], ICU4J is required by org.eclipse.help and thus part of the core RCP already - though this may be changed ([https://bugs.eclipse.org/bugs/show_bug.cgi?id=183761 bug 183761]).
+
*** BIDI bugs: adopting [[ICU4J]]? It's part of core RCP already; keep it in UI
**** ICU4J is part of the Platform already; we should start using it; can defer the changes to later since no API change, try to keep it in UI only
+
 
**** '''AI DaveD''' to follow up on the concrete [https://bugs.eclipse.org/bugs/show_bug.cgi?id=183631 bug 183631] and fix it, to get experience with ICU4J
 
**** '''AI DaveD''' to follow up on the concrete [https://bugs.eclipse.org/bugs/show_bug.cgi?id=183631 bug 183631] and fix it, to get experience with ICU4J
*** Who wants to commit patches from Kevin -- update the [[DSDP/TM/Code Ownership]] page
 
 
*** EFS: Copy(), Move(); Open Remote Project on Startup: Get rid of IResource dependencies where possible; minimal plugin activations; UI/Non-UI
 
*** EFS: Copy(), Move(); Open Remote Project on Startup: Get rid of IResource dependencies where possible; minimal plugin activations; UI/Non-UI
 
*** [https://bugs.eclipse.org/bugs/show_bug.cgi?id=182454 182454] - what is the meaning of ISystemViewElementAdapter.getAbsoluteName()
 
*** [https://bugs.eclipse.org/bugs/show_bug.cgi?id=182454 182454] - what is the meaning of ISystemViewElementAdapter.getAbsoluteName()
Line 77: Line 77:
  
 
* '''General Guidelines'''
 
* '''General Guidelines'''
** Breaking API OK this week, but
+
** Avoid breaking API changes after end of this week
*** Avoid breaking API changes if possible afterwards (keep delegate methods, use deprecation)
+
** Junior Jobs: Bugs marked as JJ: in the subject - similar to "helpwanted" keyword
** Kevin, Sanjayan currently doing Vista testing; some more internal RSE stuff
+
** Compiler Warnings - total 669 in all of RSE, as of today:
*** DaveM talking to them beginning of May about coding
+
*** 360 "Discouraged Access"
*** Sanyaan won't be with us for too much longer, but somebody else may be around
+
*** 114 Javadoc:
*** Martin sending out the keyword for marking "junior bugs"
+
***  97 deprecated
 +
*** Compiler warnings compared to [[DSDP/TM/Committer Phone Meeting 30-Jan-2007|30-Jan-2007]]
 +
**** 192 -> 99 rse.ui (mostly javadoc)
 +
**** 94 -> 120 files.ui
 +
**** 27 -> 31 subsystems.files.core
 +
**** 23 -> 18 services (javadoc)
 +
**** 12 -> 14 rse.core (deprecated)
 +
**** 3 connectorservice.dstore
 +
**** (---) -> 143 useractions
 +
**** (---) -> 11 importexport
 +
*** See the [http://www.eclipse.org/dsdp/tm/development/committer_howto.php#check_code Committer Howto] for settings
 +
*** '''The 'discouraged access''' warnings should be looked at since they may indicate API flaws
  
 
== Vacation, Away ==
 
== Vacation, Away ==

Revision as of 09:23, 2 May 2007

Meeting Title: TM Committer Phone Meeting
Date & Time: Tuesday May 2, 2007 at 1500 UTC
Dial-in: International +44 (0)1452 567588
North America +1 (866) 6161738 (toll free)
UK National 08712460713
Passcode: 0587322148 #

MartinO to start conference call - please dial in using the numbers above.
Please be available for Skype Chat in parallel to the call. MartinO will start Skype chat just prior to call.
Skype fallback dial-in - only if less than 5 participants: martin.oberhuber, ddykstal (or david_dykstal), david-k-mcknight, kushal.munir, javier.montalvoorus, tedatteddotnet, michael_scharf, and uwe.stieber.

Contents

Attendees

  • IBM - Dave McKnight, Dave Dykstal, Kushal Munir, Kevin Doyle, Sanjayan Eladchumanasamy
  • Symbian - Javier Montalvo Orús
  • Wind River - Martin Oberhuber, Uwe Stieber, Michael Scharf

This is an Open call, so anyone else can join (though we expect the talk to be interesting for committers only).

Notes

News & Review Action Items

Martin 30% Platform symbolic links; API cleanups; 100%
DaveD 100% Persistence Improvements; UDA 100%
DaveM 40% Bugfixes 40%
Kushal (?) BIDI Encodings; UDA (?)
Javier 50% FTP Parser Extension Point - Problems persisting Property Set 50%
Ted 0% - 0%
Uwe 5% - 5%
Michael 10% 10%

Time Plan and Deadlines toward M7

  • Plan towards M7 - goals and deadlines for I-builds
    • I20070503 -- Minor API changes - Final Feature Additions - Feature Freeze (Terminal integration; FTP Parser)
    • I20070510 -- Stabilization, start TM 2.0 M7 Testing on May 14th
    • I20070516 -- TM 2.0M7 (17th is a public holiday in Austria!)

Bugs and open work to be discussed

  • Current Priorities
    • Priority #1: Finish API Changes
      • Javier: FTP parallel download, allowing to register directory listing parser by extension point
        • What about Commons.Net class in the Extension Point API? What about having extension point in the Service?
      • DaveD: Persistence (for UDA) - independent of IResource?; UI/Non-UI
      • DaveM: 184322 IProgressMonitor added to file system APIs: need to document whether null is allowed as paramter or not, both for IFileService and IRemoteFileSubSystem!!
        • Should we have a common guideline of having the progress monitor first, or last?
      • bug 182221: IRemoteFileSubSystem should forward exceptions rather than handling them with dialogs - Decision was to pragmatically start converting simple methods - who should do it?
      • Martin: 177523 getInstance; 175680 cleanup ISystemRegistry - probably bring more into non-UI; 173042 Refresh; 172650 Capabilities; 176461 expand-to-level; 170910 Integrate Terminalview; 185098 constants for system types; 185097 make getCoreRegistry() simpler
      • Michael: Small API improvements for Terminal (with xyzmodem, 165893
    • Priority #2: Import/Export and User Actions
      • Import/Export should be complete - did anybody except DaveM test it? What about User Docu?
      • AI DaveD, Kushal persistence with User Actions; Dave working on API prerequisites now;
    • Priority #3: Bugs
      • Bugzilla: Severity Major, hi-priority
        • 167620 Select and Delete in the Main thread
      • Encoding Issues: Kushal single encoding control for all subsystems?
      • BIDI bugs: adopting ICU4J? It's part of core RCP already; keep it in UI
        • AI DaveD to follow up on the concrete bug 183631 and fix it, to get experience with ICU4J
      • EFS: Copy(), Move(); Open Remote Project on Startup: Get rid of IResource dependencies where possible; minimal plugin activations; UI/Non-UI
      • 182454 - what is the meaning of ISystemViewElementAdapter.getAbsoluteName()


  • General Guidelines
    • Avoid breaking API changes after end of this week
    • Junior Jobs: Bugs marked as JJ: in the subject - similar to "helpwanted" keyword
    • Compiler Warnings - total 669 in all of RSE, as of today:
      • 360 "Discouraged Access"
      • 114 Javadoc:
      • 97 deprecated
      • Compiler warnings compared to 30-Jan-2007
        • 192 -> 99 rse.ui (mostly javadoc)
        • 94 -> 120 files.ui
        • 27 -> 31 subsystems.files.core
        • 23 -> 18 services (javadoc)
        • 12 -> 14 rse.core (deprecated)
        • 3 connectorservice.dstore
        • (---) -> 143 useractions
        • (---) -> 11 importexport
      • See the Committer Howto for settings
      • The 'discouraged access warnings should be looked at since they may indicate API flaws

Vacation, Away

Action Items

  • Last Meeting Action Items
  • DaveD: Translation Testcases; Persistence Provider without IResource; Get started on ICU4J with bug 183631; User Actions
  • DaveM: Bugs
  • Kushal: BIDI bugs and Encodings
  • Martin: API cleanups; Testing Stats; Refresh improvements; Integrating Terminal with RSE
  • Javier: FTP parallel download, registering FTP Parser by extension point

Next Meeting