-
Notifications
You must be signed in to change notification settings - Fork 91
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
Add missing license identifier #73
Conversation
This fixes hyperledger#72 and adds missing license headers. It also adds Fabric's license check to the make checks target. Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Given that most of these files we all have already or will modify i would have also suggested to add also the Intel copyright, but then after rebuke you got on a similar request i guess we will have to do it on our own ... :-)
|
||
license: | ||
@echo "License: Running licence checks.." | ||
@sh ${GOPATH}/src/github.com/hyperledger/fabric/scripts/check_license.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Calling it via ${TOP}/../../hyperledger/fabric/scripts/check_license.sh seem a bit more robust (i.e., it will also work when you have multiple GOPATH components)? But i have no strong opinion on this ...
Yes, I guess there are also some missing Intel copyrights already; can you do a pass and add a commit here or reuse mine and also sign-off?! Let's not over complicate it. @bvavala Can you do this? |
Nice check. |
Signed-off-by: Bruno Vavala <bruno.vavala@intel.com>
* check ercc container and force remove * force remove in ecc_enclave * ecc check container and image for stop/delete * tlcc_enclave force rm files * typo in tlcc mrenclave generate script * fix sh complaint * check container in ecc make docker * fixes PR #73 * remove makefile variable; better error message Signed-off-by: Bruno Vavala <bruno.vavala@intel.com>
This fixes #72 and adds missing license headers.
It also adds Fabric's license check to the
make checks target.
Signed-off-by: Marcus Brandenburger bur@zurich.ibm.com