<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Unified builds broke ycm autocompletion using compilation databases"
   href="https://bugs.webkit.org/show_bug.cgi?id=182689#c22">Comment # 22</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Unified builds broke ycm autocompletion using compilation databases"
   href="https://bugs.webkit.org/show_bug.cgi?id=182689">bug 182689</a>
              from <span class="vcard"><a class="email" href="mailto:yusukesuzuki@slowstart.org" title="Yusuke Suzuki <yusukesuzuki@slowstart.org>"> <span class="fn">Yusuke Suzuki</span></a>
</span></b>
        <pre>(In reply to Carlos Eduardo Ramalho from <a href="show_bug.cgi?id=182689#c21">comment #21</a>)
<span class="quote">> (In reply to Carlos Eduardo Ramalho from <a href="show_bug.cgi?id=182689#c20">comment #20</a>)
> > (In reply to Yusuke Suzuki from <a href="show_bug.cgi?id=182689#c19">comment #19</a>)
> > > Interesting! Now I think the good way to fix this issue is adding an option
> > > disabling unified builds (which makes compile_commands.json expected one).
> > 
> > Hmm indeed! This might really be the way to go. Also, to get the
> > compile_commands.json file one does not need to compile the project, but
> > only "generate" or "configure" it with CMake e.g. with 

> Correction: actually we still need to build the project in WebKit because we
> need to resolve the ForwardingHeaders.</span >

Right.

<span class="quote">> 
> For developing, disabling unified sources might be OK because usually what
> will matter is sequential builds times and it should be improved by
> disabling unified sources actually.</span >

No. It significantly worse the build time. I think unified builds should be enabled by default.
And we should have the option disabling it.
I uploaded the patch for that option.
<a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Add ENABLE_UNIFIED_BUILDS option to cmake ports"
   href="show_bug.cgi?id=193045">https://bugs.webkit.org/show_bug.cgi?id=193045</a>

<span class="quote">> 
> (On a side note, sequential builds will be faster even with unified sources
> after <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Use directory local sequential numbers for Unified Sources filenames instead of global sequential numbers for CMake"
   href="show_bug.cgi?id=192391">bug 192391</a> lands.)

> I was able to disable unified sources and should upload a patch soon with
> the option.</span ></pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>