Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copyright headers in translator source code #80

Open
neildhickey opened this issue Oct 30, 2018 · 5 comments
Open

Copyright headers in translator source code #80

neildhickey opened this issue Oct 30, 2018 · 5 comments

Comments

@neildhickey
Copy link
Contributor

See #63

We have a confusing set of copyright headers in the translator. I think we should move to a general Khronos licence with perhaps an additional line for each company.

@neildhickey neildhickey added the agenda Tooling Call Agenda label Oct 30, 2018
@AnastasiaStulova
Copy link
Contributor

@outofcontrol do we have any Khronos guidelines for this?

@KhronosWebservices
Copy link

Moving to Apache 2.0 would be inline with Khronos' other repositories. @oddhack might be able to confirm if there are certain files which would not fall within Apache 2.0. This would also resolve any issues with applying a CLA to this repository.

@AnastasiaStulova
Copy link
Contributor

Do we have any copyright header template for the files or is it to be decided per repository?

@KhronosWebservices
Copy link

KhronosWebservices commented Mar 26, 2019

For Apache 2.0 we would use the following header:

;
; Copyright (c) YYYY The Khronos Group Inc.
;
; Licensed under the Apache License, Version 2.0 (the "License");
; you may not use this file except in compliance with the License.
; You may obtain a copy of the License at
;
; http://www.apache.org/licenses/LICENSE-2.0
;
; Unless required by applicable law or agreed to in writing, software
; distributed under the License is distributed on an "AS IS" BASIS,
; WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
; See the License for the specific language governing permissions and
; limitations under the License.
;

Additional trademarks and license would be included after the license.

edited: Fixed incorrect comment tag. oops.

@oddhack
Copy link

oddhack commented Mar 26, 2019

What we're encouraging all the WGs to do going forward is like Vulkan and OpenXR - Apache 2.0 on code-like things (code, scripts, Makefiles) and CC-BY 4.0 on documentation-like things (spec sources, document processing macros, etc.). Ratified Specifications produced by the WG in output formats (PDF, HTML) and published by us are normally placed under the proprietary Khronos specification license.

In specific cases where LGPL compatibility is required, we have added an "Exception Clause" to the Apache license. This happens very rarely; see xml/vk.xml in the Vulkan-Docs repository for an example.

Re #63. be aware that changing the copyright on something that has a copyright holder other than Khronos requires getting their affirmative consent to the change. What we've done for spec changes is ask members to assign copyright to Khronos under the current IP agreements; the only Vulkan documents that keep a vendor copyright are vendor extension specifications, and they still must be under the same Apache / CC-BY regimen as the Khronos-copyrighted files.

If you need more clarification on this, please copy me and Neil Trevett in email.

@AnastasiaStulova AnastasiaStulova removed the agenda Tooling Call Agenda label Sep 19, 2019
@svenvh svenvh mentioned this issue Apr 23, 2020
vmaksimo pushed a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this issue Sep 1, 2022
vmaksimo pushed a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this issue Sep 1, 2022
…pperBound (KhronosGroup#1350) '"master"' -> '"xmain-web"' (3 commits) (KhronosGroup#80)

* Resolve of merge (conflict) 24c1aca Support DISubrange DIExpression upperBound (KhronosGroup#1350) '"master"' -> '"xmain-web"' (3 commits)

  CONFLICT (content): Merge conflict in test/FortranArray.ll
  CONFLICT (content): Merge conflict in lib/SPIRV/SPIRVToLLVMDbgTran.cpp

  commit 24c1aca
  Author: KornevNikita <[email protected]>
  Date:   Thu Dec 23 16:30:06 2021 +0300

      Support DISubrange DIExpression upperBound (KhronosGroup#1350)

      Expand transDbgArrayType to preserve DISubrange lowerBound. And the case
      when upperBound is DIExpression.

* Conflict resolution

Co-authored-by: iclsrc <[email protected]>
Co-authored-by: Dmitry Sidorov <[email protected]>
vmaksimo pushed a commit to vmaksimo/SPIRV-LLVM-Translator that referenced this issue Sep 1, 2022
…ression upperBound (KhronosGroup#1350) '"master"' -> '"xmain-web"' (3 commits) (KhronosGroup#80)

* Resolve of merge (conflict) 24c1aca Support DISubrange DIExpression upperBound (KhronosGroup#1350) '"master"' -> '"xmain-web"' (3 commits)

  CONFLICT (content): Merge conflict in test/FortranArray.ll
  CONFLICT (content): Merge conflict in lib/SPIRV/SPIRVToLLVMDbgTran.cpp

  commit 24c1aca
  Author: KornevNikita <[email protected]>
  Date:   Thu Dec 23 16:30:06 2021 +0300

      Support DISubrange DIExpression upperBound (KhronosGroup#1350)

      Expand transDbgArrayType to preserve DISubrange lowerBound. And the case
      when upperBound is DIExpression.

* Conflict resolution

Co-authored-by: iclsrc <[email protected]>
Co-authored-by: Dmitry Sidorov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants