Uploaded image for project: 'Subversion'
  1. Subversion
  2. SVN-2880

'svn commit --changelist NO_SUCH_CHANGELIST' should warn

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • ---
    • 1.5.0
    • libsvn_client

    Description

      The thread in the URL field starts at this message:
      
         http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=129303
         From: Karl Fogel <kfogel@red-bean.com>
         To: dev@subversion.tigris.org
         Subject: If no such changelist, show we error?
         Date: Fri, 17 Aug 2007 19:31:33 -0700
         Message-Id: <87bqd5bmka.fsf@red-bean.com>
      
      ...and that message sums up the issue like so:
      
      The following IRC conversation says it all:
      
         <kfogel> Is it a deliberate decision that 'svn commit -m logmsg
                  --changelist NON_EXISTENT_CHANGELIST_NAME' does nothing?
                  (It doesn't show any error, and it doesn't commit anything,
                  even if there are local mods, though of course those local
                  mods are not in the named changelist, since it doesn't exist.)
         
         <kfogel> This behavior is true even if you pass --keep-changelist
         
         <epg>    i vote bug; it should error with "no such changelist"
         
         <kfogel> epg: I agree.
         
         <kfogel> Hmm, I'll ask dev@, just in case there was some esoteric
                  reason why this behavior was deliberate.
      
      The thread takes some interesting twists and turns, but did agree that this
      particular case is a bug.
      

      http://subversion.tigris.org/servlets/BrowseList?list=dev&by=thread&from=602449

      Attachments

        1. 1_patch.txt
          4 kB
          Daniel Rall
        2. 2_msg3.txt
          0.4 kB
          Daniel Rall

        Activity

          People

            Unassigned Unassigned
            kfogel Karl Fogel
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: