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

svn_repos_parse_fns2_t.close_revision doesn't support svn_fs_commit_txn() contract

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • trunk
    • 1.10-consider
    • libsvn_repos

    Description

      svn_fs_commit_txn() and svn_repos_fs_commit_txn() allow a
      transaction to be successfully committed and also return an
      error, that may be the result of some post-commit FS jiggling or
      a post-commit script failing.
      
      The svn_repos_parse_fns2_t's close_revision field doesn't allow
      this to be expressed, it only returns an svn_error_t, which
      requires that either post-commit errors be swallowed or wrapped
      to indicate that the commit succeeded.  Instead of doing either
      of these, a svn_repos_parse_fns3_t should be introduced with a
      close_revision field:
      
        svn_error_t *(*close_revision)(svn_revnum_t *new_rev, void *revision_baton);  
      
      Since svn_repos_get_fs_build_parser3 was revved and added in 1.7,
      ideally it should be modified to use svn_repos_parse_fns3_t.
      

      http://svn.haxx.se/dev/archive-2010-12/0437.shtml

      Attachments

        Activity

          People

            Unassigned Unassigned
            blair Blair Zajac
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: