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.
