Closed
Bug 691950
Opened 13 years ago
Closed 12 years ago
Set up directories for W3C CSSWG test suite sync to/from mozilla-central
Categories
(Core :: Layout, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: fantasai.bugs, Assigned: fantasai.bugs)
References
()
Details
(Whiteboard: revised README file provided for review)
Attachments
(6 files, 1 obsolete file)
I wanted to set up a directory in mozilla-central where we can host tests that
are sync'ed with W3C, specifically for the CSSWG test suites, which are
dual-licensed under the W3C Document License and the BSD 3-clause.
The dual-license that applies to the W3C tests:
http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright
The license grant W3C requests of test contributors is attached.
What I'd like to do is to create
other-licenses/
|-w3c-css/
|-submitted/ # tests we are submitting to W3C, our copy is master
|-received/ # tests we are receiving from W3C, their copy is master
with a LICENSE file stating the 3-clause BSD under which these files are licensed and explaining licensing terms for contributions to these directories on mozilla-central.
This bug is to get legal sign-off on the plan.
Attachment #564732 -
Flags: review?
Attachment #564734 -
Flags: review?
Comment 4•13 years ago
|
||
Reading Rigo's doc, I think the right thing is for us to outbound-license as pure BSD, as he says that's OK with him.
This all seems fine to me, but we need an official legal sign-off.
Gerv
Assignee: gerv → nobody
Group: legal
Product: mozilla.org → Legal
QA Contact: licensing → handerson
Version: other → unspecified
Comment 5•13 years ago
|
||
Luis - Can you please take a look at this and let us know if you agree with Gerv or have other thoughts?
Assignee: nobody → villalu
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: under legal review by outside counsel
Comment 6•13 years ago
|
||
Since Luis hasn't responded I'm reassigning to Jishnu.
Assignee: villalu → jmenon
Whiteboard: under legal review by outside counsel → under legal review
Can I get an ETA on this, so I know whether to set up an interim solution that doesn't require legal signoff?
Comment 8•13 years ago
|
||
Hi - I agree with Gerv, the contribution license should be pure BSD and am fine with the inbound license. Could you rewrite the README file to reflect this and re-up for a quick look?
Assignee: jmenon → fantasai.bugs
Whiteboard: under legal review → fantasai to re-upload README file with corrections
Alright jmenon, let me know if anything needs further changing.
Attachment #564734 -
Attachment is obsolete: true
Attachment #564734 -
Flags: review?
Attachment #575093 -
Flags: review?
Comment 10•13 years ago
|
||
Jishnu - looks like this is back in your court.
Assignee: fantasai.bugs → jmenon
Whiteboard: fantasai to re-upload README file with corrections → revised README file provided for review
Assignee | ||
Comment 11•13 years ago
|
||
Can I get an ETA on this, so I know whether to set up an interim solution that doesn't require legal signoff? Thanks~
Assignee | ||
Comment 12•13 years ago
|
||
Ok, I'm going to assume that legal has no problem with this since they haven't responded. Shifting into Core for implementation.
Assignee: jmenon → nobody
Group: legal
Component: Licensing → Layout
Product: Legal → Core
QA Contact: handerson → layout
Comment 13•13 years ago
|
||
Comment on attachment 564732 [details]
LICENSE file
Going through unassigned reviews. dbaron, can you review or reassign as appropriate?
Attachment #564732 -
Flags: review? → review?(dbaron)
Updated•13 years ago
|
Attachment #575093 -
Flags: review? → review?(dbaron)
Attachment #564732 -
Flags: review?(dbaron)
Attachment #575093 -
Flags: review?(dbaron)
Assignee | ||
Comment 14•13 years ago
|
||
This patch creates the submitted and received directories in layout/reftests and adds README files to explain each.
Attachment #615542 -
Flags: review?(dbaron)
Summary: Set up W3C <-> mozilla-central sync for CSSWG test suites → Set up directories for W3C CSSWG test suite sync to/from mozilla-central
Comment 15•13 years ago
|
||
Comment on attachment 615542 [details] [diff] [review]
patch
3 thoughts so far:
* the LICENSE file might not be clear enough for the "submitted" directory, since for test submissions people need to agree to W3C's license too
* you need to add your new layout/reftests/w3c-css/submitted/reftest.list to layout/reftests/reftest.list with an include line.
* your commented out "import" lines should be "include" lines instead
Comment 16•13 years ago
|
||
Comment on attachment 615542 [details] [diff] [review]
patch
>+Do not touch anything here unless you are a layout/ or automation
>+tools owner or peer, or have permission from one and know what you
>+are doing.
I don't think there's any reason to mention ownership rules; you should just say that you shouldn't make any changes here that don't come from the master repository, because if you do they'll be overwritten by the next sync.
r=dbaron with that and the issues in the previous comment fixed
Attachment #615542 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 17•13 years ago
|
||
Patch with review comments addressed.
Keywords: checkin-needed
Comment 18•12 years ago
|
||
Comment 19•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 20•12 years ago
|
||
What are the next steps here?
Comment 21•12 years ago
|
||
I think the next steps here are to start syncing things occasionally and make the process more automated as time progresses. I did a first sync from submitted/ into the W3C repository yesterday; a first pass at syncing into received/ probably requires a script for just the first pass.
Comment 22•12 years ago
|
||
Comment 23•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b64118688939 (bustage fix for previous)
Comment 24•12 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•