Sorry, we can't get you an early patch for this just yet. We'd have to do some research and testing, and right now we're swamped with work for some of our premium support customers. However, if your team wants to investigate this and write a patch, we can review it on https://reviews.reviewboard.org/.
In the meantime, please file a bug on https://hellosplat.com/s/beanbag/tickets/ so we can track this. Christian -- Christian Hammond - [email protected] Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On Wed, Nov 18, 2015 at 3:02 PM, Wesly Chen <[email protected]> wrote: > In the error message, > 2015-11-18 01:34:33,409 - ERROR - - Failed to apply pygments: no lexer > for filename u'/home/cvs/OPS/nagios/libexec/check_mysql_multirow.pl,v' > found > > Review Board might need to add a filename check for CVS repository and > remove the trailing ",v" then pass the filename without ",v" to > "_apply_pygments". > > In a short term, could you pass me the patch or fix before release since > we have an issue to upgrade from 2.0.20 to 2.5.11. > > Thanks, > > Wesly > > Wesly Chen於 2015年11月18日星期三 UTC-8下午2時36分02秒寫道： > >> Besides, the path in log >> /home/cvs/OPS/nagios/libexec/check_mysql_multirow.pl >> <http://www.google.com/url?q=http%3A%2F%2Fcheck_mysql_multirow.pl&sa=D&sntz=1&usg=AFQjCNFa-8zf-m7y5RN4TmkMwBiTAOaacg> >> ,v >> is the path on CVS repository server filesystem. >> >> My local path is (on different host) >> /home/wchen/CVS_work/OPS/nagios/libexec/check_mysql_multirow.pl >> >> Christian Hammond於 2015年11月18日星期三 UTC-8下午2時26分22秒寫道： >>> >>> How are you generating the diffs? I noticed that as well, and I don't >>> believe it should be showing the ",v". >>> >>> Christian >>> >>> -- >>> Christian Hammond - [email protected] >>> Review Board - https://www.reviewboard.org >>> Beanbag, Inc. - https://www.beanbaginc.com >>> >>> On Wed, Nov 18, 2015 at 2:25 PM, Wesly Chen <[email protected]> wrote: >>> >>>> It seems to be related to CVS repository. In CVS repository, all the >>>> file will be trailed with ",v" and the content of CVS files are not similar >>>> with original one. >>>> However, "cvs diff" can parse and generate the diff file. So the review >>>> board Diff Viewer should check and parse the lexer based on the original >>>> filename instead of the one in CVS repository with ",v". >>>> >>>> Wesly Chen於 2015年11月17日星期二 UTC-8下午5時36分20秒寫道： >>>> >>>>> 2015-11-18 01:34:33,284 - DEBUG - None - wchen - >>>>> /reviews/r/114/diff/3/fragment/1876/ - Patching file >>>>> check_mysql_multirow.pl took 0.042870 seconds >>>>> 2015-11-18 01:34:33,409 - ERROR - - Failed to apply pygments: no >>>>> lexer for filename u'/home/cvs/OPS/nagios/libexec/ >>>>> check_mysql_multirow.pl,v' found >>>>> Traceback (most recent call last): >>>>> File >>>>> "/usr/lib/python2.6/site-packages/ReviewBoard-2.0.20-py2.6.egg/reviewboard/diffviewer/chunk_generator.py", >>>>> line 231, in _get_chunks_uncached >>>>> markup_a = self._apply_pygments(old or '', source_file) >>>>> File >>>>> "/usr/lib/python2.6/site-packages/ReviewBoard-2.0.20-py2.6.egg/reviewboard/diffviewer/chunk_generator.py", >>>>> line 654, in _apply_pygments >>>>> encoding='utf-8') >>>>> File >>>>> "/usr/lib/python2.6/site-packages/Pygments-2.0.2-py2.6.egg/pygments/lexers/__init__.py", >>>>> line 149, in get_lexer_for_filename >>>>> raise ClassNotFound('no lexer for filename %r found' % _fn) >>>>> ClassNotFound: no lexer for filename u'/home/cvs/OPS/nagios/libexec/ >>>>> check_mysql_multirow.pl,v' found >>>>> 2015-11-18 01:34:33,425 - DEBUG - None - wchen - >>>>> /reviews/r/114/diff/3/fragment/1876/ - Begin: Generating diff chunks for >>>>> self.filediff id 1876 (/home/cvs/OPS/nagios/libexec/ >>>>> check_mysql_multirow.pl,v) >>>>> 2015-11-18 01:34:33,432 - DEBUG - None - wchen - >>>>> /reviews/r/114/diff/3/fragment/1876/ - End: Generating diff chunks for >>>>> self.filediff id 1876 (/home/cvs/OPS/nagios/libexec/ >>>>> check_mysql_multirow.pl,v) >>>>> 2015-11-18 01:34:33,432 - DEBUG - None - wchen - >>>>> /reviews/r/114/diff/3/fragment/1876/ - Generating diff chunks for >>>>> self.filediff id 1876 (/home/cvs/OPS/nagios/libexec/ >>>>> check_mysql_multirow.pl,v) took 0.006171 seconds >>>>> >>>>> -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
