eXistence / fhDOOM

Modernized DOOM3/idTech4 engine: ported to modern OpenGL (core profile), enhanced visual effects, improved performance, improved editor, more to come
GNU General Public License v3.0
180 stars 24 forks source link

Linux - errors on compilation #5

Closed volumetricsteve closed 7 years ago

volumetricsteve commented 7 years ago

Hello,

As described in the "Building on Linux" issue, I was getting errors about intptr_t and Class.h

I fixed this by adding:

include

to the beginning of Class.h

Apparently implementations, of intptr_t vary based on your environment or compiler, so sometimes it's implied, sometimes it's not, I'm guessing it's implied with the Microsoft stuff, but not with GCC 6.3

Either way, I've gotten past that now - just letting you know what fixed that particular issue.

Cheers

BielBdeLuna commented 7 years ago

you would be far better if you add a pull request

volumetricsteve commented 7 years ago

Github makes zero sense to me. I'll try to figure it out......I guess.

*edit

Nope. I hate this interface. If you want to, go for it, I have no patience for the way github is laid out. I'll provide tips, hopefully the main guy 'eXistence' sees them, agrees with them, and makes the changes he wants to his thing. If he doesn't like them, then cool. It's his thing, he should keep it his way.

Depending on how involved he still is (I hope you're out there, man) I may fork this and slowly pick away at issues with compiling in linux. Ideally I'd make it linux-only because windows is dead to me, but I'll take this one fight at a time.

BielBdeLuna commented 7 years ago

if you make the change in the file you just press pull request (in order to save your changes) and you're set

eXistence commented 7 years ago

I am on vacation next week, so i can't look into the issue right now, but i will do so as soon as i am back.

volumetricsteve commented 7 years ago

I think I made the pull request work...The videos I saw on google for what to do in -no way- resembled the perfectly straight forward thing you suggested, BielBdeLuna. Either way, hopefully when eXistence gets back he can do it the way he wants.

BielBdeLuna commented 7 years ago

perfect!

eXistence commented 7 years ago

fixed missing header (see pull request), thx for reporting :)