<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On May 1, 2013, at 18:43 , Anton Yartsev <<a href="mailto:anton.yartsev@gmail.com">anton.yartsev@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
<meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
<div text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 02.05.2013 5:27, Jordan Rose wrote:<br>
</div>
<blockquote cite="mid:50B7D77C-1FB2-4AC8-BD28-99184EBB18B0@apple.com" type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<br>
<div>
<div>On May 1, 2013, at 18:24 , Anton Yartsev <<a moz-do-not-send="true" href="mailto:anton.yartsev@gmail.com">anton.yartsev@gmail.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
<div text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 02.05.2013 5:10, Jordan Rose
wrote:<br>
</div>
<blockquote cite="mid:0E6FC7B3-0987-47CF-861C-DD70843DB7C3@apple.com" type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<br>
<div>
<div>On May 1, 2013, at 18:08 , Anton Yartsev <<a moz-do-not-send="true" href="mailto:anton.yartsev@gmail.com">anton.yartsev@gmail.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
<div text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 01.05.2013 21:10,
Jordan Rose wrote:<br>
</div>
<blockquote cite="mid:5F254812-A40E-4198-90F5-0E29F8B95977@apple.com" type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
<br>
<div>
<div>On May 1, 2013, at 6:38 , Anton Yartsev
<<a moz-do-not-send="true" href="mailto:anton.yartsev@gmail.com">anton.yartsev@gmail.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite">
<meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
<div text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 01.05.2013
6:29, Jordan Rose wrote:<br>
</div>
<blockquote cite="mid:8369578E-5301-43AE-A5D0-26B0C9D0E3A0@apple.com" type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
<div>Comments:</div>
<div><br>
</div>
<div>
<div>
<blockquote type="cite">+use English;</blockquote>
<br>
</div>
</div>
<div>According to <a moz-do-not-send="true" href="http://perldoc.perl.org/English.html">http://perldoc.perl.org/English.html</a>,
it's probably better to use </div>
<div><br>
</div>
<div>use English qw( -no_match_vars ) ;</div>
<div><br>
</div>
<div>...not that scan-build's startup time
matters <i>that</i> much.</div>
</blockquote>
Currently this is used only for $OSNAME
while short names are used for all other
special perl variables.<br>
Considering that using English affects
performance, what about rolling back to $^O
and adding explanatory comment instead?<br>
</div>
</blockquote>
</div>
<br>
<div>Ah, right, fair enough.</div>
<div><br>
</div>
<div>This looks good. Please go ahead and commit,
but make sure that c++-analyzer is marked as
executable for Unix-y systems.</div>
</blockquote>
Committed at r180900. Attributes of c++-analyzer
are: -rwxr-xr-x<br>
</div>
</blockquote>
</div>
<br>
<div>Um, doesn't seem to be working for me. You may have
to manually set <a moz-do-not-send="true" class="moz-txt-link-freetext" href="svn:executable">svn:executable</a>.</div>
</blockquote>
Updated at r180901. Did this helped?<br>
</div>
</blockquote>
</div>
<br>
<div>Hm, no, it did not. For some reason it still thinks it's a
symlink on my system:</div>
<div><br>
</div>
<div>
<div>lrwxr-xr-x 1 jrose staff 203B May 1 18:25
c++-analyzer@ -> #!/usr/bin/env perl??use Cwd qw/ abs_path
/;?use File::Basename qw/ dirname /;?# Add scan-build dir to
the list of places where perl looks for modules.?use lib
dirname(abs_path($0));??do 'ccc-analyzer';?</div>
</div>
<div><br>
</div>
<div>Admittedly this is passing through git-svn, but maybe it's
best to just remove the file entirely and then add it back?</div>
<div><br>
</div>
<div>Jordan</div>
</blockquote>
Created a new plain file, copied the c++analyzer contents, removed
c++analyzer from repository and added the new one.<br></div></blockquote></div><br><div>Missing the svn:executable. ;-) But it is showing up as a plain file now for me, so that should do it.</div><div><br></div><div>Jordan</div></body></html>