Skip to content

vendor: github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c - #5304

Merged
crazy-max merged 1 commit into
moby:masterfrom
thaJeztah:vendor_pkg_browser
Sep 5, 2024
Merged

vendor: github.com/pkg/browser v0.0.0-20240102092130-5ac0b6a4141c#5304
crazy-max merged 1 commit into
moby:masterfrom
thaJeztah:vendor_pkg_browser

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

It's an indirect dependency of docker/cl; https://github.com/docker/cli/blob/v27.2.0/vendor.mod#L83

full diff: pkg/browser@ce105d0...5ac0b6a

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah self-assigned this Sep 5, 2024
@github-actions github-actions Bot added area/windows area/dependencies Pull requests that update a dependency file labels Sep 5, 2024
@crazy-max

crazy-max commented Sep 5, 2024

Copy link
Copy Markdown
Member

https://github.com/docker/cli/blob/v27.2.0/vendor.mod#L83

don't see // indirect on this module, looks like it should be in the top require block?

@crazy-max

crazy-max commented Sep 5, 2024

Copy link
Copy Markdown
Member

@thompson-shaun Seems area/windows label is wrong, I think we should exclude vendor:

# area:windows
area/windows:
- changed-files:
- any-glob-to-any-file: "**/*_windows.go"

@thaJeztah

Copy link
Copy Markdown
Member Author

Interesting output; wondering if we should set things with a different format;

#28 [linux/amd64->darwin/arm64 releaser 4/4] RUN --mount=from=binaries   --mount=source=/tmp/.version,target=/tmp/.version,from=buildkit-version   mkdir -p /out && tar czvf "/out/buildkit-$(cat /tmp/.version).$(echo darwin/arm64 | sed 's/\//-/g').tar.gz" --mtime='2015-10-21 00:00Z' --sort=name --transform 's/^./bin/' .
#28 0.048 tar: Option --mtime: Treating date '2015-10-21 00:00Z' as 2015-10-21 00:00:00

@thaJeztah

Copy link
Copy Markdown
Member Author

don't see // indirect on this module, looks like it should be in the top require block?

Good catch! Yes it should. I really need to look for ways to catch those; ISTR someone has an action or linter to check for that.

@crazy-max

Copy link
Copy Markdown
Member

Interesting output; wondering if we should set things with a different format;

#28 [linux/amd64->darwin/arm64 releaser 4/4] RUN --mount=from=binaries   --mount=source=/tmp/.version,target=/tmp/.version,from=buildkit-version   mkdir -p /out && tar czvf "/out/buildkit-$(cat /tmp/.version).$(echo darwin/arm64 | sed 's/\//-/g').tar.gz" --mtime='2015-10-21 00:00Z' --sort=name --transform 's/^./bin/' .
#28 0.048 tar: Option --mtime: Treating date '2015-10-21 00:00Z' as 2015-10-21 00:00:00

Ah good catch, seems interpreted in UTC format but seconds are missing when converted. We might need to set TZ=UTC0 or just use 2015-10-21 00:00:00.

@crazy-max
crazy-max merged commit 3a9ce1d into moby:master Sep 5, 2024
@thaJeztah
thaJeztah deleted the vendor_pkg_browser branch September 5, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dependencies Pull requests that update a dependency file area/windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants