0ct0pu5/moby
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity
39986 commits 14 branches 412 tags 338 MiB
Commit graph

38 commits

Author SHA1 Message Date
Eng Zer Jun
a916414b0b refactor: move from io/ioutil to io and os package 
The io/ioutil package has been deprecated in Go 1.16. This commit
replaces the existing io/ioutil functions with their new definitions in
io and os packages.

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
(cherry picked from commit c55a4ac779)
Signed-off-by: Cory Snider <csnider@mirantis.com>
 2023-02-24 16:11:55 -05:00
Sebastiaan van Stijn
a1150245cc
Update to Go 1.17.0, and gofmt with Go 1.17 
Movified from 686be57d0a, and re-ran
gofmt again to address for files not present in 20.10 and vice-versa.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 686be57d0a)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2022-04-07 23:27:50 +02:00
Sebastiaan van Stijn
6793ff26d8
pkg/fileutils: TestMatches: remove cases no longer valid for go1.16 
These tests were no longer valid on Go 1.16; related to https://tip.golang.org/doc/go1.16#path/filepath

> The Match and Glob functions now return an error if the unmatched part of
> the pattern has a syntax error. Previously, the functions returned early on
> a failed match, and thus did not report any later syntax error in the pattern.

Causing the test to fail:

    === RUN   TestMatches
        fileutils_test.go:388: assertion failed: error is not nil: syntax error in pattern: pattern="a\\" text="a"
    --- FAIL: TestMatches (0.00s)

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 2842639e0e)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2021-07-15 22:24:50 +02:00
Sebastiaan van Stijn
9f0b3f5609
bump gotest.tools v3.0.1 for compatibility with Go 1.14 
full diff: https://github.com/gotestyourself/gotest.tools/compare/v2.3.0...v3.0.1

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2020-02-11 00:06:42 +01:00
Sebastiaan van Stijn
9419024554
gosec: add ignore comments for reported issues that can be ignored 
```
builder/remotecontext/remote.go:48:        G107: Potential HTTP request made with variable url (gosec)
builder/remotecontext/git/gitutils.go:145: G107: Potential HTTP request made with variable url (gosec)
builder/remotecontext/git/gitutils.go:147: G107: Potential HTTP request made with variable url (gosec)
pkg/fileutils/fileutils_test.go:185:       G303: File creation in shared tmp directory without using ioutil.Tempfile (gosec)
pkg/tarsum/tarsum_test.go:7:               G501: Blacklisted import `crypto/md5`: weak cryptographic primitive (gosec)
pkg/tarsum/tarsum_test.go:9:               G505: Blacklisted import `crypto/sha1`: weak cryptographic primitive (gosec)
```

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2019-09-18 12:57:43 +02:00
mooncake
35739fd290 Remove duplicated words in pkg files 
Signed-off-by: mooncake <xcoder@tenxcloud.com>
 2018-10-05 22:32:14 +08:00
Vincent Demeester
3845728524
Update tests to use gotest.tools 👼 
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
 2018-06-13 09:04:30 +02:00
chenyuzhu
5ac0d2f0b8 Fix typos 
Signed-off-by: chenyuzhu <chenyuzhi@oschina.cn>
 2018-05-16 15:31:12 +08:00
Daniel Nephin
6be0f70983 Automated migration using 
gty-migrate-from-testify --ignore-build-tags

Signed-off-by: Daniel Nephin <dnephin@docker.com>
 2018-03-16 11:03:43 -04:00
Daniel Nephin
4f0d95fa6e Add canonical import comment 
Signed-off-by: Daniel Nephin <dnephin@docker.com>
 2018-02-05 16:51:57 -05:00
Michael Crosby
5a9b5f10cf Remove solaris files 
For obvious reasons that it is not really supported now.

Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
 2017-10-24 15:39:34 -04:00
Daniel Nephin
372670b507 Add goimports to linters. 
Signed-off-by: Daniel Nephin <dnephin@docker.com>
 2017-08-21 18:15:08 -04:00
Derek McGowan
1009e6a40b
Update logrus to v1.0.1 
Fixes case sensitivity issue

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
 2017-07-31 13:16:46 -07:00
Daniel Nephin
9484c3bd81 Enable a unit test on windows. 
Signed-off-by: Daniel Nephin <dnephin@docker.com>
 2017-04-21 12:31:16 -04:00
unclejack
20d6f5c2a9 pkg/file{utils,notify}: don't compare to bool 
Signed-off-by: Cristian Staretu <cristian.staretu@gmail.com>
 2017-03-30 16:38:01 +03:00
Tonis Tiigi
adeb3e3684 Fix inefficient file paths filter 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
 2017-03-08 18:59:25 -08:00
Doug Davis
376bb84c5c Fix use of **/ in .dockerignore 
.dockerignore pattern of **/.foo incorrectly matched **/bar.foo
because **/.foo was getting converted into a .*\.foo regex
instead of (.*/)*\.foo

Closes #29014

Signed-off-by: Doug Davis <dug@us.ibm.com>
 2016-12-02 03:58:54 -08:00
Elena Morozova
a37027f6bd all: use strings.Contains instead Index 
Signed-off-by: Elena Morozova <lelenanam@gmail.com>
 2016-10-13 15:55:41 -07:00
allencloud
4e959ef2f7 fix typos 
Signed-off-by: allencloud <allen.sun@daocloud.io>
 2016-07-23 11:32:23 +08:00
allencloud
d2dd1a1f7d add fileutils_darwin.go in pkg/fileutils to support darwin platform 
Signed-off-by: allencloud <allen.sun@daocloud.io>
 2016-07-01 13:02:26 +08:00
Amit Krishnan
86d8758e2b Get the Docker Engine to build clean on Solaris 
Signed-off-by: Amit Krishnan <krish.amit@gmail.com>
 2016-05-23 16:37:12 -07:00
allencloud
3f77baf2ce fix typos 
Signed-off-by: allencloud <allen.sun@daocloud.io>
 2016-05-06 18:23:11 +08:00
John Howard
3e78ad7be2 Windows CI: Turn off failing unit test pkg\fileutils 
Signed-off-by: John Howard <jhoward@microsoft.com>
 2016-03-02 19:05:33 -08:00
John Howard
b368a9f9b7 Windows CI: Fixes panic in test-unit for FileUtils 
Signed-off-by: John Howard <jhoward@microsoft.com>
 2016-02-23 13:31:45 -08:00
Justas Brazauskas
927b334ebf Fix typos found across repository 
Signed-off-by: Justas Brazauskas <brazauskasjustas@gmail.com>
 2015-12-13 18:04:12 +02:00
Doug Davis
eddb14a44e Support multi-dir wildcards in .dockerignore 
Closes #13113

Signed-off-by: Doug Davis <dug@us.ibm.com>
 2015-11-13 13:44:07 -08:00
John Howard
8d56108ffb Windows: Fix warning on info 
Signed-off-by: John Howard <jhoward@microsoft.com>
 2015-09-18 13:39:12 -07:00
Vincent Demeester
09adf87f23 Add missing tests and docs for pkg/fileutils 
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
 2015-07-12 22:43:42 +02:00
Lei Jitang
289ee90b04 Fix copy from a "created" container. Fixes #14420 
Signed-off-by: Lei Jitang <leijitang@huawei.com>
 2015-07-08 11:15:09 +08:00
unclejack
2f6e4fdb29 linting changes 
Signed-off-by: Cristian Staretu <cristian.staretu@gmail.com>
 2015-06-17 01:16:57 +03:00
Doug Davis
82ea6ed2bc Allow .dockerignore to ignore everything 
Change CLI error msg because it was too specific and didn't make sense
when there were errors not related to inaccessible files.

Removed some log.Error() calls since they're not really errors we should
log. Returning the error will be enough.

Closes: #13417

Signed-off-by: Doug Davis <dug@us.ibm.com>
 2015-05-30 04:16:00 -07:00
Vincent Demeester
8454e1a3b2 Add coverage on pkg/fileutils 
Should fix #11598

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
 2015-04-29 16:27:12 +02:00
buddhamagnet
6fd8e485c8 add support for exclusion rules in dockerignore 
Signed-off-by: Dave Goodchild <buddhamagnet@gmail.com>
 2015-04-28 18:56:45 +01:00
Antonio Murdaca
c30a55f14d Refactor utils/utils, fixes #11923 
Signed-off-by: Antonio Murdaca <me@runcom.ninja>
 2015-04-14 01:37:36 +02:00
Antonio Murdaca
01724c1cf1 Refactor ultis/utils_daemon, fixes #11908 
Signed-off-by: Antonio Murdaca <me@runcom.ninja>
 2015-04-03 20:20:04 +02:00
Antonio Murdaca
6f4d847046 Replace aliased imports of logrus, fixes #11762 
Signed-off-by: Antonio Murdaca <me@runcom.ninja>
 2015-03-26 23:22:04 +01:00
Alexandr Morozov
7c62cee51e Use logrus everywhere for logging 
Fixed #8761

Signed-off-by: Alexandr Morozov <lk4d4@docker.com>
 2014-10-24 15:03:06 -07:00
Rafe Colton
73f4bfed81 Move Matches() file path matching function into pkg/fileutils 
This is the second of two steps to break the archive package's
dependence on utils so that archive may be moved into pkg. `Matches()`
is also a good candidate pkg in that it is small, concise, and not
specific to docker internals

Signed-off-by: Rafe Colton <rafael.colton@gmail.com>
 2014-09-29 23:21:41 -07:00