Skip to content

Commit 3b6fa86

Browse files
committed
pkg/cdi: fix a 'go vet' failure on windows.
Signed-off-by: Krisztian Litkey <[email protected]>
1 parent 538cc1a commit 3b6fa86

File tree

3 files changed

+50
-3
lines changed

3 files changed

+50
-3
lines changed

pkg/cdi/cache_test.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ import (
2121
"path/filepath"
2222
"strings"
2323
"sync"
24-
"syscall"
2524
"testing"
2625
"time"
2726

@@ -770,10 +769,10 @@ devices:
770769
for {
771770
select {
772771
case _ = <-stopCh:
773-
go syscall.Sync()
772+
go osSync()
774773
return
775774
case _ = <-sync.C:
776-
go syscall.Sync()
775+
go osSync()
777776
sync.Reset(2 * time.Second)
778777
}
779778
}

pkg/cdi/cache_test_unix.go

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
//go:build !windows
2+
// +build !windows
3+
4+
/*
5+
Copyright © 2021 The CDI Authors
6+
7+
Licensed under the Apache License, Version 2.0 (the "License");
8+
you may not use this file except in compliance with the License.
9+
You may obtain a copy of the License at
10+
11+
http://www.apache.org/licenses/LICENSE-2.0
12+
13+
Unless required by applicable law or agreed to in writing, software
14+
distributed under the License is distributed on an "AS IS" BASIS,
15+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16+
See the License for the specific language governing permissions and
17+
limitations under the License.
18+
*/
19+
20+
package cdi
21+
22+
import "syscall"
23+
24+
func osSync() {
25+
syscall.Sync()
26+
}

pkg/cdi/cache_test_windows.go

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
//go:build windows
2+
// +build windows
3+
4+
/*
5+
Copyright © 2021 The CDI Authors
6+
7+
Licensed under the Apache License, Version 2.0 (the "License");
8+
you may not use this file except in compliance with the License.
9+
You may obtain a copy of the License at
10+
11+
http://www.apache.org/licenses/LICENSE-2.0
12+
13+
Unless required by applicable law or agreed to in writing, software
14+
distributed under the License is distributed on an "AS IS" BASIS,
15+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16+
See the License for the specific language governing permissions and
17+
limitations under the License.
18+
*/
19+
20+
package cdi
21+
22+
func osSync() {}

0 commit comments

Comments
 (0)