Skip to main content

Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.

Jump to: navigation, search

Difference between revisions of "DSDP/TM/Committer Phone Meeting 30-Aug-2006"

< DSDP‎ | TM
(Action Items)
Line 27: Line 27:
 
* Latest News
 
* Latest News
 
** Martin -- compiler warnings, extension point cleanup (*.exsd doc), scheduled release review
 
** Martin -- compiler warnings, extension point cleanup (*.exsd doc), scheduled release review
** DaveD --  
+
** DaveD -- not much time this week, JUnit submitted
** DaveM --  
+
** DaveM -- not much time either, fixed a few bugs, going to do drag&drop next
** Kushal --  
+
** Kushal -- not much time either;
 +
*** rseConfigDefaults: IBM specific config.ini, setting System Properties; Kushal to document this (Reference docs for System Properties in ISV docs); Write Migration docs for IBM;
 +
*** passwordPersistence: Dont allow password add/edit on the Prefs page, only remove. Do any IBM products (TPF) pre-populate the passwords? Martin: probably use an "earlyStartup" extension point?
 +
*** refactoring: persistence provider is very tightly integrated with model objects like filters, IHost, ISystemProfile etc. Do System
 +
**** DaveM to try moving interfaces on Thursday; others, NO CHECKIN on Thursday!
 
** Javier -- vacation
 
** Javier -- vacation
 
** Other News --
 
** Other News --
Line 35: Line 39:
 
** '''M4 bugs''': PLEASE REVIEW [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&component=RSE&target_milestone=1.0+M4&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&cmdtype=doit M4 BUGS]!!
 
** '''M4 bugs''': PLEASE REVIEW [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&component=RSE&target_milestone=1.0+M4&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&cmdtype=doit M4 BUGS]!!
 
** '''Inbox''': REVIEW [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&component=RSE&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&emailassigned_to1=1&emailtype1=exact&email1=dsdp.tm.rse-inbox%40eclipse.org&cmdtype=doit Assigned to Inbox]
 
** '''Inbox''': REVIEW [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&component=RSE&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&emailassigned_to1=1&emailtype1=exact&email1=dsdp.tm.rse-inbox%40eclipse.org&cmdtype=doit Assigned to Inbox]
** '''Compiler Warnings''': See [http://www.eclipse.org/dsdp/tm/development/compiler_warnings.php Committer Howto]
+
** '''Timing''': Martin on vacation most of October; Release Review Sept.27, M5 is Sept.22
*** Currently 484 Java warnings + 642 Javadoc warnings (Martin already fixed about 150 + 250)
+
*** Martins fixes will be in by Sept.27; manual test plans will exist
*** Find bugs, Increase contributor confidence, Improve code readability, Fix javadoc
+
*** Executing the test plans, and bugfixing after that to be done by IBM mostly
*** Dont chase the warnings (yet) but fix them while you are working: You can apply a filter in the Problems view if you want, e.g. "don't show String Javadoc:"
+
*** Currently 142 bugs open, 18 P1/P2: P1/P2 bugs MUST be fixed by Sept. 15!
*** No more checkins of code with warnings (may be checked automatically in the future)
+
** '''Source Code Style & Cleanness'''
*** Be '''careful''' when fixing warnings: assume side effects of method calls!
+
*** We deliver not only a "working product" but also source code
** '''Code Ownership''':
+
*** Clean source code increases contributor confidence, improves code readability, helps avoiding bugs
*** It is beneficial to have clear ownership and responsibility
+
*** '''Compiler Warnings''': See [http://www.eclipse.org/dsdp/tm/development/compiler_warnings.php Committer Howto]
*** External contributors can also be owners, but have an associated committer as well
+
**** Currently 484 Java warnings + 642 Javadoc warnings (Martin already fixed about 150 + 250)
*** See (and edit) [[DSDP/TM/Code Ownership]]
+
**** Find bugs, Fix javadoc
 +
**** Dont chase the warnings (yet) but fix them while you are working: You can apply a filter in the Problems view if you want, e.g. "don't show String Javadoc:"
 +
**** No more checkins of code with warnings (may be checked automatically in the future)
 +
**** Be '''careful''' when fixing warnings: assume side effects of method calls!
 +
*** '''Code Ownership''':
 +
**** It is beneficial to have clear ownership and responsibility
 +
**** External contributors can also be owners, but have an associated committer as well
 +
**** See (and edit) [[DSDP/TM/Code Ownership]]
 +
**** Martin to take Ssh, Releng; Examples; FTP, Local, Contentassist for now
 +
*** '''Code Style and Format''':
 +
**** Not been a focus so far
 +
**** RSE itself is inconsistent - e.g. if... without {} block is problematic
 +
**** Use JDT autoformat as much as possible: perhaps define an RSE-style and check in with the projects?
 +
**** Or use the standard eclipse style: might be problematic when merging RSE7 code into openRSE
 +
** '''Change Requests'''
 +
*** Default colors for command view - white background, black or dark blue foreground
 +
*** Remove rseConfigDefaults extension point - Kushal
 +
*** Remove passwordPersistence extension point - Kushal
 +
*** Deprecated getRemoteProperty() --> Document property namespaces, reserve "org.eclipse.rse" namespace
 
** '''Features''': Jakarta-commons, JUnit-framework, Discovery, EFS-Experimental, WR-Terminal ([https://bugs.eclipse.org/bugs/show_bug.cgi?id=152826 bug 152826])
 
** '''Features''': Jakarta-commons, JUnit-framework, Discovery, EFS-Experimental, WR-Terminal ([https://bugs.eclipse.org/bugs/show_bug.cgi?id=152826 bug 152826])
 
** '''Refactorings''': This week we should really be done!
 
** '''Refactorings''': This week we should really be done!
 
*** Subsystem Factory vs. Configuration: Going with Configuration for now, Martin started renaming *factory where he found it while working on compiler warnings.
 
*** Subsystem Factory vs. Configuration: Going with Configuration for now, Martin started renaming *factory where he found it while working on compiler warnings.
 
*** Kushal IConnectorService
 
*** Kushal IConnectorService
*** Kushal passwordPersistence
 
 
** API Things
 
** API Things
 
*** Martin did some minor extension point cleanup (rename), see [http://download.eclipse.org/dsdp/tm/downloads/drops/N20060829-0100/buildNotes.php build notes of N-builds]
 
*** Martin did some minor extension point cleanup (rename), see [http://download.eclipse.org/dsdp/tm/downloads/drops/N20060829-0100/buildNotes.php build notes of N-builds]
Line 59: Line 80:
 
*** Making stuff '''internal''' - ssh, ftp: Martin; Core: List of candidates from DaveD?
 
*** Making stuff '''internal''' - ssh, ftp: Martin; Core: List of candidates from DaveD?
 
*** Obsolete API that's in the Platform now --> See [[RSE 2.0 Planning]] page
 
*** Obsolete API that's in the Platform now --> See [[RSE 2.0 Planning]] page
**** Kushal to investigate rseConfigDefaults
 
 
** Cleanup: API Doc (duplicates in interface and implementation!), JUnit tests, Context help, Test Plans, RSE-7-migration-docs, Copyrights, licenses
 
** Cleanup: API Doc (duplicates in interface and implementation!), JUnit tests, Context help, Test Plans, RSE-7-migration-docs, Copyrights, licenses
 
** '''Bug Work - please observe priorities'''
 
** '''Bug Work - please observe priorities'''
Line 70: Line 90:
 
** EFS Experimental feature to be created by Martin
 
** EFS Experimental feature to be created by Martin
 
* Free discussion -- feelings, comments, critics
 
* Free discussion -- feelings, comments, critics
** Want to do a committer call next week in addition to the open call?
+
** Want to do a committer call next week in addition to the open call
 +
** Monday - Labor Day in US + Canada
  
 
== Action Items ==
 
== Action Items ==

Revision as of 10:44, 30 August 2006

Meeting Title: TM Committer Phone Meeting
Date & Time: Wednesday Aug 30, 2006 at 8.30am Rochester / 9.30am Toronto / 2.30pm London / 3.30pm Salzburg
Dial-in: Martin to start a Skype conference - you'll be called:
Skype martin.oberhuber, ddykstal (or david_dykstal), david-k-mcknight, kushal_munir, javier.montalvoorus

Fixed-line fallback dial-in:

  • International +1 314-655-1411
  • North America +1 877-422-0035 (toll free)
  • Passcode: 764918

Attendees

  • IBM - Dave Dykstal, Dave McKnight, Kushal Munir
  • (Symbian - Javier Montalvo Orús - vacation)
  • Wind River - Martin Oberhuber

Agenda

  • Our top goals
    • Grow the Communities - active users and adopters --> tutorial, docs, mailinglist help
    • Get the APIs Right --> enable public API discussion --> ISV docs, Wiki API discussion, [api] bugzilla's
    • Get our Processes in place --> JUnit, nightly builds, infocenter, update site
  • Latest News
    • Martin -- compiler warnings, extension point cleanup (*.exsd doc), scheduled release review
    • DaveD -- not much time this week, JUnit submitted
    • DaveM -- not much time either, fixed a few bugs, going to do drag&drop next
    • Kushal -- not much time either;
      • rseConfigDefaults: IBM specific config.ini, setting System Properties; Kushal to document this (Reference docs for System Properties in ISV docs); Write Migration docs for IBM;
      • passwordPersistence: Dont allow password add/edit on the Prefs page, only remove. Do any IBM products (TPF) pre-populate the passwords? Martin: probably use an "earlyStartup" extension point?
      • refactoring: persistence provider is very tightly integrated with model objects like filters, IHost, ISystemProfile etc. Do System
        • DaveM to try moving interfaces on Thursday; others, NO CHECKIN on Thursday!
    • Javier -- vacation
    • Other News --
  • Upcoming Work
    • M4 bugs: PLEASE REVIEW M4 BUGS!!
    • Inbox: REVIEW Assigned to Inbox
    • Timing: Martin on vacation most of October; Release Review Sept.27, M5 is Sept.22
      • Martins fixes will be in by Sept.27; manual test plans will exist
      • Executing the test plans, and bugfixing after that to be done by IBM mostly
      • Currently 142 bugs open, 18 P1/P2: P1/P2 bugs MUST be fixed by Sept. 15!
    • Source Code Style & Cleanness
      • We deliver not only a "working product" but also source code
      • Clean source code increases contributor confidence, improves code readability, helps avoiding bugs
      • Compiler Warnings: See Committer Howto
        • Currently 484 Java warnings + 642 Javadoc warnings (Martin already fixed about 150 + 250)
        • Find bugs, Fix javadoc
        • Dont chase the warnings (yet) but fix them while you are working: You can apply a filter in the Problems view if you want, e.g. "don't show String Javadoc:"
        • No more checkins of code with warnings (may be checked automatically in the future)
        • Be careful when fixing warnings: assume side effects of method calls!
      • Code Ownership:
        • It is beneficial to have clear ownership and responsibility
        • External contributors can also be owners, but have an associated committer as well
        • See (and edit) DSDP/TM/Code Ownership
        • Martin to take Ssh, Releng; Examples; FTP, Local, Contentassist for now
      • Code Style and Format:
        • Not been a focus so far
        • RSE itself is inconsistent - e.g. if... without {} block is problematic
        • Use JDT autoformat as much as possible: perhaps define an RSE-style and check in with the projects?
        • Or use the standard eclipse style: might be problematic when merging RSE7 code into openRSE
    • Change Requests
      • Default colors for command view - white background, black or dark blue foreground
      • Remove rseConfigDefaults extension point - Kushal
      • Remove passwordPersistence extension point - Kushal
      • Deprecated getRemoteProperty() --> Document property namespaces, reserve "org.eclipse.rse" namespace
    • Features: Jakarta-commons, JUnit-framework, Discovery, EFS-Experimental, WR-Terminal (bug 152826)
    • Refactorings: This week we should really be done!
      • Subsystem Factory vs. Configuration: Going with Configuration for now, Martin started renaming *factory where he found it while working on compiler warnings.
      • Kushal IConnectorService
    • API Things
      • Martin did some minor extension point cleanup (rename), see build notes of N-builds
      • Service Error Reporting - bug 149472 - DaveD - API ok so far, final resolution? --> Create new different bug id for that one.
      • Parallel Services - bug 142478 all queries in background - DaveM
      • SystemRegistry.createHost for Javier
        • bug 150168 dont commit immediately - DaveD
        • bug 150265 allow specifying subsystem types - DaveM - wait for Javier to return
      • Making stuff internal - ssh, ftp: Martin; Core: List of candidates from DaveD?
      • Obsolete API that's in the Platform now --> See RSE 2.0 Planning page
    • Cleanup: API Doc (duplicates in interface and implementation!), JUnit tests, Context help, Test Plans, RSE-7-migration-docs, Copyrights, licenses
    • Bug Work - please observe priorities
    • Legal: JUnit (next monday?); Jakarta-commons (deferred!); WR Terminalview (passed!)
  • Other items
    • Tutorial: Update at the end of the release process, change instructions (re-use example code: We dont want to teach PDE)
    • Manual test plans - in work by Martin
    • EFS Experimental feature to be created by Martin
  • Free discussion -- feelings, comments, critics
    • Want to do a committer call next week in addition to the open call
    • Monday - Labor Day in US + Canada

Action Items

  • Last Meeting Action Items
  • DaveD - JUnit, Review bugs assigned to M4, Checkin mappings.csv, New bug for moving DTD, Send "Team" test plans to Martin. SystemRegistry API. Team fixes: Check collapsing persistence Properties nodes to fewer files. Compile a list of suggestions for making classes / packages internal.
  • DaveM - bug fixing by priority; look at Kushal's bugs and assign yourself; get rid of Service calls on UI thread; hygiene changes
  • Kushal - Review bugs assigned to M4, refactoring IConnectorService / Persistence provider (send out proposal); send file encoding test resources to martin; review & get rid of rseConfigDefaults; talk to Don about persistenceProvider; bug fixing
  • Martin - EFS feature, Add Discovery, Build scripts for CDT and Discovery, Manual test plan, Release Review Slides, API Review, Jakarta-commons, WR-terminalview; Review if IShellService is sufficient for terminal
  • Javier - vacation; hook up with Scott Lewis once SD is committed
  • Everyone - List obsolete API on RSE 2.0 Planning page

Next Meeting

Back to the top