Skip to content

(release_30)partialRevert: restore compilation order on profile loading#416

Merged
vadi2 merged 1 commit intoMudlet:release_30from
SlySven:(release_30)bugFix_restoreScriptCompilationOrder
Mar 12, 2017
Merged

(release_30)partialRevert: restore compilation order on profile loading#416
vadi2 merged 1 commit intoMudlet:release_30from
SlySven:(release_30)bugFix_restoreScriptCompilationOrder

Conversation

@SlySven
Copy link
Copy Markdown
Member

@SlySven SlySven commented Mar 10, 2017

In commit 1875075 (Pull Request #373) a change was made that reordered the compilation of the Lua scripts for Mudlet items. This commit reverts that aspect to restore the compilation of parent (e.g. folder/container) items before their children.

Signed-off-by: Stephen Lyons slysven@virginmedia.com

In commit-18750755c8a8acbb35a5dd65478333923da6a8e1 (Pull Request Mudlet#373) a
change was made that reordered the compilation of the Lua scripts for
Mudlet items.  This commit reverts that aspect to restore the compilation
of parent (e.g. folder/container) items before their children.

Signed-off-by: Stephen Lyons <slysven@virginmedia.com>
@SlySven SlySven self-assigned this Mar 10, 2017
@SlySven SlySven requested a review from vadi2 March 10, 2017 20:27
@vadi2
Copy link
Copy Markdown
Member

vadi2 commented Mar 11, 2017

Agreed, best to put the cat back in the bag and sort this out later.

This fixes Svof loading for me. @keneanung or @bunster, could you confirm the fix works for you as well? Cloning the https://github.com/SlySven/Mudlet.git repository and checking out the (release_30)bugFix_restoreScriptCompilationOrder branch would do the trick.

@macjabeth
Copy link
Copy Markdown
Contributor

👍 This also fixed the problem for me. Didn't run into any startup errors.

@vadi2
Copy link
Copy Markdown
Member

vadi2 commented Mar 12, 2017

Great, thanks for testing and the fix @SlySven . We move on! :)

@vadi2 vadi2 merged commit 52cf3e0 into Mudlet:release_30 Mar 12, 2017
@SlySven SlySven deleted the (release_30)bugFix_restoreScriptCompilationOrder branch October 12, 2017 04:02
@SlySven SlySven restored the (release_30)bugFix_restoreScriptCompilationOrder branch June 22, 2020 18:06
@SlySven SlySven deleted the (release_30)bugFix_restoreScriptCompilationOrder branch June 22, 2020 18:20
mehulmathur16 pushed a commit to mehulmathur16/Mudlet that referenced this pull request Feb 16, 2024
This fixes an error reported by Windows Application Verifier.
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.

3 participants