Re: [Nel] problems compiling.

[HERNANDEZ CORDOBA RODRIGO JOSE]

Well, nevermind, as I said that patch is stale, no longer relevant, HUGE
because it has a lot of cvs related $Id line changes, not even compiles,
and so on, so don't worry about it. :)

I'll let you know when there is something to show on my side, in the
meantime, could you (or someone else) help me out and give me the proper
command line args to "cvs diff" or "diff" to get small nice patches?

Thanks.

----- Mensaje Original -----
De: "Andi [ Debug ]" <address@hidden>
Fecha: Martes, Noviembre 2, 2004 1:12 pm
Asunto: Re: [Nel] problems compiling.

> it is a huge patch the best bet is doing it per file
> 
> mesh.cpp.patch
> mech_blender.cpp.patch
> 
> and so forth
> 
> preferably made as close to the time you send them as possible
> it just seemed like a huge patch from someone who doesnt know NeL 
> as well as 
> maybe
> they should before making such earth shattering changes :)
> 
> 
> 
> ----- Original Message ----- 
> From: "HERNANDEZ CORDOBA RODRIGO JOSE" <address@hidden>
> To: "Developer's list for the NeL platform" <address@hidden>
> Sent: Tuesday, November 02, 2004 12:53 PM
> Subject: Re: [Nel] problems compiling.
> 
> 
> >
> > Gee, I was trying to contribute (not impose) my ideas, time and 
> efford,> which is why I did set up my own cvs fork, you where the 
> one who told me
> > a couple of weeks ago to post a patch, I did, and it came up 
> today, 2
> > weeks late and way stale.
> >
> > I did thought at the time that it was too early to post a patch, and
> > figured you just wanted to use it to find a solution to my 
> problem, it
> > wasn't meant to be merged.
> >
> > I can't IRC for about 5 more hours, maybe then I will log in and see
> > what happens, it would be nice to at least know if the changes would
> > possibly be merged once finished or not.
> >
> > Cheers!
> >
> > P.S. Sorry about the all caps sender field, I am not retarded, the
> > people running my ISP's webmail are :)
> >
> > ----- Mensaje Original -----
> > De: "Andi [ Debug ]" <address@hidden>
> > Fecha: Martes, Noviembre 2, 2004 12:10 pm
> > Asunto: Re: [Nel] problems compiling.
> >
> >> in my opinion someone who has been using NeL for a couple of days
> >> should not
> >> be submitting patches of this magnitude at all
> >>
> >> i maintain the VC 7.1 solutions which are work in progress and add
> >> more
> >> every week when nevrax has time they work without fail and i have
> >> not failed
> >> in guiding anyone through compile yet with MSVC 6.0, 7.0 and 7.1
> >> so if you
> >> had issues compiling your the minority tell us your issues and we
> >> will reply
> >> with how to get past them :)
> >>
> >> i know of several people who use them and several unix users i
> >> know most of
> >> the people who use NeL full stop and not one has had issues with
> >> paths in
> >> the solutions.( which by the way relative paths are INTEDED lol
> >> they are the
> >> correct way to do it )
> >>
> >> maybe you should come to IRC and talk to us before throwing a huge
> >> patch
> >> into mailing list see if you are just reinventing the wheel or
> >> even
> >> potentially changing something that is inteded to be the way it is.
> >>
> >> irc.freenode.net
> >> #nel
> >>
> >>
> >>
> >> ----- Original Message ----- 
> >> From: "Rodrigo Hernandez" <address@hidden>
> >> To: "Developer's list for the NeL platform" <address@hidden>
> >> Sent: Saturday, October 23, 2004 12:38 PM
> >> Subject: Re: [Nel] problems compiling.
> >>
> >>
> >> >
> >> > I have created a patch (attached) however I do not believe this
> >> is the
> >> > optimal form, as it is 2 mb uncompressed and I am sure I did not
> >> made that
> >> > many changes, its not entirely functional (the code still does
> >> not
> >> > entirely
> >> > compile under MSYS) and has many '$Id:' changes due to the
> >> import that I
> >> > don't think are right.
> >> >
> >> > I am submitting this patch with the hopes that it may be useful
> >> for
> >> > example
> >> > to get the VC++ 7.1 projects and solution files to a more
> >> portable state,
> >> > since they have relative paths included in themselves, and some
> >> other> changes related to MSYS compilation (autotools and defines
> >> mostly).>
> >> > Also so I can get some help with the problems I described 
> before :)
> >> >
> >> > Once I get the source tree to properly compile I will create a
> >> proper
> >> > patch.
> >> >
> >> > Thanks :)
> >> >
> >> > At 06:49 PM 10/22/2004, you wrote:
> >> >>if you could make .patch files and throw them at the mailing
> >> list that
> >> >>would be handy indeed
> >> >>
> >> >>then vianney lecroart can patch the NeL cvs tree with ones that
> >> are
> >> >>approved
> >> >>
> >> >>
> >> >>----- Original Message ----- From: "HERNANDEZ CORDOBA RODRIGO 
> JOSE">> >><address@hidden>
> >> >>To: "Developer's list for the NeL platform" <address@hidden>
> >> >>Sent: Friday, October 22, 2004 5:04 PM
> >> >>Subject: Re: [Nel] problems compiling.
> >> >>
> >> >>
> >> >>>Hi Andi,
> >> >>>
> >> >>>Great! I updated my cvs yesterday, but before I did was still
> >> having the
> >> >>>replicate symbols errors on VC++7.1.
> >> >>>
> >> >>>also on the clamp asm right at the top I realized after I sent
> >> the email
> >> >>>that instead of:
> >> >>>
> >> >>>mov eax, v
> >> >>>mov ebx, v
> >> >>>
> >> >>>You should do
> >> >>>
> >> >>>mov eax, v
> >> >>>mov ebx, eax
> >> >>>
> >> >>>since moving data from register to register is faster than
> >> memory to
> >> >>>register, in any case in both cases eax and ebx end up with the
> >> same>>>value, you can check my copy of the cvs tree to see the
> >> inline gas
> >> >>>version, I also changed some other asm around and added a 
> check for
> >> >>>3DNow! on cpu_info.cpp I think it is.
> >> >>>
> >> >>>Cheers! ;)
> >> >>>
> >> >>>
> >> >>>----- Mensaje Original -----
> >> >>>De: "Andi [ Debug ]" <address@hidden>
> >> >>>Fecha: Viernes, Octubre 22, 2004 4:06 pm
> >> >>>Asunto: Re: [Nel] problems compiling.
> >> >>>
> >> >>>>i wlil have a look maybe i need to update the projects 
> based on
> >> >>>>the new cvs
> >> >>>>tree i do this only once a week
> >> >>>>on fridays
> >> >>>>
> >> >>>>oh thats today :)
> >> >>>>
> >> >>>>
> >> >>>>----- Original Message ----- From: "HERNANDEZ CORDOBA RODRIGO
> >> JOSE">>>><address@hidden>
> >> >>>>To: <address@hidden>
> >> >>>>Sent: Friday, October 22, 2004 2:53 PM
> >> >>>>Subject: [Nel] problems compiling.
> >> >>>>
> >> >>>>
> >> >>>> > Hello all,
> >> >>>> > I am having some troubles compiling, some I am not sure 
> if its
> >> >>>>due to my
> >> >>>> > changes or due to the latest changes on CVS, hope you can
> >> help me:
> >> >>>> >
> >> >>>> > On VC++7.1 I get tons of already defined symbol errors,
> >> here is
> >> >>>>a sample:
> >> >>>> >
> >> >>>> > nlmisc_d.lib(debug.obj) : error LNK2005: 
> _RtlSecureZeroMemory>> >>>>already> defined in nlmisc_d.lib(log.obj)
> >> >>>> > nlmisc_d.lib(debug.obj) : error LNK2005: _InterlockedAnd64
> >> already>>>> > defined in nlmisc_d.lib(log.obj)
> >> >>>> > nlmisc_d.lib(debug.obj) : error LNK2005: _InterlockedOr64
> >> already>>>> > defined in nlmisc_d.lib(log.obj)
> >> >>>> > nlmisc_d.lib(debug.obj) : error LNK2005: _InterlockedXor64
> >> already>>>> > defined in nlmisc_d.lib(log.obj)
> >> >>>> > nlmisc_d.lib(debug.obj) : error LNK2005:
> >> _InterlockedIncrement64>>>>already> defined in 
> nlmisc_d.lib(log.obj)>> >>>> > nlmisc_d.lib(debug.obj) : error 
> LNK2005:>> _InterlockedDecrement64>>>>already> defined in 
> nlmisc_d.lib(log.obj)>> >>>> > nlmisc_d.lib(debug.obj) : error 
> LNK2005: _InterlockedExchange64
> >> >>>>already> defined in nlmisc_d.lib(log.obj)
> >> >>>> > nlmisc_d.lib(debug.obj) : error LNK2005:
> >> _InterlockedExchangeAdd64>>>> > already defined in
> >> nlmisc_d.lib(log.obj)>>>> >
> >> >>>> > Those seem to be all windows functions so I have no clue 
> about>> >>>>what is
> >> >>>> > wrong.
> >> >>>> >
> >> >>>> > Then on device_manager.cpp there is the line
> >> >>>> >
> >> >>>> > #include "nel/misc/device_manager.h"
> >> >>>> >
> >> >>>> > However the file device_manager.h is no where to be 
> found, the
> >> >>>>file only
> >> >>>> > has an empty constructor so I asume everything is inlined
> >> in the
> >> >>>>header> or the class is empty, I created an empty class 
> inside the
> >> >>>>cpp file and
> >> >>>> > removed the include line, but I dont know if this is ok 
> or not.
> >> >>>> >
> >> >>>> > I am making changes so it is possible to compile NEL on MSYS
> >> >>>>with MinGW.
> >> >>>> >
> >> >>>> > I also found a bug on the fastClamp8(sint &v) function in
> >> >>>>algo.h, the
> >> >>>> > asm reads:
> >> >>>> > // clamp v in 0..255 (no cond jmp)
> >> >>>> > __asm
> >> >>>> > {
> >> >>>> > mov esi, v
> >> >>>> > mov eax, [esi]
> >> >>>> > mov ebx, eax
> >> >>>> > // clamp to 0.
> >> >>>> > add eax, 0x80000000
> >> >>>> > sbb ecx, ecx
> >> >>>> > not ecx
> >> >>>> > and ebx, ecx
> >> >>>> > // clamp to 255.
> >> >>>> > add eax, 0x7FFFFF00
> >> >>>> > sbb ecx, ecx
> >> >>>> > or ebx, ecx
> >> >>>> > and ebx, 255
> >> >>>> > // store
> >> >>>> > mov [esi], ebx
> >> >>>> > }
> >> >>>> >
> >> >>>> > but it should be:
> >> >>>> >
> >> >>>> > __asm
> >> >>>> > {
> >> >>>> >    mov eax, v
> >> >>>> >    mov ebx, v
> >> >>>> >    // clamp to 0.
> >> >>>> >    add eax, 0x80000000
> >> >>>> >    sbb ecx, ecx
> >> >>>> >    not ecx
> >> >>>> >    and ebx, ecx
> >> >>>> >    // clamp to 255.
> >> >>>> >    add eax, 0x7FFFFF00
> >> >>>> >    sbb ecx, ecx
> >> >>>> >    or ebx, ecx
> >> >>>> >    and ebx, 255
> >> >>>> >    // store
> >> >>>> >    mov v, ebx
> >> >>>> > }
> >> >>>> >
> >> >>>> > Anyway, I have a public CVS where I keep my imported 
> copy of
> >> >>>>Nel, if
> >> >>>> > someone wants to take a look the CVSROOT is
> >> >>>> >
> >> >>>> > :pserver:address@hidden:/cvsrepo
> >> >>>> >
> >> >>>> > the module is 'nevrax' just hit enter for the password, 
> and be
> >> >>>>warned, I
> >> >>>> > added the game data to it, so its not quite dial up 
> friendly,>> >>>>you should
> >> >>>> > be able to run a diff against the official CVS, though I
> >> dont really
> >> >>>> > know how yet :)
> >> >>>> >
> >> >>>> > Thanks for any help.
> >> >>>> >
> >> >>>> >
> >> >>>> >
> >> >>>> >
> >> >>>> >
> >> >>>> > _______________________________________________
> >> >>>> > Nel mailing list
> >> >>>> > address@hidden
> >> >>>> > http://lists.nongnu.org/mailman/listinfo/nel-all
> >> >>>> >
> >> >>>> >
> >> >>>>
> >> >>>>
> >> >>>>
> >> >>>>
> >> >>>>_______________________________________________
> >> >>>>Nel mailing list
> >> >>>>address@hidden
> >> >>>>http://lists.nongnu.org/mailman/listinfo/nel-all
> >> >>>
> >> >>>
> >> >>>
> >> >>>_______________________________________________
> >> >>>Nel mailing list
> >> >>>address@hidden
> >> >>>http://lists.nongnu.org/mailman/listinfo/nel-all
> >> >>>
> >> >>
> >> >>
> >> >>
> >> >>
> >> >>_______________________________________________
> >> >>Nel mailing list
> >> >>address@hidden
> >> >>http://lists.nongnu.org/mailman/listinfo/nel-all
> >> >
> >> > Rodrigo Hernandez, lonewolf programmer
> >> > Aeon Games
> >> > http://www.aeongames.com
> >> >
> >>
> >>
> >> ----------------------------------------------------------------
> ---
> >> -------------
> >>
> >>
> >> > _______________________________________________
> >> > Nel mailing list
> >> > address@hidden
> >> > http://lists.nongnu.org/mailman/listinfo/nel-all
> >> >
> >>
> >>
> >>
> >>
> >> _______________________________________________
> >> Nel mailing list
> >> address@hidden
> >> http://lists.nongnu.org/mailman/listinfo/nel-all
> >>
> >
> >
> >
> > _______________________________________________
> > Nel mailing list
> > address@hidden
> > http://lists.nongnu.org/mailman/listinfo/nel-all
> >
> > 
> 
> 
> 
> 
> _______________________________________________
> Nel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/nel-all
>