Opened 21 years ago

Closed 20 years ago

Last modified 9 years ago

#1800 closed enhancement (remind)

swig: patch for swig runtime support

Reported by: lars@… Owned by: kvv@…
Priority: Normal Milestone:
Component: ports Version: 1.0
Keywords: Cc:
Port: swig

Description (last modified by ryandesign (Ryan Carsten Schmidt))

SWIG has a runtime. The runtime is necessary for bindings that span multiple compilation units (.i files), but share definitions. For simpler projects, it is optional.

The current Portfile does not support the swig runtime. Again the subversion bindings need the swig runtime.

Attached are two patches. Patch 1 builds and installs the runtime by default, Patch 2 creates a variant.

I prefer Patch 1 and believe installing the SWIG runtime will do more good than harm.

Patch 1:

Index: Portfile
===================================================================
RCS file: /Volumes/src/cvs/od/proj/darwinports/dports/devel/swig/Portfile,v
retrieving revision 1.5
diff -u -r1.5 Portfile
--- Portfile    5 Apr 2004 02:44:18 -0000       1.5
+++ Portfile    29 Apr 2004 18:37:15 -0000
@@ -14,4 +14,5 @@
 checksums      md5 88ff5b99be3eafebb8a6e20f4f986c04
 worksrcdir     SWIG-1.3.21
 patchfiles     patch-Makefile.in
-build.target   swig
+build.target   swig runtime
+destroot.target install install-runtime

Patch 2 with runtime variant:

Index: Portfile
===================================================================
RCS file: /Volumes/src/cvs/od/proj/darwinports/dports/devel/swig/Portfile,v
retrieving revision 1.5
diff -u -r1.5 Portfile
--- Portfile    5 Apr 2004 02:44:18 -0000       1.5
+++ Portfile    29 Apr 2004 19:16:19 -0000
@@ -15,3 +15,8 @@
 worksrcdir     SWIG-1.3.21
 patchfiles     patch-Makefile.in
 build.target   swig
+
+variant runtime {
+    build.target-append runtime
+       destroot.target-append install-runtime
+}

Change History (5)

comment:1 Changed 20 years ago by blb@…

Owner: changed from darwinports-bugs@… to kevin@…

Assign to port maintainer

comment:2 Changed 20 years ago by bob@…

Here is a revised Patch 1 for SWIG 1.2.24. Note that the patch is no longer necessary and should be removed if this is applied.

Index: Portfile
===================================================================
RCS file: /Volumes/src/cvs/od/proj/darwinports/dports/devel/swig/Portfile,v
retrieving revision 1.5
diff -u -r1.5 Portfile
--- Portfile    5 Apr 2004 02:44:18 -0000       1.5
+++ Portfile    11 Mar 2005 20:34:21 -0000
@@ -2,7 +2,7 @@
 
 PortSystem 1.0
 name           swig
-version                1.3.21
+version                1.3.24
 categories     devel
 maintainers    kevin@opendarwin.org
 description    Tool to wrap C/C++ libraries in high-level languages.
@@ -11,7 +11,7 @@
 homepage       http://www.swig.org/
 platforms      darwin
 master_sites   sourceforge
-checksums      md5 88ff5b99be3eafebb8a6e20f4f986c04
-worksrcdir     SWIG-1.3.21
-patchfiles     patch-Makefile.in
-build.target   swig
+checksums      md5 c5fc655dbbb6fe0cfab2211747dadbe0
+worksrcdir     SWIG-1.3.24
+build.target   swig runtime
+destroot.target        install install-runtime
Last edited 9 years ago by ryandesign (Ryan Carsten Schmidt) (previous) (diff)

comment:3 Changed 20 years ago by kevin@…

Status: newassigned

The "runtime" and "install-runtime" targets don't appear in swig-1.3.24.

comment:4 Changed 20 years ago by kevin@…

Resolution: remind
Status: assignedclosed

Upgraded to 1.3.24. I don't know if this includes runtime support by default or not. Let me know if it doesn't, or feel free to close the bug if it does.

/cvs/od/proj/darwinports/dports/devel/swig/Portfile,v <-- Portfile new revision: 1.6; previous revision: 1.5

comment:5 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)
Port: swig added
Note: See TracTickets for help on using tickets.