Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

rev 3967 does not build on linux (and maybe others)

<< < (2/2)

stahta01:

--- Quote from: juhas on May 20, 2007, 09:36:54 pm ---
--- Quote ---Found, my patch listed in thread below. I don't remember if it worked or not. My final patch did, but not sure this is my final patch.
http://forums.codeblocks.org/index.php/topic,5681.msg43657.html#msg43657

--- End quote ---

It works, thanks a lot!
Maybe developers could apply this patch in new rev?


--- End quote ---

I worked on the patch over a month ago, I just don't have time to confirm it's safe to use and keep updating it. Code::Blocks is changing too fast for me to maintain patches I don't care about.
To me it is easier to use the libraries as built-in, but I am a windows user it defaults to built-in normally, I saw the issue under cygwin testing.

It's in the bug filed on the problem.

http://developer.berlios.de/bugs/?func=detailbug&bug_id=10881&group_id=5358

Tim S

Biplab:
As commented on the bug report, the patch will avoid compilation error. But it may create a False sense of an Advanced Feature.

IMHO, it would not be a good idea to keep that option enabled if wxRE_ADVANCED is not at all available during compilation. If an user tries to use advanced regex function with wxRE_ADVANCED set to 0, the function will not work appropriately and we may see another bug report reporting that the advanced regex function is not working properly.

This is my concern about the patch. I think, it'd be better if the advanced regex functions are #ifdef'ed.

Regards,

Biplab

Navigation

[0] Message Index

[*] Previous page