From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45226) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1db8vl-0003r9-Q1 for guix-patches@gnu.org; Fri, 28 Jul 2017 13:20:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1db8vi-0004D4-Fp for guix-patches@gnu.org; Fri, 28 Jul 2017 13:20:05 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:57591) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1db8vi-0004Co-CI for guix-patches@gnu.org; Fri, 28 Jul 2017 13:20:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1db8vi-0002NZ-4a for guix-patches@gnu.org; Fri, 28 Jul 2017 13:20:02 -0400 Subject: [bug#27344] [PATCH v2 08/12] gnu: Add ngspice. Resent-Message-ID: References: <20170622191005.25422-1-theodoros.for@openmailbox.org> <20170622191005.25422-8-theodoros.for@openmailbox.org> <20170723133712.2239b7d9@scratchpost.org> <20170723151646.013b6408@scratchpost.org> From: Theodoros Foradis Message-ID: <87r2x08orz.fsf@openmailbox.org> In-reply-to: <20170723151646.013b6408@scratchpost.org> Date: Fri, 28 Jul 2017 20:17:40 +0300 MIME-Version: 1.0 Content-Type: text/plain List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: 27344@debbugs.gnu.org Hello Danny, > apparently you are trying to provide the shared object in package "libngspice" and the executables in package "ngspice". Is that so? Correct. Ngspice, does not build both the executable and shared library in the same run. The configure script must anyway be run twice, and this is why I split the package, to begin with. > I've checked the resulting output directories for ngspice and libngspice and there's some duplication going on. > > For example, both packages provide cmpp, the man pages for ALL the executables and the ngspice include files, and shared libraries as lib/ngspice/*.cm. > > Was that intended? That was not intended. In the first version of the patch set, I had the "ngspice" package copy the shared library from "libngspice" to its output. So, there would only be one public package. After splitting it up, I missed on fixing the duplicates. > I've also made the build reproducible by substituting "`date`" in the file "configure". Thanks for fixing that. > My current version of your patch is below. > > After it, the following duplicates remain: > * ./bin/cmpp > * ./share/ngspice/dlmain.c Should be in libngspice. > * ./lib/ngspice/analog.cm > * ./lib/ngspice/digital.cm > * ./lib/ngspice/spice2poly.cm > * ./lib/ngspice/xtradev.cm > * ./lib/ngspice/xtraevt.cm Should be in both packages. > * ./share/ngspice/scripts/setplot > * ./share/ngspice/scripts/spectrum > * ./share/ngspice/scripts/spinit Should be in ngspice. > Also, how can other programs find libngspice? There's no pkg-config file and also no replacement (a la gtk-config) that I can see. > > cmpp seems to be a preprocessor for xspice extensions - should it be part of ngspice or libngspice ? Is libngspice useful without cmpp being there ? > The program I have tested libngspice with, is Kicad (after I get this patchset over with, I will submit a patch to update Kicad, and build with ngspice support). I guess it finds libngspice through LIBRARY_PATH or something? Also, it builds without cmpp being in libngspice (so libngspice is useful without it), but cmpp should be there nonetheless. I have updated your patch, to delete the appropriate files from ngspice and libngspice respectively. I will send it with the updated patch series. Regards, -- Theodoros Foradis