Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

manos does not compile on Windows 7 #113

Open
MarkKharitonov opened this issue Jul 10, 2011 · 2 comments
Open

manos does not compile on Windows 7 #113

MarkKharitonov opened this issue Jul 10, 2011 · 2 comments

Comments

@MarkKharitonov
Copy link

"z:\dev\3rd_party\Manos\src\Manos\Manos.WithHints.proj" (default target) (1) -> (CoreCompile target) ->
  Manos.Http\HttpEntity.cs(521,5): error CS0165: Use of unassigned local variable 'completeWatcher' [z:\dev\3rd_party\Manos\src\Manos\Manos.WithHints.proj]
  Manos.Http\HttpTransaction.cs(176,5): error CS0165: Use of unassigned local variable 'handler' [z:\dev\3rd_party\Manos\src\Manos\Manos.WithHints.proj]
  Manos\AppHost.cs(208,6): error CS0165: Use of unassigned local variable 'timer' [z:\dev\3rd_party\Manos\src\Manos\Manos.WithHints.proj]

    1 Warning(s)
    3 Error(s)

Time Elapsed 00:00:01.00

z:\dev\3rd_party\Manos\src\Manos>

Where Manos.WithHints.proj is simply my wrapper around Manos.csproj to avoid modifying it:

<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build" ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <ItemGroup>
    <Reference Include="Mono.Posix" >
      <HintPath>..\..\Dependencies\Mono.Posix.dll</HintPath>
    </Reference>
    <Reference Include="Mono.CSharp" >
      <HintPath>..\..\Dependencies\Mono.CSharp.dll</HintPath>
    </Reference>
    <Reference Include="Mono.C5" >
      <HintPath>..\..\Dependencies\Mono.C5.dll</HintPath>
    </Reference>
  </ItemGroup>
  <Import Project="Manos.csproj" />
</Project>

I am using revision 33a14b0 pulled July 10.

@imphasing
Copy link
Contributor

I've submitted a pull request that fixes this issue. If you'd like to check out my patch, it's very simple. I just assign the closed-over variable the compiler is complaining about in the delegate to null first, so it's not unsure if it's ever going to be assigned.

@MarkKharitonov
Copy link
Author

I did fix it in my working copy, it is just that it was a bit unexpected.

2011/7/13 imphasing <
[email protected]>

I've submitted a pull request that fixes this issue. If you'd like to check
out my patch, it's very simple. I just assign the closed-over variable the
compiler is complaining about in the delegate to null first, so it's not
unsure if it's ever going to be assigned.

Reply to this email directly or view it on GitHub:
#113 (comment)

Be well and prosper.

"There are two kinds of people.Those whose guns are loaded and those who
dig."
("The good, the bad and the ugly")
So let us drink for our guns always be loaded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants