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

LDFLAG gets redefined in the code, though if it is set in the Makefile #30

Closed
praveenfff opened this issue Aug 14, 2022 · 4 comments
Closed

Comments

@praveenfff
Copy link

Sometimes the libmemif.so can be generated at a custom path rather than in the standard path /usr/lib.

extras/libmemif/adapter.go
extras/libmemif/error.go

Has the definition of LDFLAGS in the code which redefines the one which is set from Makefile
#cgo LDFLAGS: -lmemif

Could you please make this flexible so that it can be set via Makefile

@ondrej-fabry
Copy link
Member

ondrej-fabry commented Aug 19, 2022

I would like to note that the extras/libmemif is not maintained and most likely does not work at all anymore with any of the recent VPP releases.

There is however something similar implemented in the VPP repository that might provide the same functionality and it seems to be more recently updated: https://github.com/FDio/vpp/tree/master/extras/gomemif

P.S: I will update the README in the GoVPP's extras/libmemif to provide this info for users.

@praveenfff
Copy link
Author

praveenfff commented Aug 23, 2022 via email

@ondrej-fabry
Copy link
Member

I know, the Go code in the VPP repository was not implemented according to any Go standards or workflows.

Try adding go.mod file manually.

@dwallacelf
Copy link
Collaborator

This Issue is stale as the gomemif codbase has been moved from the VPP repo to this repo. The issue does not exist in the currently maintained codebase.

@dwallacelf dwallacelf closed this as not planned Won't fix, can't repro, duplicate, stale Dec 15, 2022
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

No branches or pull requests

3 participants