Skip to content

Include upstream copyright notice #419

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

Merged
merged 1 commit into from
Jun 3, 2016
Merged

Conversation

gagern
Copy link
Contributor

@gagern gagern commented Jun 2, 2016

The the libxml2 vendor subdirectory of libxmljs does not contain the upstream copyright notice even though the MIT license wording states that

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software

as the sole condition of use. We should make sure to include them in the source repository, and in the sources as provided by the npm registry.

@gagern gagern mentioned this pull request Jun 2, 2016
@defunctzombie defunctzombie merged commit d461c8e into libxmljs:master Jun 3, 2016
murrayju added a commit to murrayju/libxmljs-mt that referenced this pull request Jul 18, 2017
* origin/libxmljs-master: (26 commits)
  v0.18.7
  Add node 7 & 8 builds (libxmljs#475)
  v0.18.6
  Fix macOS build
  Fix env
  Fix macOS builds
  Use before_deploy
  v0.18.5
  Precompiled linux/macOS binaries
  Remove -Werror, avoid libxml shared lib
  v0.18.4
  v0.18.3
  Bump NAN to v2.5.x
  v0.18.2
  New build archs, deploy as tar.gz, always use MSVS 2015
  v0.18.1
  Update binary package name
  Better memory usage check
  Windows builds for Node 6 & 7
  Include upstream copyright notice for libxml2 (libxmljs#419)
  ...
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

Successfully merging this pull request may close these issues.

2 participants