[Mono-list] System.Reflection and MethodImplOptions

[Paolo Molaro]

On 01/07/02 Matt Kimball wrote:
> except some of the stuff in System.Reflection didn't build cleanly because some
> of the source files pull in both System.Runtime.CompilerServices and
> System.Runtime.InteropServices, and then expect to use constants from
> MethodImplOptions from CompilerServices without fully qualifying the where they
> are coming from.  
> 
> So, I was getting build errors when building corlib.dll.

The patch is wrong: MethodImplOptions is defined only in
CompilerServices, not in InteropServices, so there is no need to
qualify it. Maybe you have a stale MethodImplOptions.cs in the tree
(I checked the snapshots and it's not there, I removed it from cvs a few
weeks ago). So, your fix would be to delete
System.Runtime.InteropServices/MethodImplOptions.cs.

The real fix is: change the build system to not use *.cs, but list the
individual files that must be build; it's a bit more work when you add a
file, but it gets rid of some potential build problems like this.

lupus

-- 
-----------------------------------------------------------------
lupus@debian.org                                     debian/rules
lupus@ximian.com                             Monkeys do it better