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

Enhancements to the CB Build System

<< < (9/9)

MortenMacFly:

--- Quote from: scarphin on February 01, 2016, 02:06:32 pm ---Sorry again.

--- End quote ---
No worries... just tell if you have to pull the trigger. :-)

scarphin:
Sorry for the long delay. After hours of struggling with memory/compiler problems, I finally managed to get cb to compile on my crappy laptop with an old gcc (v4.8.1). I also separated the big patch (by hand, not in the form of valid patches currently) to submit incremental patches. I will prioritize basic/core patches first then will come the feature patches.

1- Attached patch allows for $(project_dir) etc... style macros in 'settings->compiler->other settings tab->advanced options->commands tab->generated files'. I moved the macro replacements to 'macrosmanager.h' because these macros are required elsewhere in the upcoming patches, it's better to handle these in a single location.

Please test.

scarphin:
So much waiting for such a simple patch. Anyway, I attached another patch (unfortunately also including the one in my previous post) as a next step.

2- This one fixes a couple of errors in the build system which I tried to mention in comments and includes/fixes generated files build enhancements/support. As an example see below for what it fixes:
http://forums.codeblocks.org/index.php/topic,20768.msg141598.html
http://forums.codeblocks.org/index.php/topic,20075.msg136997.htmll

Ok, so I stop here unless I get some developer input. Otherwise it will grow into the same big patch again which I don't see any point in achieving. Thanks.

oBFusCATed:
Some comments after a quick look at the patch:
1. If you messed the @return comment line in the AddFile function
2. REPLACE_FILE_MACROS mustn't not be a macro, either a normal or template function!
3. I'd make generatingCompilerIDs to be vector<wxString> instead of wxArrayString. wxArrayString is terrible container, because there is wxString& operator [](...) const that allows the user to modify the contents of const objects.

Do you have a sample project that can be used to test this patch?
I don't see anything wrong in the patch, but I'm not very knowledgeable about the internals cb's build system.

Navigation

[0] Message Index

[*] Previous page