Skip to content

Conversation

zeyus
Copy link
Contributor

@zeyus zeyus commented May 19, 2025

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 cboulay deleted the branch sccn:dev July 13, 2025 05:16
@cboulay cboulay closed this Jul 13, 2025
@cboulay cboulay reopened this Jul 13, 2025
@cboulay cboulay deleted the branch sccn:dev July 13, 2025 05:24
@cboulay cboulay closed this Jul 13, 2025
@zeyus
Copy link
Contributor Author

zeyus commented Jul 13, 2025

@cboulay is there something I should do differently for this PR? Or is it not functionality that you would be interested in adding?

@cboulay
Copy link
Collaborator

cboulay commented Jul 13, 2025

Hi @zeyus
Sorry, this one got auto-closed when I deleted a branch and I failed to retarget it. I'll try to fix that now.

@cboulay cboulay reopened this Jul 13, 2025
@cboulay cboulay changed the base branch from master to dev July 13, 2025 15:51
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.

2 participants