Skip to content

vendor: update spdx/tools-golang to d6f58551be3f#3450

Merged
jedevc merged 1 commit intomoby:masterfrom
jedevc:vendor-spdx-8e09d22f514a
Jan 5, 2023
Merged

vendor: update spdx/tools-golang to d6f58551be3f#3450
jedevc merged 1 commit intomoby:masterfrom
jedevc:vendor-spdx-8e09d22f514a

Conversation

@jedevc
Copy link
Copy Markdown
Member

@jedevc jedevc commented Jan 4, 2023

Followup to 9c7c061.

Removes the temporary replacement for jedevc/spdx-tools-golang, since the required commits have been merged upstream here.

See jedevc/spdx-tools-golang@json-tags...spdx:tools-golang:main for the full diff - these changes affect parsers that we don't use, so this should not result in any concrete changes.

CC @thaJeztah.

@jedevc jedevc requested review from crazy-max and tonistiigi January 4, 2023 10:17
@crazy-max
Copy link
Copy Markdown
Member

We can remove the replace directive then?

buildkit/go.mod

Line 156 in 617b78c

replace github.com/spdx/tools-golang => github.com/jedevc/spdx-tools-golang v0.0.0-20221205121515-6fe9d226281a

Followup to 9c7c061.

Remove the temporary replacement for jedevc/spdx-tools-golang, since the
required commits have been merged upstream.

Signed-off-by: Justin Chadwell <me@jedevc.com>
@jedevc jedevc force-pushed the vendor-spdx-8e09d22f514a branch from 6e281a9 to 3ea031c Compare January 4, 2023 16:15
@jedevc jedevc changed the title vendor: update spdx/tools-golang to 8e09d22f514a vendor: update spdx/tools-golang to d6f58551be3f Jan 5, 2023
@jedevc
Copy link
Copy Markdown
Member Author

jedevc commented Jan 5, 2023

Have updated the replace directive, should be good to go now.

@jedevc jedevc merged commit 8ccd3b3 into moby:master Jan 5, 2023
@jedevc jedevc deleted the vendor-spdx-8e09d22f514a branch January 5, 2023 11:31
@tonistiigi tonistiigi mentioned this pull request Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants