[136949] trunk/dports/devel/dylibbundler

[ryandesign at macports.org]

Revision: 136949
          https://trac.macports.org/changeset/136949
Author:   ryandesign at macports.org
Date:     2015-05-30 21:17:42 -0700 (Sat, 30 May 2015)
Log Message:
-----------
dylibbundler: fix off-by-one error in computing the filename from the "otool" output which resulted in an extra space at the end of every filename; also use rfind instead of find to properly handle files whose names contain the search string " ("

Modified Paths:
--------------
    trunk/dports/devel/dylibbundler/Portfile

Added Paths:
-----------
    trunk/dports/devel/dylibbundler/files/patch-src-DylibBundler.cpp.diff

Modified: trunk/dports/devel/dylibbundler/Portfile
===================================================================
--- trunk/dports/devel/dylibbundler/Portfile	2015-05-31 03:22:29 UTC (rev 136948)
+++ trunk/dports/devel/dylibbundler/Portfile	2015-05-31 04:17:42 UTC (rev 136949)
@@ -5,7 +5,7 @@
 
 name                dylibbundler
 version             0.4.4
-revision            1
+revision            2
 categories          devel
 platforms           darwin
 maintainers         strasweb.fr:rudloff openmaintainer
@@ -38,6 +38,7 @@
 }
 
 patchfiles          patch-makefile.diff \
+                    patch-src-DylibBundler.cpp.diff \
                     patch-src-main.cpp.diff
 
 post-patch {

Added: trunk/dports/devel/dylibbundler/files/patch-src-DylibBundler.cpp.diff
===================================================================
--- trunk/dports/devel/dylibbundler/files/patch-src-DylibBundler.cpp.diff	                        (rev 0)
+++ trunk/dports/devel/dylibbundler/files/patch-src-DylibBundler.cpp.diff	2015-05-31 04:17:42 UTC (rev 136949)
@@ -0,0 +1,20 @@
+--- src/DylibBundler.cpp.orig	2013-09-08 04:44:52.000000000 -0500
++++ src/DylibBundler.cpp	2015-05-30 23:09:27.000000000 -0500
+@@ -86,7 +86,7 @@
+         if( lines[n].find(".framework") != std::string::npos ) continue; //Ignore frameworks, we can not handle them
+         
+         addDependency( // trim useless info, keep only library name
+-                       lines[n].substr(1, lines[n].find(" (") )
++                       lines[n].substr(1, lines[n].rfind(" (") - 1)
+                        );
+     }
+ }
+@@ -112,7 +112,7 @@
+                 if( lines[n].find(".framework") != std::string::npos ) continue; //Ignore frameworks, we can not handle them
+                 
+                 addDependency( // trim useless info, keep only library name
+-                               lines[n].substr(1, lines[n].find(" (") )
++                               lines[n].substr(1, lines[n].rfind(" (") - 1)
+                                );
+             }//next
+         }//next
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20150530/00ac4c34/attachment.html>