Skip to content

add main module field to go bin metadata#1026

Merged
luhring merged 3 commits intoanchore:mainfrom
jonasagx:specify-main-module-for-go-bin-metadata
Jun 3, 2022
Merged

add main module field to go bin metadata#1026
luhring merged 3 commits intoanchore:mainfrom
jonasagx:specify-main-module-for-go-bin-metadata

Conversation

@jonasagx
Copy link
Copy Markdown
Contributor

@jonasagx jonasagx commented Jun 3, 2022

This PR adds mainModule to Go binary metadata implementing Dan's suggestion: all golang packages will get a mainModule, including the main module itself (where name == metadata.mainModule).

JSON output schema was updated to 3.2.4. Minor version bump because of new field added to golang's metadata.

Closes: #908

Signed-off-by: Jonas Xavier jonasx@anchore.com

Signed-off-by: Jonas Xavier <jonasx@anchore.com>
@jonasagx jonasagx marked this pull request as draft June 3, 2022 17:54
@jonasagx jonasagx marked this pull request as ready for review June 3, 2022 18:52
Copy link
Copy Markdown
Contributor

@luhring luhring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty good, but also this is a schema change for our output, so we need to update that.

jonasagx added 2 commits June 3, 2022 15:50
Signed-off-by: Jonas Xavier <jonasx@anchore.com>
Signed-off-by: Jonas Xavier <jonasx@anchore.com>
@jonasagx jonasagx requested a review from luhring June 3, 2022 22:59
Copy link
Copy Markdown
Contributor

@luhring luhring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@luhring luhring enabled auto-merge (squash) June 3, 2022 23:09
@luhring luhring merged commit 0aea55f into anchore:main Jun 3, 2022
@jonasagx jonasagx deleted the specify-main-module-for-go-bin-metadata branch June 3, 2022 23:25
spiffcs added a commit to jonasagx/syft that referenced this pull request Jun 27, 2022
* main: (70 commits)
  fix: add php catalogers to all catalogers (anchore#1065)
  feat: add use-all-catalogers flag (anchore#1050)
  Updates parsing of `yarn.lock` to use `resolved` URLs that are pulled from yarn and npm registries (anchore#926)
  remove OSS Meetup message (anchore#1057)
  add pom.xml cataloger (anchore#1055)
  Add support for CBL-Mariner distroless images (anchore#1045)
  Add catalogers configuration (anchore#1038)
  add template output (anchore#1051)
  update stereoscope to latest version (anchore#1052)
  update zip_read_closer to incorporate zip64 support (anchore#1041)
  Add pacman (alpm) parser support (anchore#943)
  Update of README.md (anchore#1027)
  bump cosign to v1.9.0 to resolve reporting of GHSA-66x3-6cw3-v5gj (anchore#1025)
  add workflows to test new project automation (anchore#1023)
  improve LanguageByName and add unit tests (anchore#1034)
  Read Description from dpkg status files (anchore#996)
  Add announcement for Anchore OSS Virtual Meetup (anchore#1033)
  add main module field to go bin metadata (anchore#1026)
  Add filters to package cataloger (anchore#1021)
  change draft to false for release process (anchore#1016)
  ...

Signed-off-by: Christopher Phillips <christopher.phillips@anchore.com>
aiwantaozi pushed a commit to aiwantaozi/syft that referenced this pull request Oct 20, 2022
* add main module field to go bin metadata

Signed-off-by: Jonas Xavier <jonasx@anchore.com>

* udpate json ouput schema to 3.2.4

Signed-off-by: Jonas Xavier <jonasx@anchore.com>

* clean up fixture

Signed-off-by: Jonas Xavier <jonasx@anchore.com>
GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
* add main module field to go bin metadata

Signed-off-by: Jonas Xavier <jonasx@anchore.com>

* udpate json ouput schema to 3.2.4

Signed-off-by: Jonas Xavier <jonasx@anchore.com>

* clean up fixture

Signed-off-by: Jonas Xavier <jonasx@anchore.com>
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.

Specify the "main module" in Go binary metadata for packages

2 participants