#4 Error trapping on cvsroot parse should be improved.

closed
cvs (10)
5
2003-11-24
2003-09-16
Anonymous
No

The attached build file generates an exception.
Barring the fact that the :ssci protocol is not
supported the error handling for parsing cvsroot should
be improved.

Discussion

  • Clayton Harbour

    Clayton Harbour - 2003-09-16
     
    Attachments
  • Clayton Harbour

    Clayton Harbour - 2003-11-24

    Logged In: YES
    user_id=677222

    Following exception is outprinted executing following task:

    <cvs-checkout destination="${deploy.dir}"
    cvsroot="${cvs.root}" module="${module.name}" />

    where cvs.root is :local:v:/cvsrep . and modulename is
    PROJ or PROJ/SUB. Of course these directories are exists in
    the V:/cvsrep directory.

    Same exception ifs thrown when module name do not match
    existing in CVS in your sample tag.

    BUILD FAILED

    INTERNAL ERROR

    System.ArgumentOutOfRangeException: Length cannot be less
    than zero. Parameter name: length
    at System.String.Substring(Int32 startIndex, Int32 length)
    at ICSharpCode.SharpCvsLib.Misc.CvsRoot..ctor(String cvsroot)
    at NAnt.SourceControl.Tasks.AbstractCvsTask.ExecuteTask()
    at NAnt.Core.Task.Execute()
    at NAnt.Core.Target.Execute()
    at NAnt.Core.Project.Execute(String targetName)
    at NAnt.Core.Project.Execute()
    at NAnt.Core.Project.Run()

     
  • Clayton Harbour

    Clayton Harbour - 2003-11-24
    • assigned_to: nobody --> drakmar
     
  • Clayton Harbour

    Clayton Harbour - 2003-11-24

    Logged In: YES
    user_id=677222

    Now an invalid cvsroot throws a CvsRootParseException rather
    than throwing an ArgumentOutOfRangeException. Also a
    discription of the error is printed out. Finally the
    addition of a username is dependant on protocol, so although
    there are currently no protocols supported that do not
    require a username the parser will handle this.

     
  • Clayton Harbour

    Clayton Harbour - 2003-11-24
    • status: open --> closed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks