Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

transport: add git-send-pack support to local/ssh. #163

Merged
merged 6 commits into from
Dec 9, 2016
Merged

transport: add git-send-pack support to local/ssh. #163

merged 6 commits into from
Dec 9, 2016

Conversation

smola
Copy link
Collaborator

@smola smola commented Dec 6, 2016

No description provided.

if b[len(b)-1] == '\n' {
b = b[:len(b)-1]
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bytes.TrimSuffix?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

if b[len(b)-1] == '\n' {
b = b[:len(b)-1]
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bytes.TrimSuffix?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@smola smola requested a review from mcuadros December 8, 2016 08:52
@smola
Copy link
Collaborator Author

smola commented Dec 9, 2016

This PR fails on Travis because it uses git 1.8.x. See #170 for the fix and merge it before this one.

@smola smola merged commit cb30e88 into src-d:master Dec 9, 2016
@smola smola deleted the send-pack branch December 9, 2016 13:44
mcuadros pushed a commit that referenced this pull request Jan 31, 2017
* protocol/packp: add Packfile field to ReferenceUpdateRequest.
* protocol/packp: add NewReferenceUpdateRequestFromCapabilities.
* NewReferenceUpdateRequestFromCapabilities can be used to create
  a ReferenceUpdateRequest with initial capabilities compatible with
  the server.
* protocol/packp: fix new line handling on report status.
* transport/file: test error on unexisting command.
gsalingu-ovhus pushed a commit to gsalingu-ovhus/go-git that referenced this pull request Mar 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants