Side navigation
#2908 closed feature (fixed)
Opened May 20, 2008 11:06PM UTC
Closed May 25, 2008 10:10PM UTC
Patch - Validate Plugin - Custom Messages in Metadata
Reported by: | DaneOConnor | Owned by: | joern |
---|---|---|---|
Priority: | major | Milestone: | 1.2.4 |
Component: | plugin | Version: | 1.2.3 |
Keywords: | validate | Cc: | |
Blocked by: | Blocking: |
Description
Attached is a patch for the validate plugin. The patched plugin supports custom messages specified in metadata. It also includes a small example using the new feature in a few different ways.
If there is anything I should change let me know :)
Attachments (4)
Change History (8)
Changed May 21, 2008 12:14AM UTC by comment:1
Changed May 21, 2008 02:37PM UTC by comment:2
Ok, I converted the patch. It should now have Unix line endings, though I'm not totally sure. Does it work?
Changed May 22, 2008 11:49AM UTC by comment:3
That patch still tries to replace everything, that isn't helpful. It looks like you need to convert line-endings before creating the patch.
Changed May 22, 2008 02:56PM UTC by comment:4
Ok, sorry about all that. I've never really had to deal with that issue before. I test applied the patch and it looks like its working. If, this doesn't work I suppose I'll just list my changes manually.
Additionally, the change @ lines 124-131 doesn't really change behavior. It just made it a lot easier for me to debug when I was playing with some ideas.
Changed May 25, 2008 11:31AM UTC by comment:5
I've commited test in [5690]. The latest patch works much better, but I still have trouble applying it. I'll commit it once I figured it out.
Changed May 25, 2008 11:32AM UTC by comment:6
owner: | → joern |
---|
Changed May 25, 2008 03:11PM UTC by comment:7
I have no idea what's going on with these patches. I manually added the changes from Dane's patch and created a new patch (meta-messages.patch). Even though the patch was made with Unix line endings, Eclipse is still converting the entire file to Windows line-endings when applying the patch.
Right now, you should be able to apply my patch and get the changes. If Eclipse does switch the line endings when you apply, just use File -> Convert Line Delimiters To -> Unix. I'm really not sure what Eclipse's problem is right now. Perhaps we should consider setting eol-style: native for all source files (I've already done this for core and UI).
I didn't actually have time to review the patch, but from a quick look it seems fine.
Changed May 25, 2008 10:10PM UTC by comment:8
resolution: | → fixed |
---|---|
status: | new → closed |
Fixed in [5695], thanks Dane for the patch and Scott for the help.
Dane, can you please re-create the patch with Unix line-endings? The patch currently shows that every line has changed and I can't apply it to my working copy to review the changes. Thanks.