Merge pull request #84 from otiai10/feature/fix-78

Skip device files by default: Use opt.Specials

Author: otiai10

all_test.go

@@ -197,6 +197,13 @@ func TestOptions_Skip(t *testing.T) {
 	})
 }
 
+func TestOptions_Specials(t *testing.T) {
+	if runtime.GOOS == "linux" || runtime.GOOS == "darwin" {
+		err := Copy("/dev/null", "test/data.copy/dev-null", Options{Specials: false})
+		Expect(t, err).ToBe(nil)
+	}
+}
+
 func TestOptions_PermissionControl(t *testing.T) {
 	info, err := os.Stat("test/data/case07/dir_0555")
 	Expect(t, err).ToBe(nil)

copy.go

@@ -26,6 +26,11 @@ func Copy(src, dest string, opt ...Options) error {
 // switchboard switches proper copy functions regarding file type, etc...
 // If there would be anything else here, add a case to this switchboard.
 func switchboard(src, dest string, info os.FileInfo, opt Options) (err error) {
+
+	if info.Mode()&os.ModeDevice != 0 && !opt.Specials {
+		return err
+	}
+
 	switch {
 	case info.Mode()&os.ModeSymlink != 0:
 		err = onsymlink(src, dest, opt)

options.go

@@ -17,6 +17,9 @@ type Options struct {
 	// Skip can specify which files should be skipped
 	Skip func(src string) (bool, error)
 
+	// Specials includes special files to be copied. default false.
+	Specials bool
+
 	// AddPermission to every entities,
 	// NO MORE THAN 0777
 	// @OBSOLETE
@@ -98,6 +101,7 @@ func getDefaultOptions(src, dest string) Options {
 		AddPermission:     0,                  // Add nothing
 		PermissionControl: PerservePermission, // Just preserve permission
 		Sync:              false,              // Do not sync
+		Specials:          false,              // Do not copy special files
 		PreserveTimes:     false,              // Do not preserve the modification time
 		CopyBufferSize:    0,                  // Do not specify, use default bufsize (32*1024)
 		WrapReader:        nil,                // Do not wrap src files, use them as they are.