Skip to content
This repository was archived by the owner on Sep 7, 2021. It is now read-only.

Fixes for Windows-specific compilation issues#2

Merged
montegoulding merged 2 commits intomontegoulding:feature/gypify_lcidlfrom
livecodefraser:feature/gypify_lcidl-win_fixes
Oct 8, 2015
Merged

Fixes for Windows-specific compilation issues#2
montegoulding merged 2 commits intomontegoulding:feature/gypify_lcidlfrom
livecodefraser:feature/gypify_lcidl-win_fixes

Conversation

@livecodefraser
Copy link
Copy Markdown

Fixes the Windows compilation failures encountered when CI-testing livecode#3017.

On Windows, Gyp is sometimes too eager in interpreting arguments
to scripts as paths rather than literals. As I've not been able
to track down the source of this issue, a work-around in the
affected script is required.
montegoulding pushed a commit that referenced this pull request Oct 8, 2015
Fixes for Windows-specific compilation issues
@montegoulding montegoulding merged commit fb83987 into montegoulding:feature/gypify_lcidl Oct 8, 2015
Comment thread util/encode_source.pl
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Perhaps an explanation should be included as a comment in the script. This looks pretty odd otherwise.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Quite right! That's what happens when I decide to turn my quick-fix into a PR ;)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I'll add something to save you doing it @livecodefraser ... I've already merged the PR ;-)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

done

montegoulding pushed a commit that referenced this pull request Aug 16, 2016
montegoulding pushed a commit that referenced this pull request Oct 7, 2016
[[Dictionary]] item/last/lines/long/me variables
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants