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

Remove DS command #2113

Closed
wants to merge 1 commit into from
Closed

Remove DS command #2113

wants to merge 1 commit into from

Conversation

Mab879
Copy link
Member

@Mab879 Mab879 commented May 29, 2024

Keeping the code around as it used in other places in OpenSCAP.

Closes #2086

Keeping the code around as it used in other places in OpenSCAP.
@Mab879 Mab879 added this to the 1.4 milestone May 29, 2024
@jan-cerny
Copy link
Member

You will have to do something with failing CTest tests. Some of them will have to be removed, some of them will have to be changed so that they don't use the oscap ds subcommand.

For example, I suggest changing tests/bz2/test_bz2_datastream.sh so that it doesn't compose the data stream during the test but instead it evaluates a static data stream stored in the repository (you can use the removed command oscap ds sds-compose to generate the data 🙉 ).

@jan-cerny jan-cerny self-assigned this Jun 4, 2024
@Mab879
Copy link
Member Author

Mab879 commented Jun 20, 2024

Closing in favor of #2127

@Mab879 Mab879 closed this Jun 20, 2024
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.

Drop oscap ds feature
2 participants