Bug 126773 - make make_names.pl default to default system compiler
Summary: make make_names.pl default to default system compiler
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Other
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 128598
  Show dependency treegraph
 
Reported: 2014-01-10 12:32 PST by Koop Mast
Modified: 2014-02-11 06:20 PST (History)
4 users (show)

See Also:


Attachments
Switch to system default compiler (1.56 KB, patch)
2014-01-10 12:34 PST, Koop Mast
no flags Details | Formatted Diff | Diff
Drop root changelog entry, set review flag (1.03 KB, patch)
2014-01-15 10:11 PST, Koop Mast
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Koop Mast 2014-01-10 12:32:06 PST
As I understand it cc and c++ are the default C and C++ system compilers. So as the final fall though the script should use them instead of hoping gcc might be installed.

This was found because on FreeBSD 10 we don't have gcc anymore in base on the x86* archs.
Comment 1 Koop Mast 2014-01-10 12:34:18 PST
Created attachment 220877 [details]
Switch to system default compiler
Comment 2 Csaba Osztrogonác 2014-01-13 05:26:05 PST
Please set r? flag if you would like to ask for review.
Comment 3 Alexey Proskuryakov 2014-01-13 10:01:50 PST
Comment on attachment 220877 [details]
Switch to system default compiler

View in context: https://bugs.webkit.org/attachment.cgi?id=220877&action=review

> ChangeLog:4
> +        Use system default compiler instead of gcc, as final fall through.
> +        https://bugs.webkit.org/show_bug.cgi?id=126773

Please revert this change to root ChangeLog, updating Source/WebCore/ChangeLog is sufficient.
Comment 4 Koop Mast 2014-01-15 10:11:41 PST
Created attachment 221281 [details]
Drop root changelog entry, set review flag

Is there a general rule for when to update the root Changelog or not?
Comment 5 Csaba Osztrogonác 2014-01-15 10:14:43 PST
> Is there a general rule for when to update the root Changelog or not?
yes, when Tools/Scripts/prepare-ChangeLog automatically does it ;)
Comment 6 Alexey Proskuryakov 2014-01-15 14:58:40 PST
Comment on attachment 221281 [details]
Drop root changelog entry, set review flag

View in context: https://bugs.webkit.org/attachment.cgi?id=221281&action=review

> Source/WebCore/dom/make_names.pl:73
> +    $gccLocation = "/usr/bin/cc";

Should rename the variable one day.
Comment 7 WebKit Commit Bot 2014-02-06 08:40:15 PST
Comment on attachment 221281 [details]
Drop root changelog entry, set review flag

Clearing flags on attachment: 221281

Committed r163534: <http://trac.webkit.org/changeset/163534>
Comment 8 WebKit Commit Bot 2014-02-06 08:40:18 PST
All reviewed patches have been landed.  Closing bug.