Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sbomnix: Remove command-line argument: --type #99

Merged

Conversation

henrirosten
Copy link
Collaborator

@henrirosten henrirosten commented Dec 11, 2023

Remove --type command-line argument from sbomnix replacing it with --buildtime argument to align the command-line usage with other tools in this repository. Also, update relevant documentation and tests.

Note: this change breaks the command line usage of the sbomnix tool for those users who use the --type argument, and it should be replaced as follows:

  • --type=buildtime should be replaced with --buildtime after this change
  • --type=both should be replaced with --buildtime after this change
  • --type=runtime is still the default mode (i.e. the absence of --buildtime)

- Remove `--type` command-line argument from sbomnix. Instead, change
  sbomnix so that it support `--buildtime` argument to align the
  command-line usage with other tools.
- Update relevant documentation and tests

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
@henrirosten henrirosten force-pushed the sbomnix-replace-type-with-buildtime branch from affcc40 to d451ca2 Compare December 11, 2023 14:20
@henrirosten henrirosten requested a review from a team December 11, 2023 14:24
@henrirosten henrirosten marked this pull request as ready for review December 11, 2023 14:24
Copy link

@mnokka-unikie mnokka-unikie left a comment

Choose a reason for hiding this comment

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

Changes are sound

@henrirosten henrirosten merged commit c243db5 into tiiuae:main Dec 13, 2023
@henrirosten henrirosten deleted the sbomnix-replace-type-with-buildtime branch December 15, 2023 05:58
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.

3 participants