Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 350734 - Refresh Branches operation now fails when encountering certain files
Summary: Refresh Branches operation now fails when encountering certain files
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: CVS (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: platform-cvs-inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
: 350608 (view as bug list)
Depends on:
Blocks: 350608
  Show dependency tree
 
Reported: 2011-06-29 14:00 EDT by Neil Hauge CLA
Modified: 2019-11-14 03:18 EST (History)
2 users (show)

See Also:


Attachments
Problematic file (79 bytes, application/octet-stream)
2011-06-29 14:00 EDT, Neil Hauge CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Neil Hauge CLA 2011-06-29 14:00:10 EDT
Created attachment 198850 [details]
Problematic file

After moving up from Helios to Indigo it was soon noticed that previously existing CVS branches in WTP could not be displayed for a particular module using Eclipse 3.7.  An exception was logged (1).  I entered bug 350608 as a result as I was unsure of the cause of this issue.  After doing some debugging I found that a particular file (attached) in our repository was causing the exception.  This file looked a bit questionable, but it hadn't caused problems in the past.  After some more digging, it appears that changes made for bug 76386 were the cause of this issue occurring only in 3.7.

In LogListener, line 199, the new call to this.getBranchRevision(revision) appears to assume that the revision will be an actual revision, and not a COMMENT of BRANCH_REVISION.  I don't claim to fully understand the workings of the containing method, but thought I would point out where the problem appears to be. 

I was able to work around this problem by removing the offending file from the repository, but this of course is generally not an ideal solution.

(1) 
java.lang.IllegalArgumentException: Revision malformed: branchRevision
    at
org.eclipse.team.internal.ccvs.core.client.listeners.LogListener.getBranchRevision(LogListener.java:262)
    at
org.eclipse.team.internal.ccvs.core.client.listeners.LogListener.messageLine(LogListener.java:199)
Comment 1 David Williams CLA 2011-09-21 13:12:08 EDT
*** Bug 350608 has been marked as a duplicate of this bug. ***
Comment 2 Missing name Mising name CLA 2012-03-19 07:32:15 EDT
(In reply to comment #0)
> Created attachment 198850 [details]
> Problematic file
> 
> After moving up from Helios to Indigo it was soon noticed that previously
> existing CVS branches in WTP could not be displayed for a particular module
> using Eclipse 3.7.  An exception was logged (1).  I entered bug 350608 as a
> result as I was unsure of the cause of this issue.  After doing some debugging
> I found that a particular file (attached) in our repository was causing the
> exception.  This file looked a bit questionable, but it hadn't caused problems
> in the past.  After some more digging, it appears that changes made for bug
> 76386 were the cause of this issue occurring only in 3.7.
> 
> In LogListener, line 199, the new call to this.getBranchRevision(revision)
> appears to assume that the revision will be an actual revision, and not a
> COMMENT of BRANCH_REVISION.  I don't claim to fully understand the workings of
> the containing method, but thought I would point out where the problem appears
> to be. 
> 
> I was able to work around this problem by removing the offending file from the
> repository, but this of course is generally not an ideal solution.
> 
> (1) 
> java.lang.IllegalArgumentException: Revision malformed: branchRevision
>     at
> org.eclipse.team.internal.ccvs.core.client.listeners.LogListener.getBranchRevision(LogListener.java:262)
>     at
> org.eclipse.team.internal.ccvs.core.client.listeners.LogListener.messageLine(LogListener.java:199)

I'm running into the same problem. How did you find the file causing the error?
Comment 3 Neil Hauge CLA 2012-03-20 16:45:48 EDT
(In reply to comment #2)
> (In reply to comment #0)
> 
> I'm running into the same problem. How did you find the file causing the error?

I had to debug the org.eclipse.team.internal.ccvs code, and probably put a breakpoint on IllegalArgumentException.
Comment 4 Lars Vogel CLA 2019-11-14 03:18:02 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

If the bug is still relevant, please remove the "stalebug" whiteboard tag.