WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
134096
[EFL] Remove the WebKit1 EFL code
https://bugs.webkit.org/show_bug.cgi?id=134096
Summary
[EFL] Remove the WebKit1 EFL code
Ryuan Choi
Reported
2014-06-20 00:14:14 PDT
rm -rf Source/WebKit/efl
Attachments
Patch
(1.44 MB, patch)
2014-06-20 01:34 PDT
,
Ryuan Choi
gyuyoung.kim
: review+
Details
Formatted Diff
Diff
Patch
(1.45 MB, patch)
2014-06-20 02:02 PDT
,
Ryuan Choi
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Ryuan Choi
Comment 1
2014-06-20 01:34:01 PDT
Created
attachment 233412
[details]
Patch
Gyuyoung Kim
Comment 2
2014-06-20 01:37:07 PDT
Comment on
attachment 233412
[details]
Patch rs=me
Ryuan Choi
Comment 3
2014-06-20 02:02:55 PDT
Created
attachment 233416
[details]
Patch Added one missing file
WebKit Commit Bot
Comment 4
2014-06-20 14:24:11 PDT
Comment on
attachment 233416
[details]
Patch Rejecting
attachment 233416
[details]
from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'validate-changelog', '--check-oops', '--non-interactive', 233416, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: = self._filesystem.open_text_file_for_reading(self.path) File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py", line 204, in open_text_file_for_reading return codecs.open(path, 'r', 'utf8', errors) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/codecs.py", line 881, in open file = __builtin__.open(filename, mode, buffering) IOError: [Errno 2] No such file or directory: u'/Volumes/Data/EWS/WebKit/Source/WebKit/efl/ChangeLog' Full output:
http://webkit-queues.appspot.com/results/6170918799802368
Ryuan Choi
Comment 5
2014-06-20 15:27:26 PDT
(In reply to
comment #4
)
> (From update of
attachment 233416
[details]
) > Rejecting
attachment 233416
[details]
from commit-queue. > > Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'validate-changelog', '--check-oops', '--non-interactive', 233416, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit > > Last 500 characters of output: > = self._filesystem.open_text_file_for_reading(self.path) > File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py", line 204, in open_text_file_for_reading > return codecs.open(path, 'r', 'utf8', errors) > File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/codecs.py", line 881, in open > file = __builtin__.open(filename, mode, buffering) > IOError: [Errno 2] No such file or directory: u'/Volumes/Data/EWS/WebKit/Source/WebKit/efl/ChangeLog' > > Full output:
http://webkit-queues.appspot.com/results/6170918799802368
Hmm, it looks that modified_changelogs() in checkout.py does not consider whether changelog is deleted or not. I am not sure how I can land this before fixing Tools. Below is the log when I used direct-land. ( ./Tools/Scripts/webkit-patch land-
attachment 233416
[details]
) Can't open /home/ryuan/workspace/project/webkit/efl-webkit/Source/WebKit/efl/ChangeLog at /home/ryuan/workspace/project/webkit/efl-webkit/Tools/Scripts/commit-log-editor line 221. Failed to run "[u'/home/ryuan/workspace/project/webkit/efl-webkit/Tools/Scripts/commit-log-editor', '--print-log', u'/home/ryuan/workspace/project/webkit/efl-webkit/Source/WebKit/ChangeLog', u'/home/ryuan/workspace/project/webkit/efl-webkit/Source/WebKit/efl/ChangeLog']" exit_code: 2 Failed to run "[u'/home/ryuan/workspace/project/webkit/efl-webkit/Tools/Scripts/commit-log-editor', '--print-log', u'/home/ryuan/workspace/project/webkit/efl-webkit/Source/WebKit/ChangeLog', u'/home/ryuan/workspace/project/webkit/efl-webkit/Source/WebKit/efl/ChangeLog']" exit_code: 2
Csaba Osztrogonác
Comment 6
2014-06-23 06:35:02 PDT
Landed in
https://trac.webkit.org/changeset/170289
manually, git svn dcommit is our friends when webkit-patch land can't handle something. ;)
Ryuan Choi
Comment 7
2014-06-23 17:11:44 PDT
(In reply to
comment #6
)
> Landed in
https://trac.webkit.org/changeset/170289
manually, git svn dcommit > is our friends when webkit-patch land can't handle something. ;)
Thanks. :)
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug