Page MenuHomePhabricator

Add a thing to download a unified diff
Closed, DeclinedPublic

Description

It needs a thing to download a unified diff. At least I think it does; I'm not sure what a unified diff is, but someone said that's what the thing I wanted was called so here's a bug for that.


Version: unspecified
Severity: enhancement

Details

Reference
bz55415

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:12 AM
bzimport set Reference to bz55415.
bzimport added a subscriber: Unknown Object (MLST).

Do you mean CodeReview or Gerrit?

If Gerrit, this is a dupe of bug 38100.
If CodeReview: it's always had unified diffs

As a workaround, you can produce any diff from SVN on the command-line with a local checkout, something like this:

svn diff -r12345:12346 > mypatch.diff

CodeReview extension is no longer maintained (it was purpose-built for MediaWiki's SVN and we've since moved to Git using Gerrit as a front-end) so unless someone plans to take it over for someone else's use this isn't likely to be added.

Closing as WONTFIX; if someone is interested in maintaining please feel free to reopen and add such a feature.

As you may know, some other projects do still use this extension.

Jack Phoenix requested I file this bug because it came up on another such project. As much as he'd like to maintain it properly, however, as it is an extension deployed on a wikimedia wiki he hasn't gotten very far with trying to get proper access.

I would like him to fix this, though, and having the bug here would be helpful to that end.

(In reply to comment #3)

Jack Phoenix requested I file this bug because it came up on another such
project. As much as he'd like to maintain it properly, however, as it is an
extension deployed on a wikimedia wiki he hasn't gotten very far with trying
to
get proper access.

What on earth do you mean? I've seen no requests on mw.org for ownership of the extension, nor do I see any outstanding patches.

If someone's caring about maintaining this extension, they've not made it clear.

Jack's more than welcome to take over this.