Bug 323258 (MONO80570) - [PATCH] gacutil thinks absolute *nix paths for libraries are options
Summary: [PATCH] gacutil thinks absolute *nix paths for libraries are options
Status: RESOLVED FIXED
Alias: MONO80570
Product: Mono: Tools
Classification: Mono
Component: tools (show other bugs)
Version: 1.0
Hardware: Other Other
: P3 - Medium : Enhancement
Target Milestone: ---
Assignee: Mono Bugs
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-21 16:57 UTC by Rafael Teixeira
Modified: 2007-09-15 21:24 UTC (History)
0 users

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wiest 2007-09-15 20:23:42 UTC


---- Reported by rafaelteixeirabr@hotmail.com 2007-01-21 09:57:22 MST ----

>>gacutil /i /xxx/xxx/xxx/assembly.dll
Option /xxx/xxx/assembly.dll takes 1 argument

Patch on mcs/tools/gacutil:

Index: driver.cs
===================================================================
--- driver.cs   (revision 71418)
+++ driver.cs   (working copy)
@@ -574,7 +574,7 @@
 
                private static bool IsSwitch (string arg)
                {
-                       return (arg [0] == '-' || arg [0] == '/');
+                       return (arg [0] == '-' || (arg [0] == '/' &&
!arg.EndsWith(".dll") && arg.IndexOf('/', 1) < 0 ) );
                }
 
                private static Command GetCommand (string arg)
Index: ChangeLog
===================================================================
--- ChangeLog   (revision 71418)
+++ ChangeLog   (working copy)
@@ -1,3 +1,7 @@
+2007-01-21  Rafael Teixeira <rafaelteixeirabr@hotmail.com>
+
+       * driver.cs: accept absolute *nix paths for library names without
getting them confused with options
+
 2006-07-24  Alp Toker  <alp@atoker.com>
 
        * driver.cs: Minor typo fixes in comments and informational output.



---- Additional Comments From miguel@ximian.com 2007-01-21 11:44:03 MST ----

Please commit this fix.



---- Additional Comments From rafaelteixeirabr@hotmail.com 2007-01-21 17:01:59 MST ----

Committed revision 71431.



Unknown operating system unknown. Setting to default OS "Other".