NEW 124969
Add default port feature to Tools/Scripts/set-webkit-configuration 
https://bugs.webkit.org/show_bug.cgi?id=124969
Summary Add default port feature to Tools/Scripts/set-webkit-configuration
Éva Balázsfalvi
Reported 2013-11-28 06:28:02 PST
Add default platform feature to Tools/Scripts/set-webkit-configuration (--efl, --gtk, ... ) and related scripts to be able skip passing the platform option always to build-webkit, run-webkit-tests, run-perf-tests, ...
Attachments
Patch (4.39 KB, patch)
2013-11-28 06:33 PST, Éva Balázsfalvi 		no flags
DetailsFormatted DiffDiff
Patch (4.32 KB, patch)
2014-02-11 07:55 PST, Éva Balázsfalvi 		bfulgham: review-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Éva Balázsfalvi
Comment 1 2013-11-28 06:33:17 PST
Created attachment 217998 [details] Patch
Éva Balázsfalvi
Comment 2 2014-02-11 07:55:29 PST
Created attachment 223863 [details] Patch
Éva Balázsfalvi
Comment 3 2014-02-17 02:07:06 PST
Ping?
Zoltan Horvath
Comment 4 2014-02-27 15:21:52 PST
Comment on attachment 223863 [details] Patch This set-webkit-configuration file is pretty outdated. Does anyone use this anymore? Is it even possible to make a 32-bit builds? Anyways, I propose to rather remove this script...
Csaba Osztrogonác
Comment 5 2014-02-28 03:38:31 PST
(In reply to comment #4) > (From update of attachment 223863 [details]) > This set-webkit-configuration file is pretty outdated. > Does anyone use this anymore? Why do you think? It works fine to set debug/release/32-bit build as default. > Is it even possible to make a 32-bit builds? Yes, but this feture works only on Apple Mac. On 64 bit Linux isn't so trivial to do same thing. It is much more easier to build on a 32 bit machine or inside a 32 bit chroot. > Anyways, I propose to rather remove this script... I don't agree with you at all. It is a big help if you don't have to pass --efl/--gtk to all scripts when you always work on the same port. Would you like if --mac would be mandatory for you? :) I think no.
Brent Fulgham
Comment 6 2016-03-14 11:11:11 PDT
Comment on attachment 223863 [details] Patch This patch is fairly out of date. Can someone in the GTK/EFL ports rebaseline and complete the review process? I'm marking it r- since the patch is out of date, not because I disagree with the change.
Note You need to log in before you can comment on or make changes to this bug.