[Bug 54573] rvdiffto parameter implementation

2014-03-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=54573

--- Comment #4 from Ricordisamoa  ---
I do not think we should go with this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 54573] rvdiffto parameter implementation

2013-09-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=54573

--- Comment #3 from Kunal Mehta (Legoktm)  ---
housekeeper's note:the patch does not apply cleanly to r10035

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 54573] rvdiffto parameter implementation

2013-09-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=54573

Kunal Mehta (Legoktm)  changed:

   What|Removed |Added

   See Also||https://sourceforge.net/p/p
   ||ywikipediabot/patches/447

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 54573] rvdiffto parameter implementation

2013-09-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=54573

--- Comment #2 from Kunal Mehta (Legoktm)  ---
Sorry for the slow slow slow slw response.

I think diffs are a useful thing to have, but I'm not quite sure what the goal
here is - what is the advantage of using rvdiffto instead of getting both
revisions and comparing them with a python diff function?

I can see the use case for, for instance, an antivandalism bot, but I'm not
quite sure how you would use it with this.

Then on the implementation - I can imagine it makes sense to store diffs for a
certain revision, but I'd expect, for instance, a dict with revid's such that

page, revid=10001
diffs = \{1: , 9000: \}, and storing e.g. revision.prev = 1.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 54573] rvdiffto parameter implementation

2013-09-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=54573

--- Comment #1 from Kunal Mehta (Legoktm)  ---
rvdiffto parameter implementation

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l