Skip to content

Convert to libedit so everyone can redistribute (second pass)#3868

Merged
andypugh merged 2 commits intoLinuxCNC:masterfrom
NTULINUX:libedit
Mar 19, 2026
Merged

Convert to libedit so everyone can redistribute (second pass)#3868
andypugh merged 2 commits intoLinuxCNC:masterfrom
NTULINUX:libedit

Conversation

@NTULINUX
Copy link
Contributor

libedit uses BSD, making it fully compatible with mixing with GPL.

Signed-off-by: Alec Ari <neotheuser@ymail.com>
@andypugh
Copy link
Collaborator

Should there also be a change to debian/configure?
https://github.com/LinuxCNC/linuxcnc/blob/master/debian/configure#L75
(Which looks like it will propagate into debian/control as a build-dep)
https://github.com/LinuxCNC/linuxcnc/blob/master/debian/control.top.in#L33

@rene-dev
Copy link
Member

I think this can also be removed:
https://github.com/LinuxCNC/linuxcnc/blob/master/src/configure.ac#L129-L138

Signed-off-by: Alec Ari <neotheuser@ymail.com>
@NTULINUX
Copy link
Contributor Author

Should there also be a change to debian/configure? https://github.com/LinuxCNC/linuxcnc/blob/master/debian/configure#L75 (Which looks like it will propagate into debian/control as a build-dep) https://github.com/LinuxCNC/linuxcnc/blob/master/debian/control.top.in#L33

https://packages.debian.org/trixie/libeditreadline-dev depends on libedit-dev so the actual package will automatically be installed if libeditreadline-dev is.

@andypugh andypugh merged commit 4fd0453 into LinuxCNC:master Mar 19, 2026
14 checks passed
@NTULINUX
Copy link
Contributor Author

Thank you so much, Andy!

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