Bug 109303 - Add gyp option to switch ENABLE(SQL_DATABASE)
Summary: Add gyp option to switch ENABLE(SQL_DATABASE)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://code.google.com/p/chromium/iss...
Keywords:
Depends on: 110356
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-08 07:47 PST by Paweł Hajdan, Jr.
Modified: 2013-02-20 14:58 PST (History)
2 users (show)

See Also:


Attachments
patch (3.33 KB, patch)
2013-02-08 07:50 PST, Paweł Hajdan, Jr.
tony: review+
tony: commit-queue-
Details | Formatted Diff | Diff
patch (3.47 KB, patch)
2013-02-14 04:49 PST, Paweł Hajdan, Jr.
no flags Details | Formatted Diff | Diff
patch (3.31 KB, patch)
2013-02-20 10:09 PST, Tony Chang
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Paweł Hajdan, Jr. 2013-02-08 07:47:40 PST
Add gyp option to switch ENABLE(SQL_DATABASE)

This will be useful to implement build with system sqlite, http://code.google.com/p/chromium/issues/detail?id=22208
Comment 1 Paweł Hajdan, Jr. 2013-02-08 07:50:57 PST
Created attachment 187319 [details]
patch
Comment 2 Tony Chang 2013-02-13 10:26:29 PST
Comment on attachment 187319 [details]
patch

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

> Source/WebKit/chromium/ChangeLog:10
> +        Add gyp option to switch ENABLE(SQL_DATABASE)
> +        
> +        This will be useful to implement build with system sqlite,
> +        http://code.google.com/p/chromium/issues/detail?id=22208
> +        
> +        https://bugs.webkit.org/show_bug.cgi?id=109303
> +
> +        Reviewed by NOBODY (OOPS!).

This is a different order than other ChangeLog entries.  The bug link should go directly under the first line and the Reviewed by should go under that.
Comment 3 Paweł Hajdan, Jr. 2013-02-14 04:49:55 PST
Created attachment 188323 [details]
patch
Comment 4 Tony Chang 2013-02-20 10:09:55 PST
Created attachment 189334 [details]
patch

Fixed ChangeLog
Comment 5 WebKit Review Bot 2013-02-20 10:13:16 PST
Comment on attachment 189334 [details]
patch

Rejecting attachment 189334 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=gce-cq-03', 'validate-changelog', '--non-interactive', 189334, '--port=chromium-xvfb']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Last 500 characters of output:
parse_author_text(self._author_text)
  File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 164, in _parse_author_text
    return [cls._parse_author_name_and_email(author) for author in authors]
  File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 156, in _parse_author_name_and_email
    return {'name': match.group("name"), 'email': match.group("email")}
AttributeError: 'NoneType' object has no attribute 'group'

Full output: http://queues.webkit.org/results/16647418
Comment 6 Tony Chang 2013-02-20 10:18:03 PST
(In reply to comment #5)
> (From update of attachment 189334 [details])
> Rejecting attachment 189334 [details] from commit-queue.
> 
> Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=gce-cq-03', 'validate-changelog', '--non-interactive', 189334, '--port=chromium-xvfb']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue
> 
> Last 500 characters of output:
> parse_author_text(self._author_text)
>   File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 164, in _parse_author_text
>     return [cls._parse_author_name_and_email(author) for author in authors]
>   File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 156, in _parse_author_name_and_email
>     return {'name': match.group("name"), 'email': match.group("email")}
> AttributeError: 'NoneType' object has no attribute 'group'
> 
> Full output: http://queues.webkit.org/results/16647418

Hah, I was hitting this bug locally.  I'm working on a fix.  The ChangeLog parser is confused by the comma in Pawel's name.
Comment 7 WebKit Review Bot 2013-02-20 14:58:37 PST
Comment on attachment 189334 [details]
patch

Clearing flags on attachment: 189334

Committed r143511: <http://trac.webkit.org/changeset/143511>
Comment 8 WebKit Review Bot 2013-02-20 14:58:40 PST
All reviewed patches have been landed.  Closing bug.