Merge pull request #4 from harshavardhana/pr_out_further_restructure

Makefile

@@ -13,36 +13,27 @@ getdeps: checkdeps
 build-utils:
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/cpu
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/unitconv
-
-build-crypto:
-	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/crypto/md5/
-	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/crypto/sha1/
-	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/crypto/sha256/
-	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/crypto/sha512/
-
-build-checksum:
-	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/checksum/crc32c
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/split
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/crypto/md5/
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/crypto/sha1/
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/crypto/sha256/
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/crypto/sha512/
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/utils/checksum/crc32c
 
 build-os:
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/os/scsi
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/os/sysctl
 
-build-fileutils: build-utils
-	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/fileutils/split
-
-build-storage: build-erasure build-storage-append build-storage-encoded
-
-build-erasure:
+build-storage:
 	@$(MAKE) $(MAKE_OPTIONS) -C pkg/storage/erasure/isal lib
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/storage/erasure
-
-build-storage-append:
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/storage/appendstorage
-
-build-storage-encoded:
 	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/storage/encodedstorage
 
-cover: build-fileutils build-checksum build-os build-storage build-crypto
+build-server:
+	@godep go test -race -coverprofile=cover.out github.com/minio-io/minio/pkg/server
+
+cover: build-storage build-os build-server build-utils
 
 install: cover
 
@@ -56,5 +47,7 @@ env:
 	@godep go env
 
 clean:
+	@echo "Cleaning up all the generated files"
 	@$(MAKE) $(MAKE_OPTIONS) -C pkg/storage/erasure/isal clean
-	@rm -v cover.out
+	@rm -fv pkg/utils/split/TESTPREFIX.*
+	@rm -fv cover.out

pkg/fileutils/split/Makefile

@@ -1,11 +0,0 @@
-all: build test
-.PHONY: all
-
-build:
-	@godep go build
-
-test: build
-	@godep go test -race -coverprofile=cover.out
-
-clean:
-	@rm -fv NEWFILE TESTPREFIX.* cover.out

pkg/server/server.go

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package gateway
+package server
 
 import (
 	"bytes"

pkg/server/server_test.go

@@ -1,4 +1,4 @@
-package gateway
+package server
 
 import (
 	"io/ioutil"

pkg/storage/appendstorage/append_storage.go

@@ -27,8 +27,8 @@ import (
 	"strconv"
 	"strings"
 
-	"github.com/minio-io/minio/pkg/checksum/crc32c"
 	"github.com/minio-io/minio/pkg/storage"
+	"github.com/minio-io/minio/pkg/utils/checksum/crc32c"
 )
 
 type appendStorage struct {

pkg/storage/encodedstorage/encoded_storage.go

@@ -29,10 +29,10 @@ import (
 	"strconv"
 	"strings"
 
-	"github.com/minio-io/minio/pkg/fileutils/split"
 	"github.com/minio-io/minio/pkg/storage"
 	"github.com/minio-io/minio/pkg/storage/appendstorage"
 	"github.com/minio-io/minio/pkg/storage/erasure"
+	"github.com/minio-io/minio/pkg/utils/split"
 	"github.com/spaolacci/murmur3"
 )
 

pkg/utils/split/split_test.go

@@ -52,9 +52,9 @@ func (s *MySuite) TestSplitStream(c *C) {
 }
 
 func (s *MySuite) TestFileSplitJoin(c *C) {
-	err := SplitFileWithPrefix("TESTFILE", 1024, "TESTPREFIX")
+	err := SplitFileWithPrefix("test-data/TESTFILE", 1024, "TESTPREFIX")
 	c.Assert(err, IsNil)
-	err = SplitFileWithPrefix("TESTFILE", 1024, "")
+	err = SplitFileWithPrefix("test-data/TESTFILE", 1024, "")
 	c.Assert(err, Not(IsNil))
 
 	devnull, err := os.OpenFile(os.DevNull, 2, os.ModeAppend)