Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

[CI/Build] include NOTICE in package dist-info #271

Merged
merged 1 commit into from
May 30, 2024

Conversation

derekk-nm
Copy link

includes the NOTICE file alongside the license files in the nm-vllm*.dist-info directory

@derekk-nm
Copy link
Author

I've installed the 3.11 package from this build and confirmed that the NOTICE file is properly copied:

(nmv040n) root@qa-test-dev-7746bbf7db-j9hlb:~/nmv040_nightly# ls ~/pyvenv/nmv040n/lib/python3.11/site-packages/nm_vllm_nightly-0.4.0.20240530.dist-info/
INSTALLER       LICENSE.fastertransformer  LICENSE.squeezellm   NOTICE     direct_url.json
LICENSE         LICENSE.gptq               LICENSE.tensorrtllm  RECORD     top_level.txt
LICENSE.apache  LICENSE.marlin             LICENSE.vllm         REQUESTED
LICENSE.awq     LICENSE.punica             METADATA             WHEEL
(nmv040n) root@qa-test-dev-7746bbf7db-j9hlb:~/nmv040_nightly# head ~/pyvenv/nmv040n/lib/python3.11/site-packages/nm_vllm_nightly-0.4.0.20240530.dist-info/NOTICE 
Neural Magic vLLM
Copyright 2024 - present / Neuralmagic, Inc. All Rights Reserved.

This product includes software developed at Neuralmagic, Inc. (https://www.neuralmagic.com).

Source code in the nm-vllm GitHub repository is variously licensed under the Apache License
Version 2.0, an Apache-compatible license or the Neural Magic Community License. Please see
https://github.com/neuralmagic/nm-vllm/blob/main/LICENSE for more information.

* For a copy of the Apache License Version 2.0, please see [Apache License, Version 2.0](http://www.apache.org/licenses/LICENSE-2.0).
(nmv040n) root@qa-test-dev-7746bbf7db-j9hlb:~/nmv040_nightly# 

@derekk-nm derekk-nm marked this pull request as ready for review May 30, 2024 15:16
@robertgshaw2-neuralmagic robertgshaw2-neuralmagic merged commit 4fabcfc into main May 30, 2024
12 checks passed
@robertgshaw2-neuralmagic robertgshaw2-neuralmagic deleted the include_NOTICE branch May 30, 2024 16:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants