Skip to content

Conversation

@zeyus
Copy link
Contributor

@zeyus zeyus commented Dec 5, 2025

Re-do of #234 but from a dedicated branch and targeting cboulay/apple_framework

On non-desktop platforms, often apps are sandboxed and don't have (read/write) access at all to $HOME, /etc/ or even the current working directory. These paths may not exist.

In addition, environment variables don't necessarily make a lot of sense in the context of a mobile app, and the application data paths might contain GUIDs or other aspects of the path that make it difficult to determine at compile-time.

This PR adds the ability to set a configuration file path for liblsl provided that the static member function lsl_set_config_filename is called before any other LSL function.

In addition, there's also an option to read config directly from a string stream via set_api_config_content which is loaded into the INI and discarded.

cboulay and others added 24 commits July 13, 2025 01:29
…ion that installed non-flat header tree by transitive dependencies.
…ee artifacts and thus were masking installation issues
@cboulay cboulay force-pushed the cboulay/apple_framework branch from 4f2d0da to d613e91 Compare January 10, 2026 23:27
…ion that installed non-flat header tree by transitive dependencies.
Initial support for Apple Frameworks

Move scripts into subfolder

Remove set(CMAKE_MACOSX_RPATH ON) because that is already the default value.

Fix framework install directory when not installing to the system.
…d for Ubuntu 18.04 which we no longer support)
…(temporarily disabled non apple runners for debugging)
mscheltienne and others added 25 commits January 11, 2026 12:24
…ion that installed non-flat header tree by transitive dependencies.
…d for Ubuntu 18.04 which we no longer support)
…(temporarily disabled non apple runners for debugging)
@zeyus zeyus force-pushed the feature/runtime-config branch from 2eeae9d to 14ffd08 Compare January 12, 2026 10:45
@zeyus
Copy link
Contributor Author

zeyus commented Jan 12, 2026

@cboulay It should now be updated, I just checked the diff and it looks good, no stray files anymore so there should be no conflicts merging to dev

edit: that said you might want to squash merge because of the rebase adding me to all the commits for some reason

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.

4 participants