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

CMake: fix time check typo (fixes build failure w/ GCC 12) #2493

Merged
merged 1 commit into from Jun 21, 2022
Merged

CMake: fix time check typo (fixes build failure w/ GCC 12) #2493

merged 1 commit into from Jun 21, 2022

Conversation

thesamesam
Copy link
Contributor

Without this fixed check, we get a build failure with GCC 12:

/var/tmp/portage/sci-chemistry/openbabel-3.1.1_p20210225/work/openbabel-08e23f39b0cc39b4eebd937a5a2ffc1a7bac3e1b/include/openbabel/obutil.h:65:14: error: ‘clock’ was not declared in this scope; did you mean ‘clock_t’?
   65 |       start= clock();
      |              ^~~~~
      |              clock_t

Bug: https://bugs.gentoo.org/851510

Without this fixed check, we get a build failure with GCC 12:
```
/var/tmp/portage/sci-chemistry/openbabel-3.1.1_p20210225/work/openbabel-08e23f39b0cc39b4eebd937a5a2ffc1a7bac3e1b/include/openbabel/obutil.h:65:14: error: ‘clock’ was not declared in this scope; did you mean ‘clock_t’?
   65 |       start= clock();
      |              ^~~~~
      |              clock_t
```

Bug: https://bugs.gentoo.org/851510
@welcome
Copy link

welcome bot commented Jun 12, 2022

Thanks for opening this pull request! Please check out our contributing guidelines and please examine any build issues.

@ghutchis ghutchis merged commit 2f34bda into openbabel:master Jun 21, 2022
@welcome
Copy link

welcome bot commented Jun 21, 2022

Congrats on merging your first pull request! 🎉 You're helping make Open Babel better for everyone!

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.

None yet

2 participants