README.md

@@ -1,5 +1,7 @@
 #goadb
-[![GoDoc](https://pkg.go.dev/github.com/timoxa0/goadb?status.svg)](https://pkg.go.dev/github.com/timoxa0/goadb)
+
+[![Build Status](https://travis-ci.org/zach-klippenstein/goadb.svg?branch=master)](https://travis-ci.org/zach-klippenstein/goadb)
+[![GoDoc](https://godoc.org/goadb?status.svg)](https://godoc.org/goadb)
 
 A Golang library for interacting with the Android Debug Bridge (adb).
 

adb/adb.go

@@ -3,8 +3,8 @@ package adb
 import (
 	"strconv"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 /*

adb/adb_test.go

@@ -3,7 +3,7 @@ package adb
 import (
 	"testing"
 
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 )

adb/device.go

@@ -7,8 +7,8 @@ import (
 	"strings"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 // MtimeOfClose should be passed to OpenWrite to set the file modification time to the time the Close
@@ -80,7 +80,6 @@ func (c *Device) DeviceInfo() (*DeviceInfo, error) {
 RunCommand runs the specified commands on a shell on the device.
 
 From the Android docs:
-
 	Run 'command arg1 arg2 ...' in a shell on the device, and return
 	its output and error streams. Note that arguments must be separated
 	by spaces. If an argument contains a space, it must be quoted with
@@ -88,7 +87,6 @@ From the Android docs:
 	will go very wrong.
 
 	Note that this is the non-interactive version of "adb shell"
-
 Source: https://android.googlesource.com/platform/system/core/+/master/adb/SERVICES.TXT
 
 This method quotes the arguments for you, and will return an error if any of them
@@ -122,84 +120,13 @@ func (c *Device) RunCommand(cmd string, args ...string) (string, error) {
 	return string(resp), wrapClientError(err, c, "RunCommand")
 }
 
-/*
-Forward, from the official adb command's docs:
-
-		Asks the ADB server to forward local connections from <local>
-	  to the <remote> address on a given device.
-	  There, <host-prefix> can be one of the
-	  host-serial/host-usb/host-local/host prefixes as described previously
-	  and indicates which device/emulator to target.
-	  the format of <local> is one of:
-	      tcp:<port>      -> TCP connection on localhost:<port>
-	      local:<path>    -> Unix local domain socket on <path>
-	  the format of <remote> is one of:
-	      tcp:<port>      -> TCP localhost:<port> on device
-	      local:<path>    -> Unix local domain socket on device
-	      jdwp:<pid>      -> JDWP thread on VM process <pid>
-	      vsock:<CID>:<port> -> vsock on the given CID and port
-	  or even any one of the local services described below.
-
-Source: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/SERVICES.TXT
-*/
-func (c *Device) Forward(local string, remote string) error {
-	req := fmt.Sprintf("forward:%s;%s", local, remote)
-	_, err := c.getAttribute(req)
-	return wrapClientError(err, c, "Forward")
-}
-
-/*
-KillForward, from the official adb command's docs:
-
-	Remove any existing forward local connection from <local>.
-
-Source: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/SERVICES.TXT
-*/
-func (c *Device) KillForward(local string) error {
-	req := fmt.Sprintf("killforward:%s", local)
-	_, err := c.getAttribute(req)
-	return wrapClientError(err, c, "KillForward")
-}
-
-/*
-KillForwardAll, from the official adb command's docs:
-
-	Remove all forward network connections.
-
-Source: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/SERVICES.TXT
-*/
-func (c *Device) KillForwardAll() error {
-	req := "killforward-all"
-	_, err := c.getAttribute(req)
-	return wrapClientError(err, c, "KillForward")
-}
-
-/*
-ListForwards, from the official adb command's docs:
-
-	List all existing forward connections from this server.
-
-Source: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/SERVICES.TXT
-*/
-func (c *Device) ListForwards() ([]Forward, error) {
-	req := "list-forward"
-	resp, err := c.getAttribute(req)
-	if err != nil {
-		return nil, wrapClientError(err, c, "ListForwards")
-	}
-	forwards, err := parseForward(resp, c.descriptor.serial)
-	return forwards, wrapClientError(err, c, "ListForwards")
-}
-
 /*
 Remount, from the official adb command’s docs:
-
 	Ask adbd to remount the device's filesystem in read-write mode,
 	instead of read-only. This is usually necessary before performing
 	an "adb sync" or "adb push" request.
 	This request may not succeed on certain builds which do not allow
 	that.
-
 Source: https://android.googlesource.com/platform/system/core/+/master/adb/SERVICES.TXT
 */
 func (c *Device) Remount() (string, error) {

adb/device_forward.go

@@ -1,61 +0,0 @@
-package adb
-
-import (
-	"strings"
-)
-
-type ForwardType int8
-
-const (
-	TypeInvalid ForwardType = iota
-	ForwardTCP
-	ForwardLOCAL
-)
-
-var forwardTypeStrings = map[string]ForwardType{
-	"tcp":   ForwardTCP,
-	"local": ForwardLOCAL,
-}
-
-type ForwardTarget string
-
-type ForwardLocal struct {
-	FType   ForwardType
-	FTarget ForwardTarget
-}
-
-type ForwardRemote struct {
-	FType   ForwardType
-	FTarget ForwardTarget
-}
-
-type Forward struct {
-	Local  ForwardLocal
-	Remote ForwardRemote
-}
-
-func parseForward(str string, deviceSerial string) ([]Forward, error) {
-	var forwards []Forward
-	for _, forwardStr := range strings.Split(str, "\n") {
-		if strings.Trim(forwardStr, "\n\t ") == "" {
-			continue
-		}
-		raw_forward := strings.Split(forwardStr, " ")
-		serial, local, remote := raw_forward[0], raw_forward[1], raw_forward[2]
-		if serial == deviceSerial {
-			raw_local := strings.Split(local, ":")
-			raw_remote := strings.Split(remote, ":")
-			forwards = append(forwards, Forward{
-				Local: ForwardLocal{
-					FType:   forwardTypeStrings[raw_local[0]],
-					FTarget: ForwardTarget(raw_local[1]),
-				},
-				Remote: ForwardRemote{
-					FType:   forwardTypeStrings[raw_remote[0]],
-					FTarget: ForwardTarget(raw_remote[1]),
-				},
-			})
-		}
-	}
-	return forwards, nil
-}

adb/device_info.go

@@ -4,7 +4,7 @@ import (
 	"bufio"
 	"strings"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 type DeviceInfo struct {
@@ -87,8 +87,5 @@ func parseDeviceAttributes(fields []string) map[string]string {
 // Parses a key:val pair and returns key, val.
 func parseKeyVal(pair string) (string, string) {
 	split := strings.Split(pair, ":")
-	if len(split) == 1 {
-		return split[0], ""
-	}
 	return split[0], split[1]
 }

adb/device_state.go

@@ -1,13 +1,14 @@
 package adb
 
-import "github.com/timoxa0/goadb/internal/errors"
+import (
+	"github.com/evrins/goadb/internal/errors"
+)
 
 // DeviceState represents one of the 3 possible states adb will report devices.
 // A device can be communicated with when it's in StateOnline.
 // A USB device will make the following state transitions:
 //
 //	Plugged in: StateDisconnected->StateOffline->StateOnline
-//	 Plugged in (Recoveory): StateDisconnected->StateOnline
 //	Unplugged:  StateOnline->StateDisconnected
 //
 //go:generate stringer -type=DeviceState

adb/device_test.go

@@ -3,8 +3,8 @@ package adb
 import (
 	"testing"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 )

adb/device_watcher.go

@@ -8,8 +8,8 @@ import (
 	"sync/atomic"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 /*

adb/device_watcher_test.go

@@ -3,8 +3,8 @@ package adb
 import (
 	"testing"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 )

adb/dialer.go

@@ -5,8 +5,8 @@ import (
 	"net"
 	"runtime"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 // Dialer knows how to create connections to an adb server.

adb/dir_entries.go

@@ -5,7 +5,7 @@ import (
 	"os"
 	"time"
 
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/wire"
 )
 
 // DirEntry holds information about a directory entry on a device.

adb/error.go

@@ -1,6 +1,6 @@
 package adb
 
-import "github.com/timoxa0/goadb/internal/errors"
+import "github.com/evrins/goadb/internal/errors"
 
 type ErrCode errors.ErrCode
 

adb/forward_string.go

@@ -1,16 +0,0 @@
-// Code generated by "stringer -type=ForwardType"; DO NOT EDIT
-
-package adb
-
-import "fmt"
-
-const _ForwardType_name = "InvalidTCPLocal"
-
-var _ForwardType_index = [...]uint8{0, 7, 10, 15}
-
-func (i ForwardType) String() string {
-	if i < 0 || i >= ForwardType(len(_ForwardType_index)-1) {
-		return fmt.Sprintf("ForwardType(%d)", i)
-	}
-	return _ForwardType_name[_ForwardType_index[i]:_ForwardType_index[i+1]]
-}

adb/server.go

@@ -7,8 +7,8 @@ import (
 	"os/exec"
 	"strings"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 const (

adb/server_mock_test.go

@@ -4,8 +4,8 @@ import (
 	"io"
 	"strings"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 // MockServer implements Server, Scanner, and Sender.

adb/server_test.go

@@ -4,7 +4,7 @@ import (
 	"fmt"
 	"testing"
 
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 )

adb/sync_client.go

@@ -5,8 +5,8 @@ import (
 	"os"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 var zeroTime = time.Unix(0, 0).UTC()

adb/sync_client_test.go

@@ -7,8 +7,8 @@ import (
 	"testing"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"

adb/sync_file_reader.go

@@ -3,8 +3,8 @@ package adb
 import (
 	"io"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 // syncFileReader wraps a SyncConn that has requested to receive a file.

adb/sync_file_reader_test.go

@@ -6,7 +6,7 @@ import (
 	"strings"
 	"testing"
 
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 )

adb/sync_file_writer.go

@@ -6,8 +6,8 @@ import (
 	"os"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/internal/errors"
+	"github.com/evrins/goadb/wire"
 )
 
 // syncFileWriter wraps a SyncConn that has requested to send a file.

adb/sync_file_writer_test.go

@@ -8,7 +8,7 @@ import (
 	"encoding/binary"
 	"strings"
 
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/wire"
 
 	"github.com/stretchr/testify/assert"
 )

adb/util.go

@@ -6,7 +6,7 @@ import (
 	"regexp"
 	"strings"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 var (

cmd/adb/main.go

@@ -2,7 +2,7 @@ package main
 
 import (
 	"fmt"
-	"github.com/timoxa0/goadb/adb"
+	"github.com/evrins/goadb/adb"
 	"io"
 	"os"
 	"path/filepath"

cmd/demo/demo.go

@@ -4,12 +4,12 @@ package main
 import (
 	"flag"
 	"fmt"
-	"github.com/timoxa0/goadb/adb"
+	"github.com/evrins/goadb/adb"
 	"io/ioutil"
 	"log"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 var (

cmd/raw-adb/raw-adb.go

@@ -5,13 +5,13 @@ import (
 	"bufio"
 	"flag"
 	"fmt"
-	"github.com/timoxa0/goadb/adb"
+	"github.com/evrins/goadb/adb"
 	"io"
 	"log"
 	"os"
 	"strings"
 
-	"github.com/timoxa0/goadb/wire"
+	"github.com/evrins/goadb/wire"
 )
 
 var port = flag.Int("p", adb.AdbPort, "`port` the adb server is listening on")

go.mod

@@ -1,4 +1,4 @@
-module github.com/timoxa0/goadb
+module github.com/evrins/goadb
 
 go 1.14
 

wire/conn.go

@@ -1,6 +1,6 @@
 package wire
 
-import "github.com/timoxa0/goadb/internal/errors"
+import "github.com/evrins/goadb/internal/errors"
 
 const (
 	// The official implementation of adb imposes an undocumented 255-byte limit

wire/scanner.go

@@ -6,7 +6,7 @@ import (
 	"io/ioutil"
 	"strconv"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 // TODO(zach): All EOF errors returned from networoking calls should use ConnectionResetError.

wire/scanner_test.go

@@ -7,7 +7,7 @@ import (
 	"io/ioutil"
 	"testing"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 
 	"github.com/stretchr/testify/assert"
 )

wire/sender.go

@@ -4,7 +4,7 @@ import (
 	"fmt"
 	"io"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 // Sender sends messages to the server.

wire/sync_conn.go

@@ -1,6 +1,6 @@
 package wire
 
-import "github.com/timoxa0/goadb/internal/errors"
+import "github.com/evrins/goadb/internal/errors"
 
 const (
 	// Chunks cannot be longer than 64k.

wire/sync_scanner.go

@@ -6,7 +6,7 @@ import (
 	"os"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 type SyncScanner interface {

wire/sync_sender.go

@@ -6,7 +6,7 @@ import (
 	"os"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 type SyncSender interface {

wire/sync_test.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 
 	"github.com/stretchr/testify/assert"
 )

wire/util.go

@@ -6,7 +6,7 @@ import (
 	"regexp"
 	"sync"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 )
 
 // ErrorResponseDetails is an error message returned by the server for a particular request.

wire/util_test.go

@@ -3,7 +3,7 @@ package wire
 import (
 	"testing"
 
-	"github.com/timoxa0/goadb/internal/errors"
+	"github.com/evrins/goadb/internal/errors"
 
 	"github.com/stretchr/testify/assert"
 )