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

ofed: Fix build with -Werror -Wdate-time #715

Closed
wants to merge 1 commit into from

Conversation

igalic
Copy link
Contributor

@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.

This fixes builds WITH_OFED_EXTRA.

PR: 270776
Upstream Pull-Request: linux-rdma/opensm#33

@hselasky
Copy link
Contributor

What is the e-mail address (previously submitted by line) you would like to appear in the commit message?

@emaste
Copy link
Member

emaste commented Apr 12, 2023

What is the e-mail address (previously submitted by line) you would like to appear in the commit message?

It should just carry though from the pull request, see e.g. 0b0ae2e

commit 0b0ae2e4cd22c21f3c1971b1cfff9893b30d9f6f
Author:     Mina Galić <freebsd@igalic.co>
AuthorDate: Tue Mar 14 21:04:40 2023 -0600
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: Tue Mar 14 21:05:33 2023 -0600
...
    Pull Request: https://github.com/freebsd/freebsd-src/pull/663

@bsdimp
Copy link
Member

bsdimp commented Apr 12, 2023

Yea, git cherry-pick just does the right thing, except to add Pull Request: line (which you need to do manually)

@hselasky
Copy link
Contributor

Yea, git cherry-pick just does the right thing, except to add Pull Request: line (which you need to do manually)

Thank you. It is not so often I pull from github into the source tree.

@igalic
Copy link
Contributor Author

igalic commented Apr 12, 2023

removed the " build"

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
Copy link
Contributor

@hselasky hselasky left a comment

Choose a reason for hiding this comment

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

Who is pushing this?

freebsd-git pushed a commit that referenced this pull request May 1, 2023
This fixes builds WITH_OFED_EXTRA=YES .

PR:		270776
Pull Request:	#715
MFC after:	1 week
Sponsored by:	NVIDIA Networking
@hselasky
Copy link
Contributor

hselasky commented May 1, 2023

This pull request can be closed now!

@igalic
Copy link
Contributor Author

igalic commented May 1, 2023

merged in d502d3f

@igalic igalic closed this May 1, 2023
@igalic igalic deleted the fix/opensm-Wdate-time branch May 1, 2023 12:03
@emaste emaste added the merged label Jun 12, 2023
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Jul 20, 2023
This fixes builds WITH_OFED_EXTRA=YES .

PR:		270776
Pull Request:	freebsd/freebsd-src#715
MFC after:	1 week
Sponsored by:	NVIDIA Networking
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants