Hi James,

Can you show me the "Error uploading diff" error when using post-review with --debug?


Christian


James Oravec wrote:
Got past this issue. It was related to my connection string in my CVS
directory's "Root" file. I was modifying it, trying to get past a
different error. The summary is my connection string was incorrect, so
when I did the "post-review --output-diff" it wasn't able to connect,
thus why it had the empty diff.

I'm now back to working on my "Error uploading diff" issue.

On Tuesday, July 2, 2013 9:38:40 AM UTC-7, James Oravec wrote:

    Hi Christian,

    Did you guys ever figure out the issue with this? I'm receiving the
    same errors both on the client and server side. Any help would be
    greatly appreciated.

    Thanks,
    James



    On Monday, September 26, 2011 4:59:39 PM UTC-7, Christian Hammond wrote:

        Hi Matthew,

        So I've seen this issue mentioned before with Review Board, but
        also with CVS in general. It's possible it's us. Can you perhaps
        privately e-mail me with the Path and Mirror Path field info on
        your Repository entry?

        Here's a link to someone who's hit it with just plain ol' CVS
        and has some workaround (though it's not very detailed):

        http://devnet.jetbrains.net/thread/158342
        <http://devnet.jetbrains.net/thread/158342>

        Christian

        --
        Christian Hammond - chi...@chipx86.com
        Review Board - http://www.reviewboard.org
        VMware, Inc. - http://www.vmware.com


        On Mon, Sep 26, 2011 at 4:26 PM, Matthew <matt...@gmail.com> wrote:

            I appreciate the quick help.

            After enabling logging and running the commands again I get
            the output
            below. I notice the absolute module reference points into
            the cvs
            repository. After checking in the repository, there is no
            "Version.java", although there is "Version.java,v". I don't
            know if
            it's significant that this repository is a new import of the
            WorldWind
            code for Review Board testing purposes, the only file that
            has ever
            been checked out is "Version.java", and it has never been
            checked back
            in.

            Matt (log follows)

            [lihm@rbvm3 WorldWind]$ more /var/www/myserver.com/logs/
            <http://myserver.com/logs/>
            reviewboard.log
            2011-09-26 16:12:33,711 - DEBUG - DiffParser.parse:
            Beginning parse of
            diff, size = 2741
            2011-09-26 16:12:33,712 - DEBUG - DiffParser.parse: Finished
            parsing
            diff.
            2011-09-26 16:12:33,832 - ERROR - Error uploading new diff: cvs
            [checkout aborted]: Absolute module reference invalid:
            `/usr/local/c
            vsroot/WorldWind/src/gov/nasa/worldwind/Version.java'
            Traceback (most recent call last):
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/webapi/resources.py", line 1478, in create
            request.FILES.get('parent_diff_path'))
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/reviews/forms.py", line 286, in create
            history)
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/diffviewer/forms.py", line 62, in create
            files = list(self._process_files(
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/diffviewer/forms.py", line 144, in _process_files
            if (revision != PRE_CREATION and
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/scmtools/core.py", line 70, in file_exists
            self.get_file(path, revision)
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/scmtools/cvs.py", line 48, in get_file
            return self.client.cat_file(path, revision)
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/scmtools/cvs.py", line 248, in cat_file
            return self._cat_specific_file(filename, revision)
            File
            "/usr/lib/python2.4/site-packages/ReviewBoard-1.6.1-py2.4.egg/
            reviewboard/scmtools/cvs.py", line 299, in _cat_specific_file
            raise SCMError(errmsg)
            SCMError: cvs [checkout aborted]: Absolute module reference
            invalid: `/
            usr/local/cvsroot/WorldWind/src/gov/nasa/worldwind/Version.ja
            va'



            On Sep 26, 4:05 pm, Christian Hammond <chip...@chipx86.com>
            wrote:
             > Hi Matthew,
             >
             > It's entirely possible that that error is misleading.
            Looking at the code,
             > it's possible that the same error code could be generated
            when there's also
             > an unexpected error on Review Board's end.
             >
             > If you have access to the reviewboard server logs, look
            for a line starting
             > with "Error uploading new diff:"
             >
             > That might give a better idea as to what's going on. Let
            me know what it
             > says.
             >
             > Christian
             >
             > --
             > Christian Hammond - chip...@chipx86.com
             > Review Board -http://www.reviewboard.org
             > VMware, Inc. -http://www.vmware.com
             >
             >
             >
             >
             >
             >
             >
             > On Mon, Sep 26, 2011 at 3:57 PM, Matthew
            <matt....@gmail.com> wrote:
             > > The cvs command issued by post-review executes, but
            post-review
             > > doesn't seem to think there are any diffs to upload.
            I'll include the
             > > result of post-review --output-diff --debug at the end
            of this message
             >
             > > Although I have a software background it has been a
            long time since
             > > I've done any sort of sysadmin work, and I'm not
            familiar with cvs or
             > > python. So please use small words.
             >
             > > Any help would be appreciated.
             >
             > > Matt (post-review session below, had to change the
            server name, but
             > > everything else is copied from xterm)
             >
             > > [lihm@rbvm3 WorldWind]$ post-review --output-diff --debug
             > > >>> RBTools 0.3.3
             > > >>> Home = /home/lihm
             > > >>> svn info
             > > >>> repository info: Path:
            myserver.com:8080/usr/local/cvsroot
            <http://myserver.com:8080/usr/local/cvsroot>, Base path:
             > > None, Supports changesets: False
             > > >>> HTTP GETting api/
             > > >>> HTTP GETtinghttp://myserver.com/rbmain/api/info/
            <http://myserver.com/rbmain/api/info/>
             > > >>> Using the new web API
             > > >>> cvs diff -uN
             > > cvs diff: Diffing .
             > > cvs diff: Diffing src
             > > cvs diff: Diffing src/config
             > > cvs diff: Diffing src/gov
             > > cvs diff: Diffing src/gov/nasa
             > > cvs diff: Diffing src/gov/nasa/worldwind
             > > Index: src/gov/nasa/worldwind/Version.java
             > >
            ===================================================================
             > > RCS file:
            /usr/local/cvsroot/WorldWind/src/gov/nasa/worldwind/
             > > Version.java,v
             > > retrieving revision 1.1.1.1
             > > diff -u -r1.1.1.1 Version.java
             > > --- src/gov/nasa/worldwind/Version.java 7 Sep 2011 18:17:04
             > > -0000 1.1.1.1
             > > +++ src/gov/nasa/worldwind/Version.java 26 Sep 2011
            22:34:40 -0000
             > > @@ -11,6 +11,8 @@
             > > * @version $Id: Version.java,v 1.1.1.1 2011/09/07
            18:17:04 root Exp
             > > $
             > > */
             >
             > > +// MML 20110916 Review Board Test
             > > +
             > > public class Version
             > > {
             > > private static final String MAJOR_VALUE = "0";
             > > cvs diff: Diffing src/gov/nasa/worldwind/applications
             > > cvs diff: Diffing src/gov/nasa/worldwind/applications/sar
             > > cvs diff: Diffing src/gov/nasa/worldwind/avlist
             > > cvs diff: Diffing src/gov/nasa/worldwind/awt
             > > cvs diff: Diffing src/gov/nasa/worldwind/cache
             > > cvs diff: Diffing src/gov/nasa/worldwind/event
             > > cvs diff: Diffing src/gov/nasa/worldwind/examples
             > > cvs diff: Diffing src/gov/nasa/worldwind/examples/applet
             > > cvs diff: Diffing src/gov/nasa/worldwind/exception
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/csv
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/dds
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/georss
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/gpx
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/nitfs
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/nmea
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/rpf
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/tiff
             > > cvs diff: Diffing src/gov/nasa/worldwind/formats/wvt
             > > cvs diff: Diffing src/gov/nasa/worldwind/geom
             > > cvs diff: Diffing src/gov/nasa/worldwind/geom/coords
             > > cvs diff: Diffing src/gov/nasa/worldwind/globes
             > > cvs diff: Diffing src/gov/nasa/worldwind/issues
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers/Earth
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers/Mars
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers/Moon
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers/placename
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers/rpf
             > > cvs diff: Diffing src/gov/nasa/worldwind/layers/rpf/wizard
             > > cvs diff: Diffing src/gov/nasa/worldwind/pick
             > > cvs diff: Diffing src/gov/nasa/worldwind/render
             > > cvs diff: Diffing src/gov/nasa/worldwind/retrieve
             > > cvs diff: Diffing src/gov/nasa/worldwind/tracks
             > > cvs diff: Diffing src/gov/nasa/worldwind/util
             > > cvs diff: Diffing src/gov/nasa/worldwind/util/wizard
             > > cvs diff: Diffing src/gov/nasa/worldwind/view
             > > cvs diff: Diffing src/gov/nasa/worldwind/wms
             > > cvs diff: Diffing src/images
             > > [lihm@rbvm3 WorldWind]$ post-review
             >
             > > Error uploading diff
             >
             > > The generated diff file was empty. This usually means
            no files were
             > > modified in this change.
             >
             > > Try running with --output-diff and --debug for more
            information.
             >
             > > Your review request still exists, but the diff is not
            attached.
             > > [lihm@rbvm3 WorldWind]$ ls -l
             >
             > > --
             > > Want to help the Review Board project? Donate today at
             > >http://www.reviewboard.org/donate/
            <http://www.reviewboard.org/donate/>
             > > Happy user? Let us know
            athttp://www.reviewboard.org/users/
            <http://www.reviewboard.org/users/>
             > > -~----------~----~----~----~------~----~------~--~---
             > > To unsubscribe from this group, send email to
             > > reviewboard...@googlegroups.com
             > > For more options, visit this group at
             > >http://groups.google.com/group/reviewboard?hl=en
            <http://groups.google.com/group/reviewboard?hl=en>

            --
            Want to help the Review Board project? Donate today at
            http://www.reviewboard.org/donate/
            <http://www.reviewboard.org/donate/>
            Happy user? Let us know at http://www.reviewboard.org/users/
            <http://www.reviewboard.org/users/>
            -~----------~----~----~----~------~----~------~--~---
            To unsubscribe from this group, send email to
            reviewboard...@googlegroups.com
            For more options, visit this group at
            http://groups.google.com/group/reviewboard?hl=en
            <http://groups.google.com/group/reviewboard?hl=en>


--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en
---
You received this message because you are subscribed to the Google
Groups "reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send
an email to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.



--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com

--
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
--- You received this message because you are subscribed to the Google Groups "reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to