[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [Subclipse-users] Subclipse's SyncInfo.getLocalContentIdentifier() returns "BASE", not revision number

From: Mark Phippard <markphip_at_gmail.com>
Date: 2007-04-04 19:08:33 CEST

On 4/4/07, Adrian Price <adrianp@quatinus.myzen.co.uk> wrote:
> Good day Subclipse Users,
>
> I am developing a generic team integration enhancement to the Jupiter
> (http://csdl.ics.hawaii.edu/Tools/Jupiter/) code review
> plug-in. The goal is to track active change sets in order to auto-populate a
> code review request for the files in a change set. I need to track the
> revision number for each file in a change set and to do this have been using
> code like:
>
> String version = null;
> for (Iterator it = activeChangeSetManagers.iterator();
> it.hasNext();) {
> ActiveChangeSetManager activeChgSetMgr = (ActiveChangeSetManager) it
> .next();
> if (activeChgSetMgr instanceof SubscriberChangeSetManager) {
> Subscriber subscriber = ((SubscriberChangeSetManager)
> activeChgSetMgr)
> .getSubscriber();
> try {
> SyncInfo syncInfo = subscriber.getSyncInfo(resource);
> if (syncInfo != null) {
> version = syncInfo.getLocalContentIdentifier();
> break;
>
> I find that the call to
> SyncInfo.getLocalContentIdentifier() always returns the
> string "BASE" for a versioned file, not the revision number. The CVS Team
> Provider always returns the CVS version number and it seems to me that it
> would be preferable (from my perspective at least) for Subclipse to be
> consistent, i.e., to return the SVN revision number. Source-level debugging
> suggests that the required revision number is available in the called
> context and could probably be returned with minimal effort.
>
> Does anyone have any opinions or insights on this?

When I look at that method in SVNStatusSyncInfo it shows this
implementation which should return a revision number.

        public String getLocalContentIdentifier() {
                return (baseStatusInfo.getLastChangedRevision() != null) ?
baseStatusInfo.getLastChangedRevision().toString() : null;
        }

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subclipse.tigris.org
For additional commands, e-mail: users-help@subclipse.tigris.org
Received on Wed Apr 4 19:08:50 2007

This is an archived mail posted to the Subclipse Users mailing list.