search

Pop2121 / quake2-gwt-port

Automatically exported from code.google.com/p/quake2-gwt-port
GNU General Public License v2.0
0 stars 0 forks source link

Patches to fix building on Windows #34

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
When trying to run the ant script on Windows I got some error, which should be 
fixed by the attached patch.

When running the game in chrome there are still issues (some missing textures, 
all models missing). Not sure if this is Windows related as I could not test on 
Linux, yet.

The patch includes the following:

jake2/qcommon/FS.java: "CreatePath"-method did not work on windows: fixed

jake2/tools/Installer.java: Compatibility.impl was not initialized: fixed

jake2/tools/Unpak.java: "createPath"-method now uses fixed method from FS (see 
above)

jake2/tools/WAVConverter: stdout and stderr form lame and oggenc were not read: 
added simple runnable reading both in seperate threads (this is just a quick 
solution. there are most likely better ways to do this)

Original issue reported on code.google.com by DerK...@gmail.com on 24 Oct 2010 at 12:12

Attachments:

GoogleCodeExporter commented 8 years ago 
the missing models do not happen in firefox 4, therefore I thinks this is not 
windows related. I'll try building this in a Linux VM for comparison soon.

Original comment by DerK...@gmail.com on 24 Oct 2010 at 12:44

GoogleCodeExporter commented 8 years ago 
Just made a quick test on a Linux VM and for me the patch to 
jake2/tools/Installer is required to build successfully there, too.

Original comment by DerK...@gmail.com on 24 Oct 2010 at 8:15

GoogleCodeExporter commented 8 years ago 
the missing textures and models are an issue of the dev version of chrome. 
works fine in the stable version. so the patch seems to make it work completely 
on windows.

Original comment by DerK...@gmail.com on 24 Oct 2010 at 8:35

GoogleCodeExporter commented 8 years ago 
I just wanted to say thanks for that patch. I tested it on the Windows install 
and it works great (better than the little hacks I tried to put together).
 I can't test on a Linux install, but I can verify that the Windows is flawless now.

Original comment by Xath...@gmail.com on 25 Oct 2010 at 9:17

GoogleCodeExporter commented 8 years ago 
Anyone using this patch, please leave a comment here and vote for this issue.

Original comment by DerK...@gmail.com on 31 Oct 2010 at 1:55

GoogleCodeExporter commented 8 years ago 
This fixed an issue I had when trying to install:

"[java] Exception in thread "main" java.lang.NullPointerException
     [java]     at jake2.qcommon.Compatibility.newString(Compatibility.java:51)"

Thanks!

p.s. I'm not running Windows - I'm on Mac OS 10.6.

Original comment by peter.os...@gmail.com on 31 Oct 2010 at 9:24

GoogleCodeExporter commented 8 years ago 
Sorry I should have said above, there was one problem with the patch for me - 
the last file to be patched failed:

"Skipping patch.
5 out of 5 hunks ignored -- saving rejects to file 
src/jake2/tools/WAVConverter.java.rej"

However, everything seems to be running fine (except for one error message I 
get when I get killed: "Result of expression 'nc' [undefined] is not an object" 
- I presume I should add this as a separate issue?)

Cheers,

Original comment by peter.os...@gmail.com on 31 Oct 2010 at 9:32

GoogleCodeExporter commented 8 years ago 
The patch fixed the newString NullPointerException that I was having (issue 35) 
on Ubuntu maverick. Thanks!

Original comment by delan.az...@gmail.com on 1 Nov 2010 at 8:28

GoogleCodeExporter commented 8 years ago 
I want to test this patch but I don't know how to apply it, how do I go about 
doing that?

Original comment by matthew...@gmail.com on 13 Dec 2010 at 9:18

GoogleCodeExporter commented 8 years ago 
Many IDEs, respository management software, and other programs have an "Apply 
Patch" function. Barring that, you can do what I did: open up the .diff file 
and apply the changes manually.

Original comment by Xath...@gmail.com on 15 Dec 2010 at 4:06

GoogleCodeExporter commented 8 years ago 
Patch works for me as well, after hand-applying it.

Original comment by james121...@gmail.com on 5 Feb 2011 at 3:49

GoogleCodeExporter commented 8 years ago 
[deleted comment]
GoogleCodeExporter commented 8 years ago 
At least for Firefox 4.0 I found a way to fix the missing texture issure. Enter 
about:config in your address line and set webgl.prefer-native-gl to true.

Original comment by fl.lett...@gmail.com on 9 Mar 2011 at 11:08

GoogleCodeExporter commented 8 years ago 
The latest Chromium build is giving a "WebGL not supported" screen because it 
is throwing an exception. The latest Firefox build had something change with 
drawing vectors that makes a few important things fail.

Original comment by Xath...@gmail.com on 9 Mar 2011 at 11:10

GoogleCodeExporter commented 8 years ago 
Good news -- Firefox 4 runs this natively again. The multiplayer only works if 
you use about:config to turn on "network.websocket.override-security-block" 
(insecure, apparently), but seems to work otherwise without an issue. Enjoy!

Original comment by Xath...@gmail.com on 25 Apr 2011 at 8:14

GoogleCodeExporter commented 8 years ago 
[deleted comment]
GoogleCodeExporter commented 8 years ago 
code commited with patch applied

Original comment by stefan.haustein on 11 Oct 2011 at 6:35