Developer forums (C::B DEVELOPMENT STRICTLY!) > Development
Move Occurrence highlighting from core into a plugin
oBFusCATed:
--- Quote from: MortenMacFly on July 30, 2013, 03:39:01 pm ---...(I assume its GIT based again)...
--- End quote ---
It is not in git format, looks like a valid svn patch.
ollydbg:
--- Quote from: oBFusCATed on July 30, 2013, 03:40:24 pm ---
--- Quote from: MortenMacFly on July 30, 2013, 03:39:01 pm ---...(I assume its GIT based again)...
--- End quote ---
It is not in git format, looks like a valid svn patch.
--- End quote ---
--- Code: ---commit 76a459e2d6c0f8d420ac4bf20807abb73e02052e
Author: Jens Lody <jens@codeblocks.org>
Date: Tue Jul 30 01:50:04 2013 +0200
* additional patch for move of occurrences highlighting
Index: acinclude.m4
===================================================================
--- acinclude.m4
+++ acinclude.m4
@@ -170,6 +170,17 @@
AC_MSG_RESULT(no)
--- End code ---
Look at the first line, it should be a git patch, Jens may use some utility tool to translate this patch a little to make it svn compatible.
oBFusCATed:
--- Quote from: ollydbg on July 30, 2013, 03:53:53 pm ---Look at the first line, it should be a git patch, Jens may use some utility tool to translate this patch a little to make it svn compatible.
--- End quote ---
You can have whatever you want before the Index lines. The patch is in a format that is almost like a patch produced by svn.
If you remove the top lines you won't be able to guess that it has been produced with git.
The thing is that Morten uses some broken patch utility, so it is not a problem that the patch is broken, but the tool Morten is using is broken.
From what I remember it is pretty hard to find working patch utility for windows. :'(
ollydbg:
--- Quote from: oBFusCATed on July 30, 2013, 04:07:09 pm ---The thing is that Morten uses some broken patch utility, so it is not a problem that the patch is broken, but the tool Morten is using is broken.
From what I remember it is pretty hard to find working patch utility for windows. :'(
--- End quote ---
I do not have any problem about the patch utility.
For the record, the most recent patch utility is in MSYS2 package
--- Quote ---New MSYS2 snapshots:
32-bit: x32-msys2-alpha-20130728.tar.xz http://sourceforge.net/projects/msys2/files/Alpha-versions/32-bit/x32-msys2-alpha-20130728.tar.xz/download
64-bit: x64-msys2-alpha-20130728.tar.xz http://sourceforge.net/projects/msys2/files/Alpha-versions/64-bit/x64-msys2-alpha-20130728.tar.xz/download
--- End quote ---
MortenMacFly:
--- Quote from: oBFusCATed on July 30, 2013, 04:07:09 pm ---The thing is that Morten uses some broken patch utility, so it is not a problem that the patch is broken, but the tool Morten is using is broken.
--- End quote ---
Nope - it works 100% if you use SVN diff. I'd say its just purely SVN oriented.
--- Quote from: oBFusCATed on July 30, 2013, 04:07:09 pm ---From what I remember it is pretty hard to find working patch utility for windows. :'(
--- End quote ---
However, thats true as well. Thing is I don't want to use a console utility if I have a full-blown GUI-based IDE. Thats basically also the reason why I prefer Windows over Linux.
If I were a console guy and there were a nice utility I wouldn't mind at all and take the time to even compile patch myself. The point is: Its actually so easy to create as working SVN patch. So why do work-arounds?
I think we had this discussion now over and over again. So please, whenever I post "I cannot apply this" just take it as a hint that besides me probably other user cannot try what you've posted and you limit the testers. If thats fine with you, everything is alright!
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version