Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [mylyn-versions-dev] ChangeSet missing base version ?

Ok, After some more thinking and discussions, I think we need to have
a base and target version,
see Bug 334784

/Alvaro


On Wed, Jan 19, 2011 at 12:24 PM, alvaro sanchez <alvsan09@xxxxxxxxx> wrote:
> I discussed it with Kilian and Matthias and we agreed to leave the
> Versions interface simple to one artifact per entry in the changeset
> and leave the application to extend it if needed.
>
> In the case of git, the predecessor is normally the one used for
> compare and can be resolved from the artifact.
>
> For CVS and base clear case, the predecessor is normally not the one
> you are willing to compare against, however the actual Review
> application can store a selected base within its own model so the
> review model for this case will need to keep the reference of two
> artifacts from the version control system.
>
> In this way the Versions model remains simple and represents the
> structure in a generic way.
>
> So lets continue in this way for the moment and will re-open the
> discussion with a bug when an issue comes up in this area.
>
> /Alvaro
>
>
> On Tue, Jan 18, 2011 at 10:15 PM, Steffen Pingel
> <steffen.pingel@xxxxxxxxxxx> wrote:
>> The model does not currently have a representation for that.
>> ScmArtifact supports resolving different revisions but revisions are
>> currently not stored in changesets. Maybe changesets should hold on to
>> change set items that point to the previous and next revision? Can you
>> file a bug and propose a solution?
>>
>> Steffen
>>
>>
>> On Tue, Jan 18, 2011 at 1:00 PM, alvaro sanchez <alvsan09@xxxxxxxxx> wrote:
>>> Hello,
>>> The current model has a ChangeSet element that contains a list of
>>> ScmArtifact, I assume the ScmArtifact is at the file level, however I
>>> see ScmArtifact is able to resolve to one FileRevision only, how do we
>>> preserve the relation between base and target ?
>>>
>>> /Alvaro
>>> _______________________________________________
>>> mylyn-versions-dev mailing list
>>> mylyn-versions-dev@xxxxxxxxxxx
>>> http://dev.eclipse.org/mailman/listinfo/mylyn-versions-dev
>>>
>>
>>
>>
>> --
>> Steffen Pingel
>> Committer, http://eclipse.org/mylyn
>> Senior Developer, http://tasktop.com
>> _______________________________________________
>> mylyn-versions-dev mailing list
>> mylyn-versions-dev@xxxxxxxxxxx
>> http://dev.eclipse.org/mailman/listinfo/mylyn-versions-dev
>>
>


Back to the top