Page MenuHomeFreeBSD

Replace deprecated git repo-config
ClosedPublic

Authored by adridg on Mar 2 2017, 10:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 18, 11:38 PM
Unknown Object (File)
Feb 26 2025, 6:27 AM
Unknown Object (File)
Feb 26 2025, 2:04 AM
Unknown Object (File)
Feb 25 2025, 7:43 AM
Unknown Object (File)
Feb 24 2025, 9:57 PM
Unknown Object (File)
Jan 23 2025, 7:49 PM
Unknown Object (File)
Dec 4 2024, 9:18 PM
Unknown Object (File)
Nov 24 2024, 4:22 AM
Subscribers

Details

Summary

repo-config is no longer supported by any upstream-supported
git version. Instead, use git config (which is implicitly
git config --file, corresponding to the earlier config-in-repo).

Every time you start qgit, or refresh a repo, you get a popup
saying that repo-config is not recognized. This makes qgit
annoying to use at best. Unless you have a really old git
installed, the intent of the config option is broken.

Test Plan

Builds for me (amd64, 10.3). Runs, too, without the annoying popup.
Since I don't have any repo's that actually use commitEncoding,
I don't have much to user-test with -- but given that after the
popup it was falling back to utf-8 anyway, this can't be a
regression.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

adridg retitled this revision from to Replace deprecated git repo-config.
adridg updated this object.
adridg edited the test plan for this revision. (Show Details)
adridg added a reviewer: tcberner.

Maintainer of the port is listed as itetcu@, but that doesn't work in Phab -- adding some KDE peeps instead.

tcberner edited edge metadata.

There is also a commit for this in the 'official' github repo
https://github.com/tibirna/qgit/commit/6a4f1086114478e5f0be9929da92aad42123d988

As your patch however contains a descriptive comment, and it's not quite clear how 'official' the so called 'official' repository on github actually is, I think it's ok to just use your patch :)

This revision is now accepted and ready to land.Mar 2 2017, 12:30 PM
rakuco edited edge metadata.

lgtm. @tcberner: feel free to land this and close any PR that might exist about this (don't forget to MFH the change).

This revision was automatically updated to reflect the committed changes.