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

Apply patch in Sync view

Revision as of 08:06, 3 December 2009 by Tomasz.Zarna.pl.ibm.com (Talk | contribs) (Source code)

Problem

Provide an alternative way of applying patches. Desired behavior, which could in future replace the current one, is to use the Synchronize View to display content of a patch, resolve potential conflicts and apply the patch eventually. This way while the comparison is being done, other features of Eclipse are available to use, we are not bounded to the current Apply Patch wizard.

This is tracked by bug 236169.

Solution

The proposed solution consist of several parts:

  • Subscriber -
  • SyncInfo -
  • IResourceVariant -
  • ModelProvider
    • model objects
    • ResourceMappings
  • org.eclipse.team.ui.synchronizeWizards extension point - Synchronize Participant Creation Wizards
  • org.eclipse.team.ui.synchronizeParticipants extension point -
  • org.eclipse.ui.navigator.navigatorContent extension point -
    • content providers -
    • label providers -
  • org.eclipse.core.runtime.adapters -

Tasks

Required changes are grouped in the following areas:

Synchronize participant, subscriber

  • Add synchronize participant + subscriber
  • IResourceVariant implementation should return patched files when asked for Storage

Synchronize wizard

  • Embed apply patch wizard into synchronize wizard so patch (input) can be selected
  • Add a page to the new wizard where user can select how the patch should be applied (traditionally, preview in the wizard or in Sync view)

Model Provider

  • <strikie>Use DiffProject, FilePatch2 and Hunk as model objects</strike>
  • Use PatchDiffNodes as model objects, they represent the result of applying a patch (not the patch strucutre as types above) and they implement ICompareInput so are compare-ready

Sync view

Context menu

  • Update means 'Apply'
  • No "Exclude" action available, replace with "Remove from the view"

View toolbar

  • 'Copy All Incoming Changes...' means 'Apply All'

Operations

Patch options

  • Re-applying patch -> synchronize again

Source code

No source code is available yet, planning to commit to a branch soon. Or at least submit a dirty patch on the bug.

Back to the top