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

Fix various MacOS build warnings #476

Closed
wants to merge 0 commits into from
Closed

Fix various MacOS build warnings #476

wants to merge 0 commits into from

Conversation

ashman-p
Copy link
Contributor

@ashman-p ashman-p commented Aug 9, 2024

Address a few warning see on MacOS M1.
oqsprovider-MacOS-warnings.txt

Fixes #467.

@ashman-p ashman-p requested a review from a user August 9, 2024 06:48
@ashman-p ashman-p marked this pull request as draft August 9, 2024 06:50
@ashman-p
Copy link
Contributor Author

ashman-p commented Aug 9, 2024

I am getting the following error running the formatter manually.
Can anyone give me some pointers to resolve?

@ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $
@ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $ find . -type f -and '(' -name '.h' -or -name '.c' -or -name '*.inc' ')' | xargs clang-format --dry-run --Werror
YAML:75:25: error: invalid boolean
AlignConsecutiveMacros: AcrossComments
^~~~~~~~~~~~~~
Error reading /workspaces/oqs-provider-fix-warnings/./.clang-format: Invalid argument
YAML:75:25: error: invalid boolean
AlignConsecutiveMacros: AcrossComments
^~~~~~~~~~~~~~

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM, except for the upgraded minimum cmake version.

CMakeLists.txt Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Aug 9, 2024

I am getting the following error running the formatter manually. Can anyone give me some pointers to resolve?

@ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $ @ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $ find . -type f -and '(' -name '.h' -or -name '.c' -or -name '*.inc' ')' | xargs clang-format --dry-run --Werror YAML:75:25: error: invalid boolean AlignConsecutiveMacros: AcrossComments ^~~~~~~~~~~~~~ Error reading /workspaces/oqs-provider-fix-warnings/./.clang-format: Invalid argument YAML:75:25: error: invalid boolean AlignConsecutiveMacros: AcrossComments ^~~~~~~~~~~~~~

What version of clang-format are you using?

@baentsch
Copy link
Member

baentsch commented Aug 9, 2024

I am getting the following error running the formatter manually. Can anyone give me some pointers to resolve?
@ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $ @ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $ find . -type f -and '(' -name '.h' -or -name '.c' -or -name '*.inc' ')' | xargs clang-format --dry-run --Werror YAML:75:25: error: invalid boolean AlignConsecutiveMacros: AcrossComments ^~~~~~~~~~~~~~ Error reading /workspaces/oqs-provider-fix-warnings/./.clang-format: Invalid argument YAML:75:25: error: invalid boolean AlignConsecutiveMacros: AcrossComments ^~~~~~~~~~~~~~

What version of clang-format are you using?

This may pertain to #473 (also a clang version related formatting issue).

@baentsch
Copy link
Member

Why did this get closed? The code looked like a valid fix and with the formatting update this should have passed CI, no?

@ashman-p
Copy link
Contributor Author

I was having problems getting coding formatting tests to pass on the branch. In my attempts to address that I sync'ed with your latest your changes in 'main'. That made things worst for the branch so, i decided to start over which closed the PR. Please stay tuned for a new PR.

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.

Various warnings building OQS Provider on Mac
2 participants