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.
Difference between revisions of "DSDP/TM/Committer Phone Meeting 30-Aug-2006"
(→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 | + | *** 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 | ||
− | |||
** 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 | ||
− | |||
** 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
Contents
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
- DaveD: Team stuff
- Open bugs: P1, P2 bugs, API bugs, Open with Patch,
- 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
- DSDP/TM/Committer Phone Meeting 6-Sep-2006 at 9.30am Toronto
- Open DSDP/TM/Phone Meeting 6-Sep-2006 at 9am PST