Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GeanyLua: update for new Scintilla 3.10 #776

Merged
merged 1 commit into from
Sep 21, 2018

Conversation

b4n
Copy link
Member

@b4n b4n commented Sep 21, 2018

@Skif-off could you take a look, as you seem to be interested in GeanyLua?

This change is required for GeanyLua to build against current Geany master.

I'm not sure of the implication of the changes in messages arguments, but this is a GeanyLua API break as well so it might be alright anyway.

Geany 1.34 broke API by using Scintilla 3.10 that removes some
long-deprecated messages. Update the list of available messages
and version dependency accordingly.
@Skif-off
Copy link
Contributor

@b4n
Thank you for PR! I also did it two-three of weeks ago and got the same changes.

@b4n b4n merged commit dca2038 into geany:master Sep 21, 2018
b4n added a commit that referenced this pull request Sep 21, 2018
GeanyLua: update for new Scintilla 3.10
@b4n
Copy link
Member Author

b4n commented Sep 21, 2018

@Skif-off thanks for the review, and sorry not to have given you the time to submit yours ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants