Track LICENSE and NOTICE issues for incubator-openwhisk-wskdeploy

[mrutkows]

Please use this issue to track any issues surrounding either the LICENSE.txt or NOTICE.txt file for the associated repository listed in the title.

Open Issues/PRs:

• Issue #175 - Remove wskdeploy specification PDFs (archives) and diagram from source release

Closed Issues/Merged PRs:

• PR #709 - Add Apache License
  • Issue #716 - Some source files are missing Apache license headers
• PR #842 - Update NOTICE file
  • Issue #86 - Review NOTICE file for incubator-openwhisk-wskdeploy
• PR #844 - Add the LICENSE and NOTICE into the binary packages
• PR #882 - Add ASF license headers to API Gateway integration test's javascript source files #882
• PR #883 - Add license header to 3 files #883
• PR #888 - Change the LICENSE file into the format defined in release #888
• PR #896 - Remove Apache long header if short header exists
  • Issue #133 - Why do wskdeploy "example' .js files have both FULL and MINI ASF headers? - Daisy volunteered to look at (issue in release repo.)
• PR #897 - Fix all files with Hash license header to fix typo
• PR #902 - Restore the LICENSE and NOTICE to fit the source code release
• PR #907 - Fix makefile ASF header & use ICF as example of curl usage.
  • Note: Revert failed on PR #911, fixed in PRs #914 and #915
• PR #912 - Fix travis yaml file asf header, change to mini
• PR #914 - Fix mini license header
  • Issue #889 - Change all manifest.yaml and deployment.yaml files to use Mini ASF license header
  • Issue #909 - Fix LICENSE headers in all YAML/.js under "tests" folder
• PR #915 - Fix last few files missing correct ASF header and EOF errors
• PR #921 - Add ASF license header to .gradle files.
• PR#952 - Add the DISCLAIMER file for Apache incubator project

PRs Closed/Reverted/Revisit:

• PR #886 - adding license files
• PR #887 - an attempt to cleanup list of deps
  • Closed - Priti determined the dependency cross-ref. was too tangled to unknot quickly. Should this be revisited in future (i.e., open a future issue?)?

[mrutkows]

@daisy-ycguo @houshengbo Addressing the discussion in PR apache/openwhisk-wskdeploy#844 (review), we have need to document that we have decided not to include the generated "i18n_resources.go" file and are unable to add a header comment to its source input file (i.e., en_US.all.json or other similar language/country coded JSON files).

IMO, we should include this decision/logic in this repo's LICENSE file.

[daisy-ycguo]

Issue was created to track the header issue: apache/openwhisk-wskdeploy#716, and also a PR was merged apache/openwhisk-wskdeploy#709.

Yet there is still a TODO: change the test files to short license header.

[daisy-ycguo]

I submitted a PR to add headers to the left 3 files apache/openwhisk-wskdeploy#883. This should be the last PR, I think. After it gets merged, there should be no files missing header in wskdeploy repo.

[houshengbo]

We can close this issue, since all the PRs have been merged.
The issue #175 is being fixed by another PR.