search

mumble-voip / mumble-releng

Mumble Release Engineering
Other
17 stars 21 forks source link

Include Cygwin as required package in README #71

Closed sregister closed 6 years ago

sregister commented 6 years ago

Cygwin with the curl and unzip packages are required for building dependencies.

sregister commented 6 years ago

I'm closing this PR for now--I think I missed a few packages (common lisp). I'll make a complete list and submit a new PR at another date.

mkrautz commented 6 years ago

You shouldn’t need Cygwin anymore. The setup bootstraps it using cygwin-bootstrap.

Did that not work for you?

sregister commented 6 years ago

Doesn't look like it. I double clicked setup.cmd to make the build directory but the cmd prompt closed almost immediately so I just assumed there were no errors. I waited a few seconds and thought everything was fine since there was a new directory structure in C:/MumbleBuild. First time I went to attempt to build the dependencies I got:

Failed to find a usable Cygwin root. Please make sure you have a Cygwin installed. which happens on if not defined MUMBLE_CYGWIN_ROOT.

If there was an error during setup.cmd (and cygwin-bootstrap) then I guess that could explain why building deps was so painful. I'll try to reproduce the error but this time invoke setup.cmd from a interactive cmd.exe so hopefully I can catch any interesting output.

mkrautz commented 6 years ago

It should be doing it: https://github.com/mumble-voip/mumble-releng/blob/master/buildenv/1.3.x/win32-static/setup.cmd#L114-L127

Are you maybe using an old clone of mumble-releng?

mkrautz commented 6 years ago

If it closes that fast, you're probably getting an error, actually. For some reason, we don't pause for errors when run interactively.

The best way to proceed is to try and run setup.cmd from a command prompt. That should show you the error.