Message to oBFusCATed or other CB Devs about patches to CB Projects.

[stahta01]

FYI:

oBFusCATed, you had NOT been applying my changes in
https://github.com/stahta01/codeblocks_sf_tims/tree/for_upstream
So, I decided that you did NOT like the changes moved those old changes to
https://github.com/stahta01/codeblocks_sf_tims/tree/old_for_upstream

I reset hard for_upstream and added 3 changes to it.

1. Location fix for VirtualBuildTargetsDlg.wxs related files it is now in src folder; but, no one updated it from the old sdk location.
2. Core app & plugins contains sdk which I consider wrong; do you agree?
    And, it is missing Abbreviations.
3. The Windows CB projects Addr2LineUI uses WX_VER instead of WX_VERSION

I am now retesting these changes with Windows 7 32 bit.
I never tested the changes with any NON windows OSes.

Tim S.

[oBFusCATed]

I'm not following your branches and I'll review and apply patches only if I'm ask to do so.

So if you have a patch you think it is ready for push - start a topic or post in one of your older topics about it.
If I don't answer within a week post a reminder message.

Sorry if there is some misunderstanding.

[oBFusCATed]

2. Core app & plugins contains sdk which I consider wrong; do you agree?
    And, it is missing Abbreviations.

I'm not really sure what is the correct setup here.
I've never used or seen it, so I can't comment much what its purpose is.
Morten and others do you know the original idea behind this target?

[MortenMacFly]

2. Core app & plugins contains sdk which I consider wrong; do you agree?
    And, it is missing Abbreviations.

I'm not really sure what is the correct setup here.
I've never used or seen it, so I can't comment much what its purpose is.
Morten and others do you know the original idea behind this target?

Sorry but I don't understand what that means...Surely core and apps and plugins need the sdk....
??

[stahta01]

2. Core app & plugins contains sdk which I consider wrong; do you agree?
    And, it is missing Abbreviations.

I'm not really sure what is the correct setup here.
I've never used or seen it, so I can't comment much what its purpose is.
Morten and others do you know the original idea behind this target?

Sorry but I don't understand what that means...Surely core and apps and plugins need the sdk....
??

The core app also need the third party libs that are NOT part of that virtual target.

My use case of the changed
1. Build/rebuild "third party libs"
2. Build/rebuild sdk
3. Build/rebuild "Core app & plugins"

NOTE: I am using Git under windows this result in very slow builds of the SDK because of "git svn info" takes a minute to 5 minutes to work.

Tim S.

[stahta01]

Removed the change to "Core app & plugins" Virtual Target.
If someone still want to apply it; moved it to this branch
https://github.com/stahta01/codeblocks_sf_tims/tree/build/rejectedPatches
I am guessing that this patch only makes sense to Git Windows users; because the main gain is it is slightly faster re-build CB using it.


I now have 6 commits for consideration in branch https://github.com/stahta01/codeblocks_sf_tims/tree/for_upstream

The first 3 are PCH usage related.
Next 3 are CB core project missing file related.

Tim S.

[MortenMacFly]

I would like to highlight the fact that besides you've been told so not all devs use git and can make use of these patches that way... It may not be clear to all... So hopefully obf will pick it up as he is the one using git most intensively...

[stahta01]

I would like to highlight the fact that besides you've been told so not all devs use git and can make use of these patches that way... It may not be clear to all... So hopefully obf will pick it up as he is the one using git most intensively...

I have many SVN patches, also; But, I have gotten the best result using Git and Obfuscated.

Link to 4 SVN patches; that I plan to re-test next. Plan to wait a week then add them to my Git up_stream branch.
https://github.com/stahta01/cb_misc/blob/master/Patches/svn/CB_project_bug_04_remove_wxscintilla_cb.patch
https://github.com/stahta01/cb_misc/blob/master/Patches/svn/CB_project_bug_05_remove_wxpropgrid.patch
https://github.com/stahta01/cb_misc/blob/master/Patches/svn/CB_project_bug_07_remove_depends_on_lib_finder.patch
https://github.com/stahta01/cb_misc/blob/master/Patches/svn/CB_project_bug_08_remove_or_fix_update_unit_filename.patch

Edit2: To download a patch just click on RAW button on web page.

Patch 4; removes incorrect reference to wxscintilla_cb.
Patch 5; removes incorrect reference to wxpropgrid.
Patch 7; Replaces depends of lib_finder with depends on wxContribItems.
Patch 8; Makes sure the update script/batch is correct for the CB Project and removes bat from Unix Projects and scripts from Windows Projects.

Note: I have only test these change using Windows 32 bit (both wx28 and wx30).
Edit: This means I did NOT test Linux/Unix or Windows64 CB Projects.

So, the changes really need tested by a Linux user before applying.

Tim S.

[oBFusCATed]

The patches look fine, but can you cjange the commit messages to something like "- projects: Bla bla blablabla (thanks stahta01)"?
This way I can commit them pretty easily and I won't have to spend time fixing the messages.

hint: Use git rebase -i HEAD^6 to do it.

[stahta01]

The patches look fine, but can you cjange the commit messages to something like "- projects: Bla bla blablabla (thanks stahta01)"?
This way I can commit them pretty easily and I won't have to spend time fixing the messages.

hint: Use git rebase -i HEAD^6 to do it.

I will try.

[stahta01]

Took me a half dozen tries; but, I think I have done it.

Using this command

Code

git rebase -i HEAD~6

After changing editor to notepad++ using this command; VIM is too much for me to try.

Found on http://stackoverflow.com/questions/1634161/how-do-i-use-notepad-or-other-with-msysgit

Code

git config --global core.editor "'C:/Program Files/Notepad++/notepad++.exe' -multiInst -notabbar -nosession -noPlugin"

[stahta01]

Just added the message prefix to three of the commits in Git branch of "PCH Fix: ".
Decided that made more sense.

Tim S.
 

[oBFusCATed]

Keep in mind that we are using the first character of the message to designate if  the commit is important to the user. '-' is used for build fixes and other internal or minimal changes. '*' is used for changes to the ui, new features, fixing important bugs, etc. This help when making a release to gather to logs of changes.

Can I ask you to do another rebase and add '-' at the beginning of every commit?

[stahta01]

Keep in mind that we are using the first character of the message to designate if  the commit is important to the user. '-' is used for build fixes and other internal or minimal changes. '*' is used for changes to the ui, new features, fixing important bugs, etc. This help when making a release to gather to logs of changes.

Can I ask you to do another rebase and add '-' at the beginning of every commit?

OK.

Will do so.

Tim S.

[stahta01]

Will take a little time

FYI: "src/sdk/resources/lexers/lexer_bibtex.sample" is causing me problem because Git is complaining about LF issues.

But, I was messing with using using an .gitattributes file.

I had to tell Git that the file is Binary in it.

Should only take a few minutes more.

Tim S.