Summary: | ng-spice-rework misses important configuration flags | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | George Reitsma <g.p.reitsma> |
Component: | New packages | Assignee: | The Soldering-Iron Brotherhood <sci-electronics> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | florent.d-halluin |
Priority: | Normal | ||
Version: | 2006.0 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | sci-electronics/ng-spice-rework-17-r2.ebuild |
Description
George Reitsma
2006-07-14 12:51:38 UTC
Has nothing in common w/ portage tools, please read the product descriptions when filing bugs. @sci - Please, make sure that the herd in metadata.xml is an existing bugzilla alias; sci-electronics is not one of them. ;) Sorry for entering this in the wrong category, it's the first time for me. Thanks, George Created attachment 93396 [details]
sci-electronics/ng-spice-rework-17-r2.ebuild
New ebuild with xspice use flag, --enable-intnoise and also added an X use flag, and the dependencies on modular X. It also seems to be linking with ncurses without being in the dependencies... Please check if the dependencies are correct.
(In reply to comment #0) > --enable-xspice > > This flag is required to enable the use of compact models from semiconductor > suppliers (spice2 format). So very important for most ng-spice users. The most > elegant solution would be to make "xspice" a USE flag. We decided earlier that this was still rather experimental (bug #109851 comment #2). Denis, do you think this is worth adding now? > --enable-intnoise > > Noise analysis, which is important for most analog circuit designers. I think > it is save to enable this always for everyone. This one should be less intrusive I'd imagine. Modular X dependencies look good but the indentation looks a bit fugly -- also if we're going to push the xspice USE flag into the tree we'll need to add the flag to profiles/use.local.desc :) Hello, Yes xspice is a bit experimental, from ./configure --help: --enable-xspice Enable XSpice enhancements, (experimental) And that's why I made it a use flag and not always enabled. Yes, I don't like the indentation either, I tried to copy the one in the migration guide... Lucas (In reply to comment #6) > Yes xspice is a bit experimental, from ./configure --help: > --enable-xspice Enable XSpice enhancements, (experimental) > And that's why I made it a use flag and not always enabled. Lucas, the problem is simpler than that. If it's broken like I think it is, and unless we can fix it by a patch, it can't even be a USE flag. Or we'll get lots of bugs due to it, even from people who don't use it. And we really don't need them. If it isn't broken, I don't see why it would need to be optional. I remember that I tried xspice and that my conclusion was that it was broken. I can't remember how it broke, but I seem to remember it did. Same with numparams. It breaks ngspice if you compile it in, even if you don't use it. And that's bad, so I had to write a patch for it. Maybe we should use the slow season to thouroughly test it (xspice in ngspice, I mean). Lucas, why don't you research a few good test cases for it, and check that it does work on a few real world examples ? If you do so, please check that it doesn't break ngspice when you compile xspice in it and don't use it. I want to be sure the thing works before we include it in. Same with intnoise. Oh and BTW, not having intnoise doesn't mean you can't do noise simulations... ;o) Finally, you don't need the USE flag to enable things. You can do so now, with the ebuild that's currently in portage : # EXTRA_ECONF="--enable-xspice --enable-intnoise" emerge ng-spice-rework Denis. Great to see you are looking into it! Currently I do use ngspice-rework-17, but I compiled it by hand with the following configure options: ./configure --enable-intnoise --enable-xspice --with-readline=yes --prefix=/usr For me it compiled without any problem, and it works fine. I've simulated several analog circuits with it, and haven't seen anything strange. With xspice you can in principle also do mixed signal simulation, but that's a part I haven't tried. The reason you practically cannot do without xspice is that ngspice is a "spice3" simulator. However, all compact models that are available from mayor semiconductor companies are "spice2" compact models, and luckily xpice can read them. It's true that you can calculate the noise power density spectrum, without int noise. But for any analog circuit, you need to know the integrated noise, so that is a very handy option. I'll try the new ebuild. The attached ebuild works nice! I've copied it into my portage tree, added "xspice" to the use flags and this ebuild to the package.keywords and builded it. It all worked! I've tried a few simulations with it, and all works great. I hope this will become a part of the new tree. Thanks! George miss --enable-xspice for example Sorry this is coming a bit late, but it needed real testing before going into the tree. There were also a few things to fix and change, see below my commit message. *ng-spice-rework-17-r2 (26 May 2007) 26 May 2007; Denis Dupeyron <calchan@gentoo.org> +ng-spice-rework-17-r2.ebuild: Enabled xspice and intnoise after testing them, which closes bug #140402. Added optional ngspice and xspice documentation. Fixed X dependencies and made them optional. Fixed collisions with external xgraph. Fixed debug messing with compilation optimizations. Made debug even more chatty. Please test this as much as you can and report back (in private is OK). I'd like to stabilize this sooner than later, so I'd rather be sure it works. All I know is that it does work with my rather heavy nutmeg scripts for automated analog design, and that's something. Denis. Hi Denis, I've compiled ngspice with the above ebuild, using the xpsice use flag, and recompiled it a while ago after switching to gcc 4.1.1. (from gcc 3.x.x). I've used it for simulating several analog circuits and it works fine for me! I haven't got any digital circuits myself, so I cannot test that part, but the analog transient and ac looks good to me. I think this tool together with "pcb" are the two only really stable tools from the "GEDA" suit. Thanks, George |