nel-all
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Nel] problems compiling.


From: HERNANDEZ CORDOBA RODRIGO JOSE
Subject: Re: [Nel] problems compiling.
Date: Tue, 02 Nov 2004 14:37:01 -0600

That works when creating patches for the same repository you are working
on, I am yet to find an option in tortoise (which is not command line)
that allows you to create a patch between 2 different repositories

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

> i use tortoise CVS on windows machne to generate patches fast and 
> per file
> 
> simply a matter of right click CVS > Make Patch
> 
> and out pops filename.patch
> 
> 
> 
> ----- Original Message ----- 
> From: "HERNANDEZ CORDOBA RODRIGO JOSE" <address@hidden>
> To: "Developer's list for the NeL platform" <address@hidden>
> Sent: Tuesday, November 02, 2004 1:24 PM
> Subject: Re: [Nel] problems compiling.
> 
> 
> > 
> > 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" <nel-
> 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
> >> 
> > 
> > 
> > 
> > _______________________________________________
> > 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
> 





reply via email to

[Prev in Thread] Current Thread [Next in Thread]