Skip to content

windows: don't mark with dllexport in static build#507

Merged
Youw merged 2 commits intomasterfrom
win-dllexport
Feb 22, 2023
Merged

windows: don't mark with dllexport in static build#507
Youw merged 2 commits intomasterfrom
win-dllexport

Conversation

@Youw
Copy link
Member

@Youw Youw commented Feb 14, 2023

Don't mark API functions with __declspec(dllexport) when building a static library on Windows.
Enforced by CMake builds.
For other builds a compile definition is available.

Related: #480

@mcuee mcuee added the Windows Related to Windows backend label Feb 15, 2023
@Youw Youw changed the title Win: don't mark with dllexport in static build windows: don't mark with dllexport in static build Feb 19, 2023
@Youw Youw merged commit 438d065 into master Feb 22, 2023
@Youw Youw deleted the win-dllexport branch February 22, 2023 00:05
Pen7art pushed a commit to Pen7art/hidapi that referenced this pull request Mar 15, 2025
* add prebuilds for arm64

* use prebuildify

* fix workflow

* fix workflow (again)

* try again

* remove accidentally committed prebuilds

* add support for hidraw variants

* fix hidraw only exists on linux

* add prebuilds and *.tgz to .gitignore

* replace readme badge

* rename `BINDING_NAME_RAW` to `BINDING_NAME_HIDRAW`

* re-enable tests

* use options from `binding-options.js`

* chore: update pkg-prebuilds

* chore: update github actions workflow

* run for prs

* update workflow

* fix alpine

* fix alpine

* fix alpine

* update readme

* trying pr 507

* Delete package.json.prebuildify

argh I am so bad at github today

* update github workflow

* update actions

* update publishing notes

* fix: macos python 3.11

---------

Co-authored-by: Hannes Rüger <hannesrueger@gmx.de>
Co-authored-by: Tod Kurt <todbot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Windows Related to Windows backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants