<div class="gmail_quote">On Tue Sep 02 2014 at 2:38:59 PM Steffen Prohaska <<a href="mailto:prohaska@zib.de">prohaska@zib.de</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Sep 2, 2014, at 1:41 AM, Justin Bogner <<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>> wrote:<br>
<br>
> Justin Bogner <<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>> writes:<br>
>> Steffen Prohaska <<a href="mailto:prohaska@zib.de" target="_blank">prohaska@zib.de</a>> writes:<br>
>>> An std::error_code of 0 indicates success.  The condition must be reversed.<br>
>>><br>
>>> See diff below for <a href="http://llvm.org/svn/llvm-project/cfe/trunk" target="_blank">http://llvm.org/svn/llvm-<u></u>project/cfe/trunk</a><br>
>>> ---<br>
>>><br>
>>> Hello,<br>
>>> I'm unsure whether this is the right way to submit a patch.  Please let me<br>
>>> know if I should prepare it in a different way.<br>
>><br>
>> Generally, we prefer patches to be sent to cfe-commits@, rather than<br>
>> cfe-dev@, for future reference.<br>
>><br>
>> Additionally, we usually like to check in a test with bug fixes. This is<br>
>> pretty obvious by inspection, but if you have a case where it fails and<br>
>> can easily be reduced, submitting that as well would be appreciated.<br>
><br>
> On further inspection, fixing this opens a whole can of worms. It looks<br>
> like we've been relying on it being broken for quite a while. I've sent<br>
> an alternative patch to cfe-commits for review:<br>
><br>
>    <a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20140901/113949.html" target="_blank">http://lists.cs.uiuc.edu/<u></u>pipermail/cfe-commits/Week-of-<u></u>Mon-20140901/113949.html</a><br>
<br>
Hmm...  Your patch looks sane, but it doesn't solve my original problem.<br>
<br>
Maybe the following is relevant.  I'm unsure, because I'm rather<br>
unfamiliar with the clang codebase.<br>
<br>
I've been working on a custom refactoring tool.  I tried to save the<br>
replacements and then apply them with clang-apply-replacements.  I<br>
wanted to collect replacements for several packages across a large<br>
project.  The problem can be illustrated with clang-modernize.<br>
<br>
I created a compile_commands.json for a custom build system that uses<br>
recursive make. Commands are executed in the subdirectory of the source<br>
file:<br>
<br>
    [<br>
        {<br>
            "directory": "/rootpath/src/package",<br>
            "command": "clang++ ... file.cpp",<br>
            "file": "file.cpp"<br>
        },<br>
        ...<br>
    ]<br>
<br>
I'd like to create serialized replacements similar to:<br>
<br>
    clang-modernize --add-override --serialize-replacements \<br>
        --serialize-dir=$(pwd)/<u></u>replacements src/package/file.cpp<br>
<br>
and then apply them with:<br>
<br>
    clang-apply-replacements -format ./replacements<br>
<br>
The setup seemed natural, and I expected it to work this way.  But it<br>
fails, because the serialized replacements contain paths that are<br>
relative to the directories in compile_commands.json (like<br>
"FilePath: file.cpp").<br>
<br>
With my fix that teaches Replacement to store absolute paths, it works<br>
as expected, because the Replacements and therefore the YAML files then<br>
contain absolute paths.<br></blockquote><div><br></div><div>The problem is that this then leaves you with replacements that are not relocatable; for example, if you run the refactoring tool distributed over multiple machines, where each machine may have a random path component (machine dependent), you'll end up with replacements you cannot apply at all.</div>
<div><br></div><div>Which format are you outputting replacements in? Can you just call make_absolute on the replacement's path when you output them?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Best,<br>
  Steffen<br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-dev</a><br>
</blockquote></div>