[Webkit-unassigned] [Bug 51509] [Qt] [Symbian] Fix build-webkit script for Symbian
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Dec 23 08:28:50 PST 2010
https://bugs.webkit.org/show_bug.cgi?id=51509
--- Comment #3 from Laszlo Gombos <laszlo.1.gombos at nokia.com> 2010-12-23 08:28:50 PST ---
Thanks for the review Eric!
.
(In reply to comment #2)
> (From update of attachment 77281 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=77281&action=review
>
> Review tool can't seem to apply the patch, so I doubt this will land w/o an update.
EWS seems to be green.
>
> > Tools/Scripts/webkitdirs.pm:303
> > + # Symbian builds do not have Release/Debug configurations either.
> > + if (($ENV{"WEBKITOUTPUTDIR"} && (isGtk() || isEfl())) || isSymbian()) {
>
> We need to make this into a function. usesPerConfigurationBuildDirectory() or similar.
Will do.
>
> > Tools/Scripts/webkitdirs.pm:1526
> > - my $dir = File::Spec->canonpath(baseProductDir());
> > - $dir = File::Spec->catfile($dir, $config) unless isSymbian();
> > + my $dir = File::Spec->canonpath(productDir());
>
> Is this a symbian specific code path? Otherwise why is this OK for other ports?
This is a Qt specific code path (Symbian being one of the QtWebKit ports). For QtWebKit productDir() returns $baseProductDir/$configuration for all Qt ports except for Symbian (see earlier part of the patch).
>
> > Tools/Scripts/webkitdirs.pm:1532
> > - push @buildArgs, "OUTPUT_DIR=" . baseProductDir() . "/$config";
> > + push @buildArgs, "OUTPUT_DIR=" . $dir;
>
> Why is this OK for other ports? Or is this just reverting previously added symbian-specific code?
Same as above.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list