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

[Enhancement] KCLVM windows artifact optimization. #379

Closed
5 tasks done
Peefy opened this issue Jan 11, 2023 · 0 comments · Fixed by #385
Closed
5 tasks done

[Enhancement] KCLVM windows artifact optimization. #379

Peefy opened this issue Jan 11, 2023 · 0 comments · Fixed by #385
Assignees
Labels
release windows Only for windows.

Comments

@Peefy
Copy link
Contributor

Peefy commented Jan 11, 2023

Enhancement

We need to further optimize KCLVM windows artifacts as follows:

@Peefy Peefy added release windows Only for windows. labels Jan 11, 2023
@Peefy Peefy added this to the v0.4.5 Release milestone Jan 11, 2023
amyXia1994 pushed a commit that referenced this issue Jan 16, 2023
rm py39-libs under scripts/build-windows/py-39-libs.
rm useless code in scripts/build-windows/build.bat.
issue #379.
zong-zhe added a commit that referenced this issue Jan 29, 2023
* Refactor(kclvm-win): rm python39 for kclvm-windows.
issue #379.

* add KCLVM_CLI_BINPATH to PATH

* rm useless files

* redirect cmd.output to stderr.

* println -> Fprintln
@Peefy Peefy reopened this Jan 29, 2023
Peefy pushed a commit that referenced this issue Jan 31, 2023
…#392)

* Feat(kclvm-win): remove the "kclvm.requirements.done.txt" on windows.
issue #379.

* fix typo

* fix comments
Peefy pushed a commit that referenced this issue Feb 9, 2023
* Fix(kclvm-windows): fix linker and test cases on windows.
issue #379.

* add runtime test

* rm doc ignore

* fix typo

* fix fmt

* rm useless import

* fix fmt

* fix ci

* fix test case

* set path in CI

* fix ci

* remove useless code
Peefy pushed a commit that referenced this issue Feb 10, 2023
* add PATH

* fix typo

* Feat(kclvm-windows): fix windows ci.
issue #379.
Peefy pushed a commit that referenced this issue Feb 13, 2023
* Fix(kclvm-windows): rm '\\?\' in windows path.
issue #379.

* add test case

* fix test case

* fix test case
Peefy pushed a commit that referenced this issue Feb 14, 2023
* Fix(kclvm-windows): rm all the '\\?\' path prefix in windows.
issue #379.

* add doc test
Peefy pushed a commit that referenced this issue Feb 15, 2023
* Fix(kclvm-windows): rm all the '\\?\' path prefix in windows.
issue #379.

* Feat(kclvm-windows): add konfig test for kclvm-windows.
issue #379.

* add windows ci

* fix typo
Peefy pushed a commit that referenced this issue Feb 16, 2023
Feat(kclvm-test): add konfig test for kclvm on unix. issue #379.
@Peefy Peefy closed this as completed Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release windows Only for windows.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants