Remove all deprecated declarations

Now that we are introducing breaking API changes, let's get rid of all
of the deprecated names that we've been accruing over the years.

Author: lhchavez

cherrypick.go

@@ -9,15 +9,13 @@ import (
 )
 
 type CherrypickOptions struct {
-	Version         uint
 	Mainline        uint
 	MergeOptions    MergeOptions
 	CheckoutOptions CheckoutOptions
 }
 
 func cherrypickOptionsFromC(c *C.git_cherrypick_options) CherrypickOptions {
 	opts := CherrypickOptions{
-		Version:         uint(c.version),
 		Mainline:        uint(c.mainline),
 		MergeOptions:    mergeOptionsFromC(&c.merge_opts),
 		CheckoutOptions: checkoutOptionsFromC(&c.checkout_opts),

deprecated.go

@@ -1,271 +1,4 @@
 package git
 
-/*
-#include <git2.h>
-*/
-import "C"
-import (
-	"unsafe"
-)
-
 // The constants, functions, and types in this files are slated for deprecation
 // in the next major version.
-
-// blob.go
-
-// Deprecated: BlobChunkCallback is not used.
-type BlobChunkCallback func(maxLen int) ([]byte, error)
-
-// Deprecated: BlobCallbackData is not used.
-type BlobCallbackData struct {
-	Callback BlobChunkCallback
-	Error    error
-}
-
-// checkout.go
-
-// Deprecated: CheckoutOpts is a deprecated alias of CheckoutOptions.
-type CheckoutOpts = CheckoutOptions
-
-// credentials.go
-
-// Deprecated: CredType is a deprecated alias of CredentialType
-type CredType = CredentialType
-
-const (
-	CredTypeUserpassPlaintext = CredentialTypeUserpassPlaintext
-	CredTypeSshKey            = CredentialTypeSSHKey
-	CredTypeSshCustom         = CredentialTypeSSHCustom
-	CredTypeDefault           = CredentialTypeDefault
-)
-
-// Deprecated: Cred is a deprecated alias of Credential
-type Cred = Credential
-
-// Deprecated: NewCredUsername is a deprecated alias of NewCredentialUsername.
-func NewCredUsername(username string) (*Cred, error) {
-	return NewCredentialUsername(username)
-}
-
-// Deprecated: NewCredUserpassPlaintext is a deprecated alias of NewCredentialUserpassPlaintext.
-func NewCredUserpassPlaintext(username string, password string) (*Cred, error) {
-	return NewCredentialUserpassPlaintext(username, password)
-}
-
-// Deprecated: NewCredSshKey is a deprecated alias of NewCredentialSshKey.
-func NewCredSshKey(username string, publicKeyPath string, privateKeyPath string, passphrase string) (*Cred, error) {
-	return NewCredentialSSHKey(username, publicKeyPath, privateKeyPath, passphrase)
-}
-
-// Deprecated: NewCredSshKeyFromMemory is a deprecated alias of NewCredentialSSHKeyFromMemory.
-func NewCredSshKeyFromMemory(username string, publicKey string, privateKey string, passphrase string) (*Cred, error) {
-	return NewCredentialSSHKeyFromMemory(username, publicKey, privateKey, passphrase)
-}
-
-// Deprecated: NewCredSshKeyFromAgent is a deprecated alias of NewCredentialSSHFromAgent.
-func NewCredSshKeyFromAgent(username string) (*Cred, error) {
-	return NewCredentialSSHKeyFromAgent(username)
-}
-
-// Deprecated: NewCredDefault is a deprecated alias fof NewCredentialDefault.
-func NewCredDefault() (*Cred, error) {
-	return NewCredentialDefault()
-}
-
-// diff.go
-
-const (
-	// Deprecated: DiffIgnoreWhitespaceEol is a deprecated alias of DiffIgnoreWhitespaceEOL.
-	DiffIgnoreWitespaceEol = DiffIgnoreWhitespaceEOL
-)
-
-// features.go
-
-const (
-	// Deprecated: FeatureHttps is a deprecated alias of FeatureHTTPS.
-	FeatureHttps = FeatureHTTPS
-
-	// Deprecated: FeatureSsh is a deprecated alias of FeatureSSH.
-	FeatureSsh = FeatureSSH
-)
-
-// git.go
-
-const (
-	// Deprecated: ErrClassNone is a deprecated alias of ErrorClassNone.
-	ErrClassNone = ErrorClassNone
-	// Deprecated: ErrClassNoMemory is a deprecated alias of ErrorClassNoMemory.
-	ErrClassNoMemory = ErrorClassNoMemory
-	// Deprecated: ErrClassOs is a deprecated alias of ErrorClassOS.
-	ErrClassOs = ErrorClassOS
-	// Deprecated: ErrClassInvalid is a deprecated alias of ErrorClassInvalid.
-	ErrClassInvalid = ErrorClassInvalid
-	// Deprecated: ErrClassReference is a deprecated alias of ErrorClassReference.
-	ErrClassReference = ErrorClassReference
-	// Deprecated: ErrClassZlib is a deprecated alias of ErrorClassZlib.
-	ErrClassZlib = ErrorClassZlib
-	// Deprecated: ErrClassRepository is a deprecated alias of ErrorClassRepository.
-	ErrClassRepository = ErrorClassRepository
-	// Deprecated: ErrClassConfig is a deprecated alias of ErrorClassConfig.
-	ErrClassConfig = ErrorClassConfig
-	// Deprecated: ErrClassRegex is a deprecated alias of ErrorClassRegex.
-	ErrClassRegex = ErrorClassRegex
-	// Deprecated: ErrClassOdb is a deprecated alias of ErrorClassOdb.
-	ErrClassOdb = ErrorClassOdb
-	// Deprecated: ErrClassIndex is a deprecated alias of ErrorClassIndex.
-	ErrClassIndex = ErrorClassIndex
-	// Deprecated: ErrClassObject is a deprecated alias of ErrorClassObject.
-	ErrClassObject = ErrorClassObject
-	// Deprecated: ErrClassNet is a deprecated alias of ErrorClassNet.
-	ErrClassNet = ErrorClassNet
-	// Deprecated: ErrClassTag is a deprecated alias of ErrorClassTag.
-	ErrClassTag = ErrorClassTag
-	// Deprecated: ErrClassTree is a deprecated alias of ErrorClassTree.
-	ErrClassTree = ErrorClassTree
-	// Deprecated: ErrClassIndexer is a deprecated alias of ErrorClassIndexer.
-	ErrClassIndexer = ErrorClassIndexer
-	// Deprecated: ErrClassSSL is a deprecated alias of ErrorClassSSL.
-	ErrClassSSL = ErrorClassSSL
-	// Deprecated: ErrClassSubmodule is a deprecated alias of ErrorClassSubmodule.
-	ErrClassSubmodule = ErrorClassSubmodule
-	// Deprecated: ErrClassThread is a deprecated alias of ErrorClassThread.
-	ErrClassThread = ErrorClassThread
-	// Deprecated: ErrClassStash is a deprecated alias of ErrorClassStash.
-	ErrClassStash = ErrorClassStash
-	// Deprecated: ErrClassCheckout is a deprecated alias of ErrorClassCheckout.
-	ErrClassCheckout = ErrorClassCheckout
-	// Deprecated: ErrClassFetchHead is a deprecated alias of ErrorClassFetchHead.
-	ErrClassFetchHead = ErrorClassFetchHead
-	// Deprecated: ErrClassMerge is a deprecated alias of ErrorClassMerge.
-	ErrClassMerge = ErrorClassMerge
-	// Deprecated: ErrClassSsh is a deprecated alias of ErrorClassSSH.
-	ErrClassSsh = ErrorClassSSH
-	// Deprecated: ErrClassFilter is a deprecated alias of ErrorClassFilter.
-	ErrClassFilter = ErrorClassFilter
-	// Deprecated: ErrClassRevert is a deprecated alias of ErrorClassRevert.
-	ErrClassRevert = ErrorClassRevert
-	// Deprecated: ErrClassCallback is a deprecated alias of ErrorClassCallback.
-	ErrClassCallback = ErrorClassCallback
-	// Deprecated: ErrClassRebase is a deprecated alias of ErrorClassRebase.
-	ErrClassRebase = ErrorClassRebase
-	// Deprecated: ErrClassPatch is a deprecated alias of ErrorClassPatch.
-	ErrClassPatch = ErrorClassPatch
-)
-
-const (
-	// Deprecated: ErrOk is a deprecated alias of ErrorCodeOK.
-	ErrOk = ErrorCodeOK
-	// Deprecated: ErrGeneric is a deprecated alias of ErrorCodeGeneric.
-	ErrGeneric = ErrorCodeGeneric
-	// Deprecated: ErrNotFound is a deprecated alias of ErrorCodeNotFound.
-	ErrNotFound = ErrorCodeNotFound
-	// Deprecated: ErrExists is a deprecated alias of ErrorCodeExists.
-	ErrExists = ErrorCodeExists
-	// Deprecated: ErrAmbiguous is a deprecated alias of ErrorCodeAmbiguous.
-	ErrAmbiguous = ErrorCodeAmbiguous
-	// Deprecated: ErrAmbigious is a deprecated alias of ErrorCodeAmbiguous.
-	ErrAmbigious = ErrorCodeAmbiguous
-	// Deprecated: ErrBuffs is a deprecated alias of ErrorCodeBuffs.
-	ErrBuffs = ErrorCodeBuffs
-	// Deprecated: ErrUser is a deprecated alias of ErrorCodeUser.
-	ErrUser = ErrorCodeUser
-	// Deprecated: ErrBareRepo is a deprecated alias of ErrorCodeBareRepo.
-	ErrBareRepo = ErrorCodeBareRepo
-	// Deprecated: ErrUnbornBranch is a deprecated alias of ErrorCodeUnbornBranch.
-	ErrUnbornBranch = ErrorCodeUnbornBranch
-	// Deprecated: ErrUnmerged is a deprecated alias of ErrorCodeUnmerged.
-	ErrUnmerged = ErrorCodeUnmerged
-	// Deprecated: ErrNonFastForward is a deprecated alias of ErrorCodeNonFastForward.
-	ErrNonFastForward = ErrorCodeNonFastForward
-	// Deprecated: ErrInvalidSpec is a deprecated alias of ErrorCodeInvalidSpec.
-	ErrInvalidSpec = ErrorCodeInvalidSpec
-	// Deprecated: ErrConflict is a deprecated alias of ErrorCodeConflict.
-	ErrConflict = ErrorCodeConflict
-	// Deprecated: ErrLocked is a deprecated alias of ErrorCodeLocked.
-	ErrLocked = ErrorCodeLocked
-	// Deprecated: ErrModified is a deprecated alias of ErrorCodeModified.
-	ErrModified = ErrorCodeModified
-	// Deprecated: ErrAuth is a deprecated alias of ErrorCodeAuth.
-	ErrAuth = ErrorCodeAuth
-	// Deprecated: ErrCertificate is a deprecated alias of ErrorCodeCertificate.
-	ErrCertificate = ErrorCodeCertificate
-	// Deprecated: ErrApplied is a deprecated alias of ErrorCodeApplied.
-	ErrApplied = ErrorCodeApplied
-	// Deprecated: ErrPeel is a deprecated alias of ErrorCodePeel.
-	ErrPeel = ErrorCodePeel
-	// Deprecated: ErrEOF is a deprecated alias of ErrorCodeEOF.
-	ErrEOF = ErrorCodeEOF
-	// Deprecated: ErrUncommitted is a deprecated alias of ErrorCodeUncommitted.
-	ErrUncommitted = ErrorCodeUncommitted
-	// Deprecated: ErrDirectory is a deprecated alias of ErrorCodeDirectory.
-	ErrDirectory = ErrorCodeDirectory
-	// Deprecated: ErrMergeConflict is a deprecated alias of ErrorCodeMergeConflict.
-	ErrMergeConflict = ErrorCodeMergeConflict
-	// Deprecated: ErrPassthrough is a deprecated alias of ErrorCodePassthrough.
-	ErrPassthrough = ErrorCodePassthrough
-	// Deprecated: ErrIterOver is a deprecated alias of ErrorCodeIterOver.
-	ErrIterOver = ErrorCodeIterOver
-	// Deprecated: ErrApplyFail is a deprecated alias of ErrorCodeApplyFail.
-	ErrApplyFail = ErrorCodeApplyFail
-)
-
-// index.go
-
-// Deprecated: IndexAddOpts is a deprecated alias of IndexAddOption.
-type IndexAddOpts = IndexAddOption
-
-// Deprecated: IndexStageOpts is a deprecated alias of IndexStageState.
-type IndexStageOpts = IndexStageState
-
-// submodule.go
-
-// Deprecated: SubmoduleCbk is a deprecated alias of SubmoduleCallback.
-type SubmoduleCbk = SubmoduleCallback
-
-// Deprecated: SubmoduleVisitor is not used.
-func SubmoduleVisitor(csub unsafe.Pointer, name *C.char, handle unsafe.Pointer) C.int {
-	sub := &Submodule{(*C.git_submodule)(csub), nil}
-
-	callback, ok := pointerHandles.Get(handle).(SubmoduleCallback)
-	if !ok {
-		panic("invalid submodule visitor callback")
-	}
-	err := callback(sub, C.GoString(name))
-	if err != nil {
-		return C.int(ErrorCodeUser)
-	}
-	return C.int(ErrorCodeOK)
-}
-
-// reference.go
-
-// Deprecated: ReferenceIsValidName is a deprecated alias of ReferenceNameIsValid.
-func ReferenceIsValidName(name string) bool {
-	valid, _ := ReferenceNameIsValid(name)
-	return valid
-}
-
-// remote.go
-
-// Deprecated: RemoteIsValidName is a deprecated alias of RemoteNameIsValid.
-func RemoteIsValidName(name string) bool {
-	valid, _ := RemoteNameIsValid(name)
-	return valid
-}
-
-// tree.go
-
-// Deprecated: CallbackGitTreeWalk is not used.
-func CallbackGitTreeWalk(_root *C.char, entry *C.git_tree_entry, ptr unsafe.Pointer) C.int {
-	root := C.GoString(_root)
-
-	callback, ok := pointerHandles.Get(ptr).(TreeWalkCallback)
-	if !ok {
-		panic("invalid treewalk callback")
-	}
-	err := callback(root, newTreeEntry(entry))
-	if err != nil {
-		return C.int(ErrorCodeUser)
-	}
-	return C.int(ErrorCodeOK)
-}

git.go

@@ -206,7 +206,7 @@ func NewOid(s string) (*Oid, error) {
 	}
 
 	if len(slice) != 20 {
-		return nil, &GitError{"Invalid Oid", ErrorClassNone, ErrGeneric}
+		return nil, &GitError{"invalid oid", ErrorClassNone, ErrorCodeGeneric}
 	}
 
 	copy(o[:], slice[:20])

merge.go

@@ -137,7 +137,6 @@ const (
 )
 
 type MergeOptions struct {
-	Version   uint
 	TreeFlags MergeTreeFlag
 
 	RenameThreshold uint
@@ -150,7 +149,6 @@ type MergeOptions struct {
 
 func mergeOptionsFromC(opts *C.git_merge_options) MergeOptions {
 	return MergeOptions{
-		Version:         uint(opts.version),
 		TreeFlags:       MergeTreeFlag(opts.flags),
 		RenameThreshold: uint(opts.rename_threshold),
 		TargetLimit:     uint(opts.target_limit),

rebase.go

@@ -183,7 +183,6 @@ func commitCreateCallback(
 
 // RebaseOptions are used to tell the rebase machinery how to operate.
 type RebaseOptions struct {
-	Version         uint
 	Quiet           int
 	InMemory        int
 	RewriteNotesRef string
@@ -226,7 +225,6 @@ func DefaultRebaseOptions() (RebaseOptions, error) {
 
 func rebaseOptionsFromC(opts *C.git_rebase_options) RebaseOptions {
 	return RebaseOptions{
-		Version:         uint(opts.version),
 		Quiet:           int(opts.quiet),
 		InMemory:        int(opts.inmemory),
 		RewriteNotesRef: C.GoString(opts.rewrite_notes_ref),