Skip to content

Conversation

@WolfGangS
Copy link
Contributor

This triggers a run of the preproc on a sync being established, populating the included file list so that if any one of them is saved the script is processed and sent to sl.

Copy link
Collaborator

@Rider-Linden Rider-Linden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is required. You should be able to pull any required files from the scriptSync.lineMappings member.

On initial subscription the plugin scans the file and generates an initial line mapping by looking for @line directives in the source.

See:
https://github.com/secondlife/sl-vscode-plugin/blob/develop/src/scriptsync.ts#L111-L119

and:
https://github.com/secondlife/sl-vscode-plugin/blob/develop/src/shared/linemapper.ts#L20-L65

@WolfGangS
Copy link
Contributor Author

WolfGangS commented Dec 4, 2025

That wont work, as the scripts on disk may have changed since the last time the one in world was edited.

We do not want what the script from SL "thinks" might be the relevant files. But the files on disk that the matched workspace script depends on.

Imagine you have updated a dependency, then go to save that again, and wonder why the script you just loaded hasn't re-saved in sl, because that wasn't included last time.

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