Skip to content

pr11650 with fix for addon paths#11707

Merged
michaelDCurran merged 27 commits into
masterfrom
fixAbspathCalls
Sep 30, 2020
Merged

pr11650 with fix for addon paths#11707
michaelDCurran merged 27 commits into
masterfrom
fixAbspathCalls

Conversation

@michaelDCurran

@michaelDCurran michaelDCurran commented Sep 30, 2020

Copy link
Copy Markdown
Member

This is a replacement of pr #11650 containing all original commits, but with one extra commit that stops an infinite while loop in addonHandler.getCodeAddon when loading an addon that has translations (such as nvdaRemote).

… made absolute as soon as possible to protect against the current directory changing later on.

Also store NVDA's app dir in globalVars.
…n places where we can't be completely sure the path is absolute.
…in NVDAHelper and nvda_slave when loading NvDAHelperRemote.
…g them to stop an infinite while loop introduced in #11650
@michaelDCurran

Copy link
Copy Markdown
Member Author

for those reviewing, only commit 2ae3811 is new.

@michaelDCurran

Copy link
Copy Markdown
Member Author

I can successfully run this commit with NVDARemote. But I would welcome testing by others.

@codeofdusk codeofdusk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks okay to me.

@josephsl josephsl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

At least add-ons are working.

@michaelDCurran michaelDCurran merged commit 40e1d07 into master Sep 30, 2020
@michaelDCurran michaelDCurran deleted the fixAbspathCalls branch September 30, 2020 03:59
@nvaccessAuto nvaccessAuto added this to the 2020.4 milestone Sep 30, 2020
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