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

[ngspice] Initial port #12695

Merged
merged 3 commits into from
Aug 5, 2020
Merged

[ngspice] Initial port #12695

merged 3 commits into from
Aug 5, 2020

Conversation

marekr
Copy link
Contributor

@marekr marekr commented Aug 2, 2020

This adds ngspice, an actively maintained open source SPICE (circuit simulator) as a shared library build for Windows.

This is currently only compatible with x86 and x64 for Windows as those are the only upstream supported arches for Windows (for now)

@ghost
Copy link

ghost commented Aug 2, 2020

CLA assistant check
All CLA requirements met.

@NancyLi1013 NancyLi1013 added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Aug 3, 2020
ports/ngspice/CONTROL Outdated Show resolved Hide resolved
ports/ngspice/portfile.cmake Outdated Show resolved Hide resolved
ports/ngspice/portfile.cmake Outdated Show resolved Hide resolved
scripts/ci.baseline.txt Outdated Show resolved Hide resolved
ports/ngspice/portfile.cmake Outdated Show resolved Hide resolved
@marekr
Copy link
Contributor Author

marekr commented Aug 3, 2020

@NancyLi1013 I applied your suggest changes but it appears CI is ignoring the !static flag and building it and failing anyway?

Edit: Added !uwp as well, it seems to respect that flag but it's still ignoring !static.

ports/ngspice/CONTROL Outdated Show resolved Hide resolved
@marekr
Copy link
Contributor Author

marekr commented Aug 4, 2020

@NancyLi1013 Now it built windows static and OSX regardless of the supports flags :/

@NancyLi1013
Copy link
Contributor

@marekr
I'm so sorry. It seems that static doesn't work in Supports filed now.

So please remove it from supports list and we still need to add ngspice:x64-windows-static=fail in ci.baseline.txt.

ports/ngspice/portfile.cmake Outdated Show resolved Hide resolved
ports/ngspice/CONTROL Outdated Show resolved Hide resolved
Add the msvc specific include folder to be copied as well

Fix 32-bit builds

Use install_msbuild
Remove fftw

Fix 32-bit build again

Update ci baseline and check linkage for dll only

Use spaces consistently

More clean

Update ports/ngspice/CONTROL

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update scripts/ci.baseline.txt

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

remove uwp support

Update ports/ngspice/CONTROL

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
marekr and others added 2 commits August 4, 2020 07:14
Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
@marekr
Copy link
Contributor Author

marekr commented Aug 4, 2020

@NancyLi1013 No worries :D

Applied static to the baseline fail.

@NancyLi1013 NancyLi1013 added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Aug 5, 2020
@strega-nil strega-nil merged commit e303126 into microsoft:master Aug 5, 2020
hellozee pushed a commit to hellozee/vcpkg that referenced this pull request Sep 11, 2020
* Add ngspice port

Add the msvc specific include folder to be copied as well

Fix 32-bit builds

Use install_msbuild
Remove fftw

Fix 32-bit build again

Update ci baseline and check linkage for dll only

Use spaces consistently

More clean

Update ports/ngspice/CONTROL

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

Update scripts/ci.baseline.txt

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

remove uwp support

Update ports/ngspice/CONTROL

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

* Update ports/ngspice/portfile.cmake

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>

* Use the suggest supports

Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR! info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants