Developer forums (C::B DEVELOPMENT STRICTLY!) > Development
exchndl related patch because win32/lib folder is empty in SVN
stahta01:
--- Quote from: scarphin on September 04, 2015, 02:55:55 pm ---
--- Quote from: stahta01 on September 04, 2015, 01:59:09 pm ---But, think of how much easier it is to switch to a new version of wxWidgets by just editing the three custom vars.
--- End quote ---
I don't understand this. wx2.8 and wx3.0 already have separate project files and separate output/intermediate folders. Switching to a minor version like say wx3.0 -> wx3.1 is as easy as modifying the associated global variable. Or am I missing something?
--- End quote ---
No, you are NOT missing anything; what you stated is clearer than what I said.
Tim S.
scarphin:
--- Quote from: scarphin on September 04, 2015, 02:55:55 pm ---
--- Quote from: stahta01 on September 04, 2015, 01:59:09 pm ---But, think of how much easier it is to switch to a new version of wxWidgets by just editing the three custom vars.
--- End quote ---
I don't understand this. wx2.8 and wx3.0 already have separate project files and separate output/intermediate folders. Switching to a minor version like say wx3.0 -> wx3.1 is as easy as modifying the associated global variable. Or am I missing something?
--- End quote ---
Well, this is the current implementation already, I wasn't referring to your patch if that's what you thought. Is your patch trying to further separate the output/intermediate folders for minor versions (i.e. wx3.0, wx3.1)? If yes, I don't think that's a good idea because then the 'update.bat's scattered all along the project and the plugins need to be modified or duplicated per minor revisions as they will ALSO be dependent on these 3 variables. I think that is very error prone.
stahta01:
--- Quote from: scarphin on September 04, 2015, 08:42:05 pm ---
--- Quote from: scarphin on September 04, 2015, 02:55:55 pm ---
--- Quote from: stahta01 on September 04, 2015, 01:59:09 pm ---But, think of how much easier it is to switch to a new version of wxWidgets by just editing the three custom vars.
--- End quote ---
I don't understand this. wx2.8 and wx3.0 already have separate project files and separate output/intermediate folders. Switching to a minor version like say wx3.0 -> wx3.1 is as easy as modifying the associated global variable. Or am I missing something?
--- End quote ---
Well, this is the current implementation already, I wasn't referring to your patch if that's what you thought. Is your patch trying to further separate the output/intermediate folders for minor versions (i.e. wx3.0, wx3.1)? If yes, I don't think that's a good idea because then the 'update.bat's scattered all along the project and the plugins need to be modified or duplicated per minor revisions as they will ALSO be dependent on these 3 variables. I think that is very error prone.
--- End quote ---
So, I guess you are for code duplication?
Because, having to change 10 places is worse than changing one place per CB project in my book.
Edit: Re-reading your post it appears to me that you never looked at my second patch; I suggest you look at it before deciding what it is doing or NOT doing!
Tim S.
stahta01:
Patch 2 and 3 that adds lib without using CB custom vars.
Patch 0003 is needed to remove an external dependency warning.
Edit: Deleted attachments; will post new message with updated attachments later today.
Tim S.
scarphin:
--- Quote from: stahta01 on September 05, 2015, 01:55:35 am ---Edit: Re-reading your post it appears to me that you never looked at my second patch; I suggest you look at it before deciding what it is doing or NOT doing!
--- End quote ---
I've looked at your 2nd patch before I posted, in fact the only download your 2nd patch has is by me. A first glance tells me it replaces the devel and obj folders with cv_bin and cb_obj variables and introduces a cb_lib variable. Now my point is if one changes the output (cb_bin) variable to, say 'output31' in accordance with wx3.1 then the update files for project, codesnippets, doxyblocks, fortran, incrementalsearch, lib_finder etc... all need to be modified to take into account this change. Am I wrong?
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version