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

Use libnitrokey directly instead of a static link #1

Open
szszszsz opened this issue Mar 25, 2023 · 0 comments
Open

Use libnitrokey directly instead of a static link #1

szszszsz opened this issue Mar 25, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@szszszsz
Copy link
Member

szszszsz commented Mar 25, 2023

Use libnitrokey directly to allow to remove unused parts of it by LTO. This would help to save space in Heads.
This is an alternative solution to #2, however this one should be preferred.

The flags below should contain these as well: -ffunction-sections -fdata-sections to make the gc-sections work.

STATIC_CFLAGS=-frandom-seed=0x12345678 -ffile-prefix-map=$(pwd)=nkstorecli -Wl,-gc-sections
STATIC_INCLUDES=-I$(INSTALL)/../crossgcc/x86_64-linux-musl/include
STATIC_LIBS=-L$(INSTALL)/lib -L$(INSTALL)/../crossgcc/x86_64-linux-musl/lib64 -lnitrokey -lhidapi-libusb -lusb-1.0 -lstdc++

@szszszsz szszszsz added the enhancement New feature or request label Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant