[Mono-bugs] [Bug 77303][Nor] New - mkbundle -z with --static breaks, missing space after -lz option

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by sebastian.nowozin at web.de.

http://bugzilla.ximian.com/show_bug.cgi?id=77303

--- shadow/77303	2006-01-19 06:30:25.000000000 -0500
+++ shadow/77303.tmp.29810	2006-01-19 06:30:25.000000000 -0500
@@ -0,0 +1,63 @@
+Bug#: 77303
+Product: Mono: Tools
+Version: 1.1
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Normal
+Component: tools
+AssignedTo: mono-bugs at ximian.com                            
+ReportedBy: sebastian.nowozin at web.de               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: mkbundle -z with --static breaks, missing space after -lz option
+
+Description of Problem:
+
+When using mkbundle with both the --static and the compress -z flag the
+internally generated command line is wrong and fails to succeed.
+
+Steps to reproduce the problem:
+1. mkbundle --static -z -o test.static --deps test.exe
+
+Actual Results:
+Note that statically linking the LGPL Mono runtime has more licensing
+restrictions than dynamically linking.
+See http://www.mono-project.com/Licensing for details on licensing.
+OS is: Linux
+Sources: 1 Auto-dependencies: True
+   embedding: /tmp/test.exe
+   compression ratio: 41.59%
+   embedding: /usr/lib/mono/1.0/mscorlib.dll
+   compression ratio: 35.22%
+Compiling:
+as -o /tmp/tmp6e777299.tmp.o temp.s 
+cc -o test.static -Wall `pkg-config --cflags mono` temp.c -lz`pkg-config
+--libs-only-L mono` -Wl,-Bstatic -lmono -Wl,-Bdynamic `pkg-config
+--libs-only-l mono | sed -e "s/\-lmono //"` /tmp/tmp6e777299.tmp.o
+/usr/bin/ld: cannot find -lz-L/usr/lib/pkgconfig/../../lib
+collect2: ld returned 1 exit status
+[Fail]
+
+
+Expected Results:
+
+Successfully producing a compressed test.static file.
+
+How often does this happen?
+
+Every time.
+
+
+Additional Information:
+
+Without -z it works.
+Notice: the -lz option _misses_ a space after it, producing a syntax error.
+The fix should be easy by appending a space after the -lz option.
+
+This is the mkbundle included with the standard Debian/sid ia32 package,
+version 1.1.13.1-1, mcs version 1.1.13.1.