<div dir="ltr">On Fri, Oct 11, 2013 at 9:19 AM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I totally misread the test lines, you're right, this is something else.<br>
<br>
I'm still keeping an eye on this thread, though, because I think there<br>
are synergies between the current FixedCompilationDatabase, what you<br>
want to do and a compilation database built from a build-system<br>
invocation, and that the current FixedCompilationDatabase could<br>
probably be made to support all three (not that I understand why<br>
multiprocessing requires a separate compilation db, anxious to hear<br>
your response to Manuel's question.)<br></blockquote><div><br></div><div>I actually think those are different issues. Your issue is to drop-in run a tool from a build system - by now I think you have convinced me enough that this is a real need, that I'd vote for accepting patches to that end.</div>
<div>Here, it looks to me like the calling process has full control over how it calls the subprocess, thus we have much more flexibility in what we require for command line arguments...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Cheers,<br>
- Kim<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Oct 11, 2013 at 3:56 PM, Edwin Vane <<a href="mailto:revane@gmail.com">revane@gmail.com</a>> wrote:<br>
> I'm not sure if this is a step toward tools acting as CXX replacements but I<br>
> renamed the class to something a little more describe of what it does as<br>
> opposed to what it's used for.<br>
><br>
><br>
> On Fri, Oct 11, 2013 at 3:13 AM, Kim Gräsman <<a href="mailto:kim.grasman@gmail.com">kim.grasman@gmail.com</a>> wrote:<br>
>><br>
>> Hi Edwin,<br>
>><br>
>> On Fri, Oct 11, 2013 at 12:34 AM, Edwin Vane <<a href="mailto:edwin.vane@intel.com">edwin.vane@intel.com</a>> wrote:<br>
>> ><br>
>> >   The problem is the FixedCompilationDatabase mangles the arguments you<br>
>> > pass. FixedCompilationDatabase as a result has some preconditions: the name<br>
>> > of the compiler should not be present nor should be the filename to operate<br>
>> > on. Check CompilationDatabase.h for a better description. As stated in the<br>
>> > patch summary we need a simpler compilation database that simply accepts a<br>
>> > full list of args and then provides those same args back upon request.<br>
>><br>
>> This sounds nice, but the name feels a little funny... Isn't it just a<br>
>> LiteralCompilationDatabase? The subprocess forwarding is just one use<br>
>> case, unless I'm missing something. I'm still interested in allowing<br>
>> tools to act as CXX replacements in build systems, for example, and it<br>
>> feels like this is as step on the way there?<br>
>><br>
>> - K<br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
><br>
><br>
> --<br>
> Edwin V<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>