Summary: | [GTK][WPE] EWS should not wipe the JHBuild in the unapply patch step | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> | ||||||||
Component: | Tools / Tests | Assignee: | Carlos Alberto Lopez Perez <clopez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aakash_jain, ap, bugs-noreply, dbates, dpino, ews-watchlist, glenn, jbedard, psaavedra, slewis, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=202361 | ||||||||||
Attachments: |
|
Description
Carlos Alberto Lopez Perez
2020-01-10 04:53:01 PST
Created attachment 387336 [details]
Patch
Created attachment 387338 [details]
Patch
typo fix
Comment on attachment 387338 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=387338&action=review > Tools/BuildSlaveSupport/ews-build/steps.py:174 > + self.setCommand(self.command + ['--keep-jhbuild-directory']) Please add following line at the end of this start method: return shell.ShellCommand.start(self) This will also fix the unit-tests. > Tools/Scripts/clean-webkit:48 > + if fs.isdir("WebKitBuild"): Would this path work if the script is invoked from another directory? (In reply to Aakash Jain from comment #3) > Comment on attachment 387338 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=387338&action=review > > > Tools/BuildSlaveSupport/ews-build/steps.py:174 > > + self.setCommand(self.command + ['--keep-jhbuild-directory']) > > Please add following line at the end of this start method: > return shell.ShellCommand.start(self) > > This will also fix the unit-tests. > ups, sure. > > Tools/Scripts/clean-webkit:48 > > + if fs.isdir("WebKitBuild"): > > Would this path work if the script is invoked from another directory? Yes and no... The current clean-webkit script its not prepared to be ran outside of the main directory. For example, if you run: cd Tools/Scripts && ./clean-webkit Then the script only cleans the files inside the dir Tools/Scripts/ And if you run from another directory (for example from /tmp) it fails completely (python backtrace) So the clean-webkit script doesn't seem at all prepared to be ran from other directory than the main one, and this patch doesn't change that. Created attachment 387355 [details]
Patch
Add missing return shell.ShellCommand.start(self)
Comment on attachment 387355 [details] Patch Clearing flags on attachment: 387355 Committed r254359: <https://trac.webkit.org/changeset/254359> All reviewed patches have been landed. Closing bug. |