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

Lexilla license not installed #3126

Closed
elextr opened this issue Feb 13, 2022 · 7 comments
Closed

Lexilla license not installed #3126

elextr opened this issue Feb 13, 2022 · 7 comments

Comments

@elextr
Copy link
Member

elextr commented Feb 13, 2022

The lexilla license file does not appear to be installed with the default autotools build.

@elextr elextr mentioned this issue Feb 13, 2022
16 tasks
@eht16
Copy link
Member

eht16 commented Feb 20, 2022

The installed Scintilla license file ScintillaLicense.txt begins with:

License for Lexilla, Scintilla, and SciTE

Might this be enough?
If we are not sure, I agree we should install the Lexilla license additionally.

@elextr
Copy link
Member Author

elextr commented Feb 20, 2022

You expect me to read the License!!!! 😁 The licenses are the same for Lexilla and for Scintilla ATM, but they are separate files in different repositories so in theory could differ. So the suggestion was simply based on the fact that the meson build installed the Lexilla license file.

@kugel-
Copy link
Member

kugel- commented Feb 20, 2022

My meson PR includes a change so that the license is also installed for autotools builds

@elextr
Copy link
Member Author

elextr commented Feb 20, 2022

My meson PR includes a change so that the license is also installed for autotools builds

👍

@eht16
Copy link
Member

eht16 commented Feb 20, 2022

Ok. I'll have a look at the autotools part.

Edit: I wrote the above without properly reading what @kugel- said before. Sorry.

@eht16
Copy link
Member

eht16 commented Apr 10, 2022

I guess we can close this since Meson is merged and the Lexilla license is installed now also via Autotools.
geany_lexilla_license

@elextr
Copy link
Member Author

elextr commented Apr 10, 2022

Done

@elextr elextr closed this as completed Apr 10, 2022
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

No branches or pull requests

3 participants