8000 motordrvCom.h needs shareLib.h by tboegi · Pull Request #176 · epics-modules/motor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

motordrvCom.h needs shareLib.h #176

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 26, 2021
Merged

motordrvCom.h needs shareLib.h #176

merged 1 commit into from
Apr 26, 2021

Conversation

tboegi
Copy link
Contributor
@tboegi tboegi commented Apr 19, 2021

We can't compile motor against this commit of EPICS base:
commit 0f428ea3346d89719b03a6419319c85afb0fee13
Author: Michael Davidsaver mdavidsaver@gmail.com
Date: Thu Apr 1 10:57:19 2021 -0700

  use DBCORE_API

Solution: Include <shareLib.h>

@tboegi
motordrvCom.h needs shareLib.h
784e419 
We can't compile motor against this commit of EPICS base:
  commit 0f428ea3346d89719b03a6419319c85afb0fee13
  Author: Michael Davidsaver <mdavidsaver@gmail.com>
  Date:   Thu Apr 1 10:57:19 2021 -0700

      use DBCORE_API

Solution: Include <shareLib.h>
@anjohnson
Copy link
Member

Note for synApps maintainers: Any header file that contains an epicsShare... keyword should also contain a #include <shareLib.h> line as @tboegi is requesting here, you can't rely on the EPICS Base headers to include that for you any more. In Base we've switched to creating per-library macros for doing name decorations, run makeAPIheader.pl -h from EPICS 7.0.4 or later for more information about that. Recent Asyn releases use the new approach but don't create the asynAPI.h file using that script so they can build with older Base versions.

@kmpeters kmpeters merged commit 6d127cd into epics-modules:master Apr 26, 2021
@tboegi tboegi deleted the motordrvCom_h-needs-shareLib_h branch April 27, 2021 08:30
@kmpeters kmpeters added this to the R7-3 milestone May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
R7-3
Development

Successfully merging this pull request may close these issues.
3 participants
@tboegi @anjohnson @kmpeters
0