Skip to content

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
@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.

@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.

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.

3 participants