@carlosmn: has this been superseded?

A bunch of this made its way into the library, but we still don't have a git_remote_delete().

Whelp then. Paging doctor @nulltoken, care to rebase and cleanup this? :)

@vmg Rebased and cleaned up!

One feature is still not implemented though.

Two unrelated commits have been removed from this PR pushed into separate ones (#1963 and #1964)

Travis is not happy. :)

@carlosmn throw a couple eyes on this please ^^

Travis is not happy. :)

This is because of the missing feature

void test_network_remote_delete__deleting_a_remote_removes_the_remote_tracking_references(void)
{
    cl_assert(false);
}

The part that's implemented looks fine, though we should move it over to the config iterators instead of callbacks. We should also add tests to make sure we do whatever git does when the fetch refspec isn't the default one.

@nulltoken Are you planning to work on this? If not, I might look into cleaning this up. I want to improve my C-fu a bit and am looking for simpler things in the project that I could work on.

@arthurschreiber Go for it. AFAIR the failing tests were pointing at what was missing from the feature perspective.

Functionality added in #2313