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

Fix compilation error due to std.xml deprecation #2160

Closed
wants to merge 1 commit into from

Conversation

cifvts
Copy link

@cifvts cifvts commented Apr 19, 2023

std.xml has been deprecated [1] and removed [2] from the standard library, therefor compilation failed due to the missing package.

As a temporary fix, add the undeaD package and use the xml library from it for the time being.

Fix #2152

[1] https://dlang.org/changelog/2.092.0.html#deprecate-xml
[2] https://dlang.org/changelog/2.101.0.html#remove-std-xml

std.xml has been deprecated [1] and removed [2] from the standard library,
therefor compilation failed due to the missing package.

As a temporary fix, add the undeaD package and use the xml library from it
for the time being.

Fix gnunn1#2152

[1] https://dlang.org/changelog/2.092.0.html#deprecate-xml
[2] https://dlang.org/changelog/2.101.0.html#remove-std-xml
@ximion
Copy link
Collaborator

ximion commented Apr 20, 2023

This patch doesn't work for Meson (which in turn is what pretty much all Linux distros use to build Tilix), and relying on undeaD is, well, a dead end ;-) - and another dependency for people using Tilix to package.

I've updated the code to not rely on std.xml and use GMarkup instead.
Thanks for the patch anyway, it is appreciated!

@ximion ximion closed this Apr 20, 2023
@cifvts
Copy link
Author

cifvts commented Apr 21, 2023

Make sense, looking for another library was the next step, I will look what you have done, and use Meson myself, and see if the README could be updated too.
Thanks

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.

error during compiling tilix
2 participants