Bug 141151 - imported/w3c/web-platform-tests should contain top level files of the W3C web-platform-tests repository
Summary: imported/w3c/web-platform-tests should contain top level files of the W3C web...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-02 03:31 PST by youenn fablet
Modified: 2015-02-03 00:45 PST (History)
4 users (show)

See Also:


Attachments
Patch (21.28 KB, patch)
2015-02-02 03:39 PST, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2015-02-02 03:31:06 PST
Top level files include help information which may be useful, as well as the W3C license, which may be needed (?).
The missing files are currently CONTRIBUTING.md, LICENSE and README.md
Comment 1 youenn fablet 2015-02-02 03:39:13 PST
Created attachment 245865 [details]
Patch
Comment 2 youenn fablet 2015-02-02 03:42:50 PST
(In reply to comment #1)
> Created attachment 245865 [details]
> Patch

Patch adds README.md and server-side.md which are useful information.
It also adds LICENSE which may be needed.
It also adds CONTRIBUTING.md which may not be relevant.
But it is worth deciding what we should add/not add.
Comment 3 Bem Jones-Bey 2015-02-02 10:01:46 PST
Other than the LICENSE file, I'm not convinced there is utility in having these files in the webkit repo. They all talk about things you need to do to add new WPT tests, but since everyone should be doing that upstream and not in WebKit, they'd have to grab the WPT repo and read the docs there anyways.
Comment 4 youenn fablet 2015-02-02 11:05:37 PST
(In reply to comment #3)
> Other than the LICENSE file, I'm not convinced there is utility in having
> these files in the webkit repo. They all talk about things you need to do to
> add new WPT tests, but since everyone should be doing that upstream and not
> in WebKit, they'd have to grab the WPT repo and read the docs there anyways.

When debuggging tests, I found useful some information inside server-side.md.
The README.md contains the list of domains that should be defined. That is not yet supported by webkit infrastructure and may explain why some bugs fail.
The CONTRIBUTING.md contains information somehow more precise than what we put in the w3c-import.log files.
Not to say that these files are crucial though, except maybe for LICENSE.

Checking blink and mozilla, they both include LICENSE, README.md and CONTRIBUTING.md.
Comment 5 Ryosuke Niwa 2015-02-02 22:45:19 PST
Comment on attachment 245865 [details]
Patch

rs=me.

It seems fine to have these files.
Comment 6 WebKit Commit Bot 2015-02-03 00:45:19 PST
Comment on attachment 245865 [details]
Patch

Clearing flags on attachment: 245865

Committed r179542: <http://trac.webkit.org/changeset/179542>
Comment 7 WebKit Commit Bot 2015-02-03 00:45:22 PST
All reviewed patches have been landed.  Closing bug.