[Mono-list] Incorrect path interpretation(gacutil)

[Rafael Teixeira]

Fixed on SVN: revision 71431.


On Sun, 2007-01-21 at 11:27 -0200, Rafael Teixeira wrote:
> PATCH in bugzilla
> 
> http://bugzilla.ximian.com/show_bug.cgi?id=80570
> 
> On Sun, 2007-01-21 at 11:16 -0200, Rafael Teixeira wrote:
> > You were right: the logic in gacutil driver, gets confused with *nix,
> > absolute paths.
> > 
> > The following patch makes it behave correctly for your case, I'll be
> > posting a bugzilla case and attaching the patch for review, but you can
> > patch locally your gacutil in the meantime.
> > 
> > :)
> > 
> > Patch
> > -------------
> > 
> > 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 at hotmail.com>
> > +
> > +       * driver.cs: accept absolute *nix paths for library names
> > without getting them confused with options
> > +
> >  2006-07-24  Alp Toker  <alp at atoker.com>
> >  
> >         * driver.cs: Minor typo fixes in comments and informational
> > output.
> > 
> > 
> > On Mon, 2007-01-15 at 07:26 -0800, monoport wrote:
> > > 
> > > 
> > > A Rafael D Teixeira wrote:
> > > > 
> > > > It is probably getting confused on parsing options. 
> > > > 
> > > > In your case it really gets confused because you have separated /i from
> > > > it's argument (the path to the library) with the unnecessary -f.  Try:
> > > > 
> > > > mono "/usr/lib/mono/lib/mono/1.0/gacutil.exe" -i
> > > > "/root/XXX/XXX/XXX/Assembly.dll"
> > > >  
> > > > It should work.
> > > > 
> > > > Try gacutil -? (or --help), to see the somewhat expanded list of options
> > > > Mono's implementation has, and also the unsupported ones.
> > > > 
> > > > :)
> > > > 
> > > > 
> > > 
> > > OK, I removed the "-f" option
> > > and again have:
> > > 
> > > gacutil -i /root/xxx/xxx/xxx/assembly.dll
> > > Option /root/xxx/xxx/xxx/assembly.dll takes 1 argument
> > > 
> > > Anyone have the mono installed on Linux, plz check.
> > > Is this working for you? I guess this is the bug.
> > > 
> > > ----
> > > monoport
> > 
> > _______________________________________________
> > Mono-list maillist  -  Mono-list at lists.ximian.com
> > http://lists.ximian.com/mailman/listinfo/mono-list
> > 
> > E-mail classificado pelo Identificador de Spam Inteligente Terra.
> > Para alterar a categoria classificada, visite
> > http://mail.terra.com.br/protected_email/imail/imail.cgi?+_u=rafael.teixeirabr&_l=1,1169385555.681673.29454.curepipe.hst.terra.com.br,5896,Des15,Des15
> > 
> > Esta mensagem foi verificada pelo E-mail Protegido Terra.
> > Scan engine: McAfee VirusScan / Atualizado em 19/01/2007 / Verso: 5.1.00/4943
> > Proteja o seu e-mail Terra: http://mail.terra.com.br/
> > 
> 
> _______________________________________________
> Mono-list maillist  -  Mono-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-list
> 
> E-mail classificado pelo Identificador de Spam Inteligente Terra.
> Para alterar a categoria classificada, visite
> http://mail.terra.com.br/protected_email/imail/imail.cgi?+_u=rafael.teixeirabr&_l=1,1169386218.347530.14763.alcuta.terra.com.br,6867,Des15,Des15
> 
> Esta mensagem foi verificada pelo E-mail Protegido Terra.
> Scan engine: McAfee VirusScan / Atualizado em 19/01/2007 / Verso: 5.1.00/4943
> Proteja o seu e-mail Terra: http://mail.terra.com.br/
>