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 build with -Werror -Wdate-time #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

igalic
Copy link

@igalic igalic commented Apr 12, 2023

FreeBSD 14 adds -Werror -Wdate-time to its build which trips up on
opensm's build. Remove it from the code, as it doesn't really add much.

igalic added a commit to igalic/freebsd-src that referenced this pull request Apr 12, 2023
FreeBSD 14 adds -Werror -Wdate-time to its build which trips up on
opensm's build. Remove it from the code, as it doesn't really add much.

This fixes builds WITH_OFED_EXTRA.

PR: 270776
Upstream Pull-Request: linux-rdma/opensm#33
opensm/osm_console.c Outdated Show resolved Hide resolved
FreeBSD 14 adds -Werror -Wdate-time to its build which trips up on
opensm's build. Remove it from the code, as it doesn't really add much.
igalic added a commit to igalic/freebsd-src that referenced this pull request Apr 12, 2023
FreeBSD 14 adds -Werror -Wdate-time to its build which trips up on
opensm's build. Remove it from the code, as it doesn't really add much.

This fixes builds WITH_OFED_EXTRA.

PR: 270776
Upstream Pull-Request: linux-rdma/opensm#33
igalic added a commit to igalic/freebsd-src that referenced this pull request Apr 27, 2023
FreeBSD 14 adds -Werror -Wdate-time to its build which trips up on
opensm's build. Remove it from the code, as it doesn't really add much.

This fixes builds WITH_OFED_EXTRA.

PR: 270776
Upstream Pull-Request: linux-rdma/opensm#33
@hselasky
Copy link

hselasky commented May 1, 2023

This change looks good to me!

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