Thread: [pygccxml-development] Patch to let gccxml work
Brought to you by:
mbaas,
roman_yakovenko
From: Allen B. <al...@vr...> - 2006-05-22 16:14:44
Attachments:
pygccxml.patch
|
I have had a patch sitting around in my pygccxml code for a while to actually allow gccxml to work on my code base. You see, some of the code I use and wrap causes gccxml to output warning messages during execution. The current pygccxml code makes the assumption that if there is any output then there was error compiling the code. This prevents it from working for me. I have attached a patch that fixed the problem for me by judging successful completion by wether or not gccxml actually generates an output file. This may not be the best way to do this, but it works for me. -Allen |
From: Roman Y. <rom...@gm...> - 2006-05-22 17:59:33
|
On 5/22/06, Allen Bierbaum <al...@vr...> wrote: > I have had a patch sitting around in my pygccxml code for a while to > actually allow gccxml to work on my code base. > > You see, some of the code I use and wrap causes gccxml to output warning > messages during execution. The current pygccxml code makes the > assumption that if there is any output then there was error compiling > the code. This prevents it from working for me. > > I have attached a patch that fixed the problem for me by judging > successful completion by wether or not gccxml actually generates an > output file. This may not be the best way to do this, but it works for m= e. I remember this patch. Also I remember it does not work for me. I will check it one more time. If it works I will commit it. If it will not work, I have another idea how we can fix it. > -Allen --=20 Roman Yakovenko C++ Python language binding http://www.language-binding.net/ |
From: Allen B. <al...@vr...> - 2006-05-30 19:56:44
|
Roman: Did you ever get a chance to try out your idea? I have other people that want to use pyplusplus on our projects to generate bindings but they can't run my generation scripts until this bug is resolved so they can use gccxml. Thanks, Allen Roman Yakovenko wrote: > On 5/22/06, Allen Bierbaum <al...@vr...> wrote: > >> I have had a patch sitting around in my pygccxml code for a while to >> actually allow gccxml to work on my code base. >> >> You see, some of the code I use and wrap causes gccxml to output warning >> messages during execution. The current pygccxml code makes the >> assumption that if there is any output then there was error compiling >> the code. This prevents it from working for me. >> >> I have attached a patch that fixed the problem for me by judging >> successful completion by wether or not gccxml actually generates an >> output file. This may not be the best way to do this, but it works >> for me. > > > I remember this patch. Also I remember it does not work for me. I will > check it one > more time. If it works I will commit it. If it will not work, I have > another idea how we > can fix it. > >> -Allen > > > |
From: Roman Y. <rom...@gm...> - 2006-05-30 20:19:38
|
On 5/30/06, Allen Bierbaum <al...@vr...> wrote: > Roman: > > Did you ever get a chance to try out your idea? I have other people > that want to use pyplusplus on our projects to generate bindings but > they can't run my generation scripts until this bug is resolved so they > can use gccxml. Allen, I am running against the time. My family is going to grow in next few days may be week. I will have son! So I try to fix critical bugs you and other users reports. May be you can try latest CVS version of GCC-XML and your bug will gone? If not my idea was simple: we can find out ( or ask from an user ) the version of GCC-XML, and then to implement different error check mode. If you can change your patch to work like this I will apply it. The main reason for such implementation - you are the only one who reports this bug. I want to disable this special error treatment for other users. Thanks -- Roman Yakovenko C++ Python language binding http://www.language-binding.net/ |
From: Roman Y. <rom...@gm...> - 2006-05-30 20:46:20
|
On 5/30/06, Allen Bierbaum <al...@vr...> wrote: > Roman Yakovenko wrote: > > > On 5/30/06, Allen Bierbaum <al...@vr...> wrote: > > > >> Roman: > >> > >> Did you ever get a chance to try out your idea? I have other people > >> that want to use pyplusplus on our projects to generate bindings but > >> they can't run my generation scripts until this bug is resolved so they > >> can use gccxml. > > > > > > Allen, I am running against the time. My family is going to grow in > > next few days > > may be week. I will have son! > > Congrats. Now I can see why you are hurrying so much to get things tied > up. :) Thanks :-) > > So I try to fix critical bugs you and other users > > reports. May be you can try latest CVS version of GCC-XML and your bug > > will gone? > > I don't think this will fix it because it is not a bug in gccxml. It is > just that gccxml detects some issues with the code it is processing so > it outputs some warnings. Unfortunately pygccxml seems to interpret any > output from gccxml as an error condition. > > > > > If not my idea was simple: we can find out ( or ask from an user ) the > > version of > > GCC-XML, and then to implement different error check mode. > > If you can change your patch to work like this I will apply it. > > Have you found problems with the idea of assuming gccxml worked > correctly if the output file is created? > > > > > The main reason for such implementation - you are the only one who > > reports > > this bug. I want to disable this special error treatment for other users. > > I think I am the only one that sees this because I have some rather > "nasty" code that causes gccxml to have problems and output warning > messages. It could be nice to have those "nasty" things within regression. I think I will implement next solution: 1. you will send me a patch 2. I will add new variable to config_t class: ignore_gccxml_output By default it will be false. So you will have to turn it on explicitly. 3. The code that actually runs gccxml will be changed, to take into account this variable? Is this good for you? Allen, I am sorry, but it could be nice if you will contribute the code. Thanks -- Roman Yakovenko C++ Python language binding http://www.language-binding.net/ |
From: Allen B. <al...@vr...> - 2006-05-30 21:02:18
Attachments:
ignore_gccxml_output.patch
|
> > It could be nice to have those "nasty" things within regression. > I think I will implement next solution: > 1. you will send me a patch > 2. I will add new variable to config_t class: > ignore_gccxml_output > By default it will be false. So you will have to turn it on > explicitly. > 3. The code that actually runs gccxml will be changed, to take into > account > this variable? > > Is this good for you? Allen, I am sorry, but it could be nice if you will > contribute the code. > > Thanks > I have attached a patch that implements what you are describing. It is not pretty but it works for me and should allow other people to ignore gccxml output as well. Let me know what you think. -Allen |
From: Roman Y. <rom...@gm...> - 2006-05-30 21:07:39
|
On 5/31/06, Allen Bierbaum <al...@vr...> wrote: > > > > > It could be nice to have those "nasty" things within regression. > > I think I will implement next solution: > > 1. you will send me a patch > > 2. I will add new variable to config_t class: > > ignore_gccxml_output > > By default it will be false. So you will have to turn it on > > explicitly. > > 3. The code that actually runs gccxml will be changed, to take into > > account > > this variable? > > > > Is this good for you? Allen, I am sorry, but it could be nice if you will > > contribute the code. > > > > Thanks > > > I have attached a patch that implements what you are describing. It is > not pretty but it works for me and should allow other people to ignore > gccxml output as well. > > Let me know what you think. Looks nice, exactly what I thought. I will apply it tommorrow. Thank you. -- Roman Yakovenko C++ Python language binding http://www.language-binding.net/ |
From: Roman Y. <rom...@gm...> - 2006-05-31 03:56:19
|
Commited. Thank you -- Roman Yakovenko C++ Python language binding http://www.language-binding.net/ |