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 - chri...@beanbaginc.com
>> Review Board - https://www.reviewboard.org
>> Beanbag, Inc. - https://www.beanbaginc.com
>>
>> On Wed, Nov 18, 2015 at 2:25 PM, Wesly Chen <wesly...@gmail.com> 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 reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to