|
jens
|
 |
« Reply #15 on: July 28, 2009, 05:18:38 pm » |
|
We parse the date with a regex, if we use automake on linux.
Linux I don't know. But he was talking about Windows. However - what's wrong with using the format as it is used in the (any) SVN repository? I mean: We *are* talking about a SVN revision here, right? I know that he is talking about windows. I don't know, why subversion uses this format for date/time of last commit (internally), but are there any objections against using a format, that's (better) human-readable for the date and time ? The only cause would be, if we have to distinguish between to revision, that came in at the same second, but we also use the revision-number, so I don't see any problem. And I don't think the patch to autorevision.cpp can break anything, unless the svn date-format changes. Better consult with Thomas first. Once I wanted to fix this issue. But he didn't want to touch that code. I forgot the exact objection.  Thomas ?
|
|
|
|
|
Logged
|
|
|
|
Biplab
Scripts wizard
Lives here!
  
Posts: 1640
|
 |
« Reply #16 on: July 28, 2009, 05:20:09 pm » |
|
We parse the date with a regex, if we use automake on linux.
Linux I don't know. But he was talking about Windows. However - what's wrong with using the format as it is used in the (any) SVN repository? I mean: We *are* talking about a SVN revision here, right? I know that he is talking about windows. I don't know, why subversion uses this format for date/time of last commit (internally), but are there any objections against using a format, that's (better) human-readable for the date and time ? The only cause would be, if we have to distinguish between to revision, that came in at the same second, but we also use the revision-number, so I don't see any problem. And I don't think the patch to autorevision.cpp can break anything, unless the svn date-format changes. Better consult with Thomas first. Once I wanted to fix this issue. But he didn't want to touch that code. I forgot the exact objection.  Thomas ? Yes!!
|
|
|
|
|
Logged
|
Be a part of the solution, not a part of the problem.
|
|
|
|
jens
|
 |
« Reply #17 on: July 28, 2009, 05:27:48 pm » |
|
I just cried for him, so he can not overread it, if he looks through unread topics.
|
|
|
|
|
Logged
|
|
|
|
|
|
squalyl
Newcomer

Posts: 9
|
 |
« Reply #19 on: July 28, 2009, 11:35:48 pm » |
|
dear codeblocks team I love you! Just noticed some new features in this nightly: automatic } when I type a {. That's lovely. It saves a lot of time. BUT: what did you do with the patch that saved me, about SDCC and static libs? here: http://forums.codeblocks.org/index.php/topic,10741.msg73652.html#msg73652I hoped it would be integrated in new nightlies  (note : the change is fully OK, is there any reason not to integrate it in the main branch?) crash report: http://www.mirari.fr/RUb0
|
|
|
|
|
Logged
|
|
|
|
|
MortenMacFly
|
 |
« Reply #20 on: July 29, 2009, 08:20:02 am » |
|
BUT: what did you do with the patch that saved me, about SDCC and static libs?
...probably overseen? The next nightly will take care. I've submitted the modification.
|
|
|
|
|
Logged
|
|
|
|
sbilsing
Newcomer

Posts: 3
|
 |
« Reply #21 on: July 30, 2009, 09:33:33 am » |
|
Hello CodeBlocks team, First, CB is great. I use the nightly builds even for production. The degree of flexibility is just beyond everything I know from certain M$ products. But the last 3 nightly builds include a bug in the command macro expansion. I filed a bug report here: http://developer.berlios.de/bugs/?func=detailbug&bug_id=16072&group_id=5358Regards S.B.
|
|
|
|
|
Logged
|
|
|
|
|
|
|
Loaden
|
 |
« Reply #23 on: July 30, 2009, 01:53:20 pm » |
|
svn build rev 5717 (2009-07-28T07: 15:12.319835 Z) gcc 4.4.1 Windows / unicode
That's not wrong, that's exactly how SVN reports the date/time. It's just "a" format. Check any "entries" file in a .svn sub-foldr if you like. It's all the same. We parse the date with a regex, if we use automake on linux. We can do something similar on windows (of course without regexes). Any objections against this patch: Index: autorevision.cpp =================================================================== --- autorevision.cpp (Revision 5716) +++ autorevision.cpp (Arbeitskopie) @@ -122,6 +122,8 @@ if(d && d->GetText()) { date = d->GetText(); + date.replace(date.find('T'),1," "); + date.resize(date.find('.')); } return 1; }
Thanks! It's ok: svn build rev 5722 (2009-07-30 09:59:52) gcc 4.4.1 Windows/unicode
|
|
|
|
|
Logged
|
Arch & XP -|- GCC & VC -|- Code::Blocks SVN Latest------------------------------------------ Index for my patches
|
|
|
|
killerbot
|
 |
« Reply #24 on: July 30, 2009, 04:04:35 pm » |
|
anyone already an idea on this regression ? wxAUi related ?
|
|
|
|
|
Logged
|
|
|
|
|
MortenMacFly
|
 |
« Reply #25 on: July 30, 2009, 04:31:16 pm » |
|
anyone already an idea on this regression ? wxAUi related ?
If this is true: Apparently the bug was introduced in svn build 5678. ...then "no". Because the wxAUI merge came later. EDIT: Cannot even reproduce. Works fine here...?!
|
|
|
|
« Last Edit: July 30, 2009, 04:44:53 pm by MortenMacFly »
|
Logged
|
|
|
|
|
Loaden
|
 |
« Reply #26 on: July 30, 2009, 06:38:06 pm » |
|
|
|
|
|
|
Logged
|
Arch & XP -|- GCC & VC -|- Code::Blocks SVN Latest------------------------------------------ Index for my patches
|
|
|
|
jens
|
 |
« Reply #27 on: July 30, 2009, 07:02:07 pm » |
|
anyone already an idea on this regression ? wxAUi related ?
If this is true: Apparently the bug was introduced in svn build 5678. ...then "no". Because the wxAUI merge came later. EDIT: Cannot even reproduce. Works fine here...?! I can reproduce here (linux 64-bit). I did not look into it deeper (no time), but I found a workaround (that might be a hint to find the bug): double the $-sign and it works again (at least here).
|
|
|
|
|
Logged
|
|
|
|
|
jens
|
 |
« Reply #28 on: July 30, 2009, 09:12:41 pm » |
|
It's commit 5636, that leads to the error. I did not (yet) look for a solution.
|
|
|
|
|
Logged
|
|
|
|
|
killerbot
|
 |
« Reply #29 on: July 30, 2009, 10:05:45 pm » |
|
which is a change to 'compilercommandgenerator.cpp' base upon work of 'Biplab' : "replace macros for compiler options". Basically 2 lines have been added, doing a macro substitution, could it be it is carried out too early ??
|
|
|
|
|
Logged
|
|
|
|
|